Interface lags while analysis jobs are open

The effect is noticeable when there are many expressions and the graph is clicked many times, though the latency is generally under a second.
Probably the most effective fix would be to only permit one analysis job to be open at any time, and cancel old jobs when a new analysis job is submitted.

This seems to be due to localerror jobs being sent for each of the active expressions. Maybe we should only send for expanded expressions.

Should be mostly solved by #167

There's no lag now, it's just a problem with waiting for the single request response to come back.


support Adaptive Banners

To support adaptive banner.  We need to expose GADCurrentOrientationAnchoredAdaptiveBannerAdSizeWithWidth to reactnative.

implemented now so closing issue


[RFC] Add Console (Symfony2 Component) to replace Phing

Hi there,
I'm currently looking for removing Phing stuffs by introducing the Console component. And here are some ideas to discuss.
build.properties
First, I wonder how to deal with build properties.

Do we need to parse build.properties file ?
Will we keep this file ?

IMO we cannot convert all build properties into console parameters, I just don't imagine what we'll need to type to build model classes…  The runtime as an XML file for the configuration.
So, we may use a generator-conf.xml to replace the build.properties existing file. That way, a command will load configurations from these XML files.
Here is the build.properties of the bookstore fixtures for instance:
<?xml version="1.0" encoding="UTF-8"?>
<generator>
    <project>
        <name>bookstore</name>
        <dir>.</dir>
    </project>
    <namespace>
        <autoPackage>false</autoPackage>
    </namespace>
    <schema>
        <autoPackage>false</autoPackage>
        <autoNamespace>false</autoNamespace>
        <autoPrefix>false</autoPrefix>
        <validate>true</validate>
        <transform>false</transform>
    </schema>
    <database>
        <platform>mysql</platform>
        <dsn>mysql:dbname=test</dsn>
        <user></user>
        <password></password>
        <schema></schema>
        <encoding></encoding>
        <tablePrefix></tablePrefix>
        <!--
            buildUrl ?
            createUrl ?
        -->
    </database>
    <mysql>
        <tableType>InnoDB</tableType>
    </mysql>
    <reverse>
        <samePhpName>false</samePhpName>
        <addVendorInfo>false</addVendorInfo>
        <addValidators></addValidators>
    </reverse>
    <model>
        <addGenericAccessors>true</addGenericAccessors>
        <addGenericMutators>true</addGenericMutators>
        <addTimeStamp>false</addTimeStamp>
        <addValidateMethod>true</addValidateMethod>
        <addHooks>true</addHooks>
        <basePrefix>Base</basePrefix>
        <classPrefix></classPrefix>
        <disableIdentifierQuoting>true</disableIdentifierQuoting>
        <targetPackage>bookstore</targetPackage>
        <packageObjectModel>false</packageObjectModel>
    </model>
    <datetime>
        <dateTimeClass>\DateTime</dateTimeClass>
        <defaultTimeStampFormat>Y-m-d H:i:s</defaultTimeStampFormat>
        <defaultTimeFormat>%X</defaultTimeFormat>
        <defaultDateFormat>%X</defaultDateFormat>
    </datetime>
    <behaviors>
        <test_all_hooks>\Propel\Tests\Helpers\Bookstore\Behavior\Testallhooksbehavior</test_all_hooks>
        <do_nothing>\Propel\Tests\Helpers\Bookstore\Behavior\DonothingBehavior</do_nothing>
        <add_class>\Propel\Tests\Helpers\Bookstore\Behavior\AddClassBehavior</add_class>
    </behaviors>
</generator>

Update: but we can also be smart by loading configuration (GeneratorConfig) from an array, and this array can be created from XML, YAML, INI, … That means, we can guess the generator-config file type by looking at its extension, and then to load the right loader.
Update2: Here is an extract of my work in progress: https://gist.github.com/f1e02b7411d016abaf33.
Should we rename some build properties ? At the moment, it's BC.
How to name the script ?
Another thing I would like to discuss is the name of the script. I suggest we call the console propel, that way we'll call it by running ./bin/propel or php bin/propel. Anyway, it's not important.
Doctrine puts it in the PATH to get a working doctrine command available everywhere (configured through PEAR).
Naming commands
Build stuffs:

model:build
sql:build

Migrations:

migration:up
migration:down
migration:status
migration:migrate [--up] [--down] // kind of alias

SQL:

sql:insert
sql:diff

Data:

data:dump
data:sql

Reverse:

schema:reverse

Tools:

tool:convert-conf
tool:graphviz

Usage
// will build the model for the current directory
./bin/propel build:model

// will build the model into /path/to/project
./bin/propel build:model --dir=/path/to/project

As usual, suggestions? Comments?  Ideas?
William

I like the idea of a configuration XML file.
It's/should be similar to e.g. PHPUnit. One should be able to overwrite a given setting from the file on a by-call-basis by passing it on the command line (for example a verbosity parameter, colored output, you name it). Furthermore it would be nice to have it working the same way, name.xml.dist gets overwritten by name.xml which gets overwritten by given command line parameters.
On the naming part, I like the propel binary and would also suggest a propel.xml and propel.xml.dist for configuration files related to this binary.

Just added an example of generator-conf.xml.

Please look at my new ideas (update sections).

I am ok with the suggestion.
Do we need to keep two different configuration files build/runtime, because database parameters are common ?
Maybe we could have one configuration file with 3 sections:
database
generator
runtime

My 2 cents:

No need for a build time config file at start. Design commands using required and optional parameters, plan sensible defaults as constants in the command classes to make the least possible required. Then, if it turns out some parameters are repeated for each command, we'll start thinking about passing a configuration object, which can be built based on a file. But that should come second.
For database connection, we must provide the ability to give separate settings for build time (devs) and runtime (web server), because the web server probably hasn't rights for table creation/drop.
For naming, we should use a common pattern like "object:verb". "sql:insert" fits but not "build:sql", which should be renamed "sql:build" - same for "build:model".
The configuration format is another debate. The current "runtime-conf" format is... verbose, to say the least. Let's wait until I finish refactoring the runtime configuration classes to see what's best.
I'm +1 for ./bin/propel for the command script.

That means the end of the build.properties file ?

Kind of runtime-conf.xml ?

agreed

4/5) ok

I am also ok 2,3,4,5

we need a config file some command are executed numbers of time in dev mode it should run without any parameters at all.
I don't want to pass the my custom model's dir every time, ...

Yes, there are a lot of build properties: builders, … So we need a config file.

Once again, I think we don't need a config file if we have sensible defaults. At least try to implement the Commands without one, just to focus on the important stuff.
For instance, the builders can be options of the commands and use the actual Generator builders as default values.
Let's wait to see what these config files really need to contain before deciding their format.

I agree, there will be tons of options to generate models…
In a first version, I'll keep only useful properties, we'll decide later.

A few questions about phing removal:
What replacement can we find for phing's Fileset, that was used to retrieve xml schemas in given directories ?
Should we introduce a new dependency to Finder component for example ?

Sure we'll add the Finder component for that, we need to remove all phing stuffs.

yeah, i think it's a good idea to to use this.

Almost done: https://github.com/propelorm/Propel2/compare/afd2845452...33f097df85

fiouu! amazing work @willdurand


[Bug]  Message.addReaction() crashes.

What's the bug?
Message.addReaction() crashes with of http error 411 when sending an emoji.
How do we reproduce it?
use Message.addReaction("flushed") (same if you specify ":flushed:").
What should have happened?
Instead of sending a reaction to the message, the it crashes deno completely.
What is actually happening?
PUT https://discord.com/api/v9/channels/*number*/messages/*another number*/reactions/flushed/@me returned 411
(unknown) undefined
      throw new DiscordAPIError({
            ^
    at BucketHandler.execute (https://deno.land/x/harmony@v2.0.0/src/rest/bucket.ts:217:13)
    at async BucketHandler.push (https://deno.land/x/harmony@v2.0.0/src/rest/bucket.ts:51:14)
    at async RESTManager.make (https://deno.land/x/harmony@v2.0.0/src/rest/manager.ts:236:12)
    at async RESTManager.put (https://deno.land/x/harmony@v2.0.0/src/rest/manager.ts:339:12)
    at async GuildTextChannel.addReaction (https://deno.land/x/harmony@v2.0.0/src/structures/textChannel.ts:90:5)

What versions you're using?

OS: Arch Linux (btw)
Deno: 1.13.0
Harmony: 2.0.0

Do you have anything to tell us more about the bug?
Mozilla docs say 411 means Length Required. I'm 90% sure i'm using the method right because this is a http error, not a TS or Deno error.

addReaction only works with unicode for default emoji.

Maybe we should see if we can make it work with aliases like :joy: etc.

But discord doesn't use unicode emoji, it uses aliases just like github on markdown. when I type : flushed : it turns into :flushed: .
Another thing, if you add a plus before it like +: flushed: it automatically becomes a reaction, not a new message. The bot does not do this, it sends the message with a plus and an emoji. This is not a problem, just an observation.
The problem is you can't send custom emojis as reactions with this. Eg. if a server has a :troll: emoji, you can't send it as a reaction using harmony.

Every custom emote is formatted as :guildid:name: or :name:guildid:
So that is how you would send custom emoji's.
Not sure how harmony goes with this. But I assume that that works

Every custom emote is formatted as :guildid:name: or :name:guildid: iirc.
So that is how you would send custom emoji's.
Not sure how harmony goes with this. But I assume that that works

Just tested it out, neither :guildid:name: nor :name:guildid: send the emoji, just the text. Just using :name: also does nothing.

Every custom emote is formatted as :guildid:name: or :name:guildid: iirc.
So that is how you would send custom emoji's.
Not sure how harmony goes with this. But I assume that that works

It's just <:AquaCry:784740386680733716> emote name and snowflake.

But discord doesn't use unicode emoji, it uses aliases just like github on markdown. when I type : flushed : it turns into 😳 .
Another thing, if you add a plus before it like +: flushed: it automatically becomes a reaction, not a new message. The bot does not do this, it sends the message with a plus and an emoji. This is not a problem, just an observation.
The problem is you can't send custom emojis as reactions with this. Eg. if a server has a :troll: emoji, you can't send it as a reaction using harmony.

Don't think bots have the +emoji shortcut.

Custom emojis are sent with <:troll:750129788332802080> and troll being the emojiname,<PHONE_NUMBER>32802080 being the id, and animated emojis sent with <a:wuthead:867501649966268476>, and regular emojis with their unicode variants, a bot is not a markdown textbox, please inform yourself on the internals of discord before making bug reports as no other framework does this.

But discord doesn't use unicode emoji, it uses aliases just like github on markdown. when I type : flushed : it turns into flushed .
Another thing, if you add a plus before it like +: flushed: it automatically becomes a reaction, not a new message. The bot does not do this, it sends the message with a plus and an emoji. This is not a problem, just an observation.
The problem is you can't send custom emojis as reactions with this. Eg. if a server has a :troll: emoji, you can't send it as a reaction using harmony.

Don't think bots have the +emoji shortcut.

Yeah they don't, it just sends the + as a message. the + thing is a feature of the discord app.

Custom emojis are sent with <:troll:750129788332802080> and troll being the emojiname,<PHONE_NUMBER>32802080 being the id, and animated emojis sent with <a:wuthead:867501649966268476>, and regular emojis with their unicode variants, a bot is not a markdown textbox, please inform yourself on the internals of discord before making bug reports as no other framework does this and it would be a lot of unnecessary work. And discord does use unicode emojis, aliases just turn the text into its unicode emote, put \ before any emote to get its unicode variant

What exactly to you mean by "id"? id of the user? server id? because the server id doesn't work. When I do what you said it sends the :troll: but without the "id". Yes, i'm not stupid, I changed the id to the server's id.
Also how am I supposed to know that?  I assumed how discord works because when i specify an alias in  Message.reply(), it sends it as an emoji. This only worked with default emojis, not with custom ones. The addReaction() method did not work with any aliases.
This is still a problem, because the documentation (if you can even call it a decumentation) says absolutely nothing. The automatic tooltip just said string | Emoji type, so i obviously assumed it expects an alias as a string or an emoji, as any normal person would. If you want to do anything useful you have to read the code of harmony because it's ~poorly~ not documented.

What exactly to you mean by "id"? id of the user? server id? because the server id doesn't work. When I do what you said it sends the :troll: but without the "id". Yes, i'm not stupid, I changed the id to the server's id.

The emoji's id.

Also how am I supposed to know that?  I assumed how discord works because when i specify an alias in Message.reply(), it sends it as an emoji. This only worked with default emojis, not with custom ones. The addReaction() method did not work with any aliases.

Read Discord's API docs, they dont exist so you can come here and complain: https://discord.com/developers/docs/reference#message-formatting

This is still a problem, because the documentation (if you can even call it a decumentation) says absolutely nothing. The automatic tooltip just said string | Emoji type, so i obviously assumed it expects an alias as a string or a unicode emoji, as any normal person would. If you want to do anything useful you have to read the code of harmony because it's poorly not documented.

While I agree that is an issue with harmony, which will be getting greater focus with the v3 rewrite, we do not need to repeat Discord's API Docs.

How did I not see this before? Thank you for the link, this looks very useful. All i could find is docs for other frameworks.
I completely forgot discord is a webapp and that you could just inspect element...
Also you do know the whole point of bug reports it to complain? Deno crashed when I tried to send an alias, that is a bug. At least i would consider it a bug, since it literally crashed. You need to take a break man, I can see something is bothering you (maybe school starting soon?) because you seem agitated. I asked for help and you have a problem with that. Well whatever is bothering you, I hope it ends well.
Good luck with the v3 rewrite, hopefully it's gonna be great!

This is (unfortunately) default behavior of JavaScript to crash on unhandled (promise) rejections. So you'll have to either await it and catch in a try/catch block, or .catch(...) it

Also you do know the whole point of bug reports it to complain? Deno crashed when I tried to send an alias, that is a bug. At least i would consider it a bug, since it literally crashed.

The whole point of bug reports is to report bugs/flaws in a program, this is clearly not a fault in Harmony as you used it wrong, would you say me writing
consol.log("hi")

and then Deno crashing is a bug in Deno? No, it's a bug in my code, same situation here

Original issue is answered, if you have additional suggestion please open a new issue.

This is (unfortunately) default behavior of JavaScript to crash on unhandled (promise) rejections. So you'll have to either await it and catch in a try/catch block, or .catch(...) it
Aliases resolving emoji names to unicode are not included because it would quite significantly increase the size of the package for something all may not even use. However, you can easily get unicode for emojis by adding a backslash before the emoji in Discord.

That's not really an issue, but custom emotes don't have unicode entries. You can't react with custom emotes with harmony like you can on the app.

For custom emotes you can either pass Emoji object, use the <a?:name:id> format or just name:id (note: this format only applies to reactions as per Discord docs).

Also you do know the whole point of bug reports it to complain? Deno crashed when I tried to send an alias, that is a bug. At least i would consider it a bug, since it literally crashed.

The whole point of bug reports is to report bugs/flaws in a program, this is clearly not a fault in Harmony as you used it wrong, would you say me writing
consol.log("hi")

and then Deno crashing is a bug in Deno? No, it's a bug in my code, same situation here

Wait, you program without an IDE? Because an IDE would definitely tell you about the error. What I did threw a http error, not a logic error. The IDE did not say anything was wrong with my code. Anyway, since i have the docs and some advice I think I can make something work. I don't usually do typescript, I'm a C++ kind of guy. Everyone tells me C++ is dying and I can't get a job with it but I still like it for it's performance.
Thanks for the help everyone. Again, good luck with v3!


deps: bump go-header

Depends on #799
Unifies VerifyAdjacent and VerifyNonAdjacent into Verify

@Wondertan https://github.com/rollkit/rollkit/blob/main/node/full_node_integration_test.go#L215 and 4 following tests

@nashqueue, cool. Let's merge this one, and I will make a separate PR/commit to remove those timeouts.

Ok, thanks for taking this and next time pls coordinate with me 🙏🏻


[wip] Rocksdb

This branch is a continuation of the degenless -> strings-attached -> trie-harder lineage. It preserves the move in earlier branches from integer to string keys, and in supporting prefix scans for building degen lists at runtime. It differs earlier branches in this family as follows:

sharding has been eliminated
the backend storage mechanism has been moved to RocksDB
as a consequence of the above, load and pack operations operate on an entire type, and operate on filenames of rocksdb database directories, not on buffers in memory

Concrete API changes vs earlier branches:

__get, __getbyprefix, _get, and _set now omit the previous shard argument
list had two modes before, a single-argument mode that would list all shards for a type, and a double-argument mode that would list all keys for a shard within a type. It now has a single mode that takes a single argument (a type) and lists all its keys (so, closer to the second behavior than the first).
loadSync's signature has changed from loadSync(buffer, type, shard) to loadSync(filename, type) where filename is a path to a RocksDB directory.
pack's signature has changed from pack(type, shard) to pack(filename, type). As above, filename is a RocksDB directory (that will be created if it doesn't exist). It now returns true upon success, rather than a buffer, as the packed output is now written to a file.
unload now accepts only a type and unloads an entire type rather than a single shard
likewise, has tests for the presence of an entire type

Next steps should be to adapt carmen@trie-harder to use this backend.
Misc other notes: some tests are commented out for now, so testing in general needs to be cleaned up a bit. Also, current build might be brittle, and assumes the presence of a system rocksdb. We may want to consider bundling, and also should figure out how this works with circle.
cc @ansis, refs https://github.com/mapbox/carmen/issues/549

@ansis moving from chat to here... so you had talked about a couple changes:

getting rid of the cache that stores rocksdb pointers since there are only two
getting rid of the in-memory cache altogether and only using rocksdb

It occurred to me after talking that I think I potentially made those harder with this commit: https://github.com/mapbox/carmen-cache/pull/61/commits/be950a82bdf1e9b373d62cb74cbd7d5cd3fdb311 though on balance I think it was a good change.
Reading a bit more of the rocksdb docs, there are two ways to open a rocksdb database, Open and OpenForReadOnly. The read only mode, besides disallowing writes, also skips lock acquisition, which I think will have two important implications for us at run-time with the geocoder: apparently (per code comments) it's much more performant for read-only workloads (which our query-time geocoder is), and it allows multiple processes to read at the same time, which I think we'll need to be able to do to continue to share our disk container amongst multiple geocoder containers, and which is disallowed with regular open.
So the commit changes loadSync (and also the accesses to the input databases in the merge operation) to be read-only calls, and only opens the databases in write mode during pack, and for the output database in merge, and in both cases doesn't retain this connections -- they get released at the end of the call so the same process or another process can loadSync them or merge them read-only.
So, a few ramifications for what you were thinking about: the create_if_missing option doesn't work for read-only databases (it's still in the code vestigially but I should delete it), so at carmen instantiation time, it now only conditionally calls loadSync if the database exists already as a result of a previous pack, rather than always calling it, which means index-time carmen probably won't have rocksdbs loaded at all, and even if it does, it will be loaded read-only when _set is called.
Even with all that aside... I'm pretty skeptical about the idea of getting rid of the memory cache. The packed form of grid entries is not ints, it's just deltas between consecutive ints in a sorted list, protobuf-encoded. The indexing process repeatedly calls _set with the append flag set to true, adding grids for each phrase that occurs in each batch of features, and if we were only storing the packed form, each _set call would need to un-protobuf each list, then expand out from deltas to full ints in a vector, then append to the vector, then sort, then re-delta-encode, then protobuf-encode again. I can't imagine this won't be dramatically slower than the current incarnation, but even if it isn't, it seems like a pointless change to me. Why not just hold onto the vectors during indexing and pack at the end like we do now?
I sort of suspect that the current implementation of constantly checking both caches feels hacky to you, and I'm sympathetic to that -- we know full well on the carmen side whether we're in indexing mode, in which case we should be able to operate exclusively on vectors in memory, or in runtime mode in which case we should be able to run exclusively from rocksdb/protobuf.
As an alternative refactoring strategy, you could consider a code structure sort of like what the DAWG cache has now, which is to have two different classes that expose a similar interface, one read/write version that's used at index time, and one read-only version that's used at query time that uses a more compact representation. That read/write version could either operate in memory, or use rocksdb with a different internal data representation that's more suitable to repeated appends (I see no particular benefit to the latter because it all fits in memory during indexing, but whichever), and it would have a pack method that would generate a read-ready rocksdb that could be used at query time that would have everything packed, and the transformation would happen once at the end of indexing. That way, the index-time version never has a rocksdb (or maybe does, but a different one, or something), and the read-only version always has both (so cacheHas can die), and can rely on it being there and in read-only mode, and their respective get/set/whatever methods don't have to constantly check both. I think this kind of change could make things a lot cleaner if you wanted to pursue it.
Overarching cautionary note, though: currently both carmen-cache and carmen tests pass, after quite a bit of work to get them there, and I'm pretty close to having a global rebuild done using the current carmen and carmen-cache. I suspect that a refactor like this (or however else you'd want to proceed) will probably break everything again and will take a bunch of work to re-fix in both repos, so if you wanted to go forward with it, I'd probably be inclined not to wait for it to be done before getting some version of the current work out the door... better to do it in a different branch and merge once you get everything back working again.

Also, forgot to mention: another possible sort of orthogonal next step is to kill the DAWG cache altogether. If checking for the existence of a key in rocksdb as as fast as or close to as fast as a DAWG cache lookup, there's probably not any point in having both, and we can probably significantly shave our runtime memory usage if we just retain the one. That would need to be evaluated, though, and again, I think the what's been done so far should go out first.


Deprecation messages are inaccurate

https://github.com/mapeditor/rs-tiled/blob/7c072c9fe5e612b6b3ed20397b6a5ab6b65c9f1b/src/map.rs#L86
Should be Loader::load_tmx_file. Happens at some other places too.

How did that pass our strict review process?


Modelling of splines and orthotropic materials

First of all: Thanks for making pycalculix an open source project!
I am looking for a python tool that is able to mesh splined based geometries such as rotor blade cross sections.
Further, handling of 3D orthotropic materials would be a requirement.
Especially the element/material coordinate system is important to be set so that the material orientation follows a 'major' spline of an area.
To give you an idea, I have attached an image of an exemplary mesh of a wind turbine rotor blade cross section where the element/material coordinate system is shown for one element in red.

It seems that pycalculix already offers a well developed framework interfacing to state-of-the-art meshing tools (gmsh and cgx) which could be extended to the above requests.
Before starting the implementation of new things into pycalculix, I would really appreciate your opinion about extending pycalculix towards:

Implementation of splines in pycalculix geometry modelling framework.
An implementation strategy to make material coordinate systems following a 'major' spline of an area.

Further, I wonder if there is a preference between gmsh and cgx for the extension ideas?

Hi, thanks for your interest in pycalculix.
Would you want your blade geometry to be plane stress, plane strain, or true 3d?
Pycalculix is already set up to do plane stress and plane strain.
What kind of loading would be on the airfoil? Pressure, self weight rotation forces?
Would you want to do a stress model or a heat transfer model?
Pycalculix only works in the stress/deflection domain right now.
If it were true 3d, calculix would need to be extended into 3d. Which is doable. It already stores multiple areas in a surface. A volume is multiple closed areas. It's just that plotting gets way more painful, and you need to worry about orientation etc..
I have some code where I was working on implementing a plotter object which allows different 2d orientations xy and yz, but I haven't posted it because I didn't finish it.
How one would best implement the features you mention would be to continue the wrapping of gmsh functionality in python. I would make a spline base class based on gmsh's spline or bspline class:
http://geuz.org/gmsh/doc/texinfo/gmsh.html#Lines
And I would also make a signed spline class. The signed spline class is needed to close a loop of lines or splines or arcs to make an area. Pycalculix already has a base line and arc class which are similarly built.
To plot the spline and closed area in pycaculix, you need to plot it using matplotlib. To draw the spline, you'd user vertices. I already use vertices to close arbitrary areas in python so that would be a fairly straightforward adaptation. The matplotlib page which describes plotting vertices is here:
http://matplotlib.org/api/path_api.html
See my plotting code where I plot a patch with vertices:
https://github.com/spacether/pycalculix/blob/master/pycalculix/geometry.py#L1573
How to make the material follow the major spline of the area.
I assume material orientation would be along the spline, normal to it, and normal to the page.
This may require a custom material for every element. One could approach this a couple ways. Maybe your area has a material generator function that takes location along spline, and normal distance from the surface and spits a material back out. You can loop through elements and generate one of these materials for every element. Or maybe you map mesh the spline, and make a material that is normal to a passed face. This is harder, and would require a perfect map mesh. The other option I mentioned feels easier.
Calculix material definition documentation is here:
http://web.mit.edu/calculix_v2.7/CalculiX/ccx_2.7/doc/ccx/node193.html
Preference on which mesher to use:
Definitely gmsh, more people use it, and it would allow a non-calculix solver to be plugged in more easily at a later date if the meshing format is separate from the solver. The gmsh meshing also provided more control when I compared its features to cgx. It is also easier to debug gmsh files with the gmsh gui.

@spacether: Thanks for your comments on the new features.
To your questions:
In a first stage I would like to use pycalculix only for very detailed 2D cross section mesh generation, which is then used to be passed to BECAS to (1) calculate beam cross sectional properties and also to (2) recover stresses and strains with the internal loads (forces and moments) from an FE beam model. BECAS is also already wrapped into Python via the BECASWrapper.
In a second stage we could indeed think of 3D splined based mesh generation (Shell or HEX20 elements) of whole blades using ccx or Elmer as solver to run benchmarks vs. Ansys or MSC Marc. This model would then be loaded by shear forces.


Add debugging instructions in PRs with failing Playwright tests

Description

This aims to give contributors guidance on how to debug Playwright tests failing in CI.
Testing Instructions

Testing in this PR. I've added a commit that will cause the tests to fail (changing a color).
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[N/A] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[N/A] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

It'd be nice if they could share a comment but I think it would cause a race condition...

I really like the idea of documenting the process of what to do if Playwright tests are failing.
One issue I still don't know how to solve correctly is updating the snapshots. The process I currently use is to run playwright tests, review the test-results contents, and if one of the -actual snapshots should be used, I rename it replacing -actual with -linux and copy-paste it to the correct snapshot directory. This is really manual and takes some time. I guess I could write a script to automate this, but maybe there is a better way?

One issue I still don't know how to solve correctly is updating the snapshots. The process I currently use is to run playwright tests, review the test-results contents, and if one of the -actual snapshots should be used, I rename it replacing -actual with -linux and copy-paste it to the correct snapshot directory. This is really manual and takes some time. I guess I could write a script to automate this, but maybe there is a better way?

Updating them is documented in the README, you just pass the -u flag.
This is documented here: https://github.com/WordPress/openverse-frontend/blob/a13e2a17deaddd53499400e59bd89a779a27f509/test/playwright/README.md#L24
Maybe it needs a clearer call out?
We could add a note in the comment that if the failures are in the visual regression tests and are expected to run the script from the readme.


i#el always live

Does that make sense to assign listening aspect instead? Eg. input with the indicated id changes on the page, but observable remains.

Moved to strui/input


Data restructure: Wallpaper and banners aren't displaying on album or track pages

(06 Mar 2022)

Fixed in 21c6220356b3769b6321bd64e85398fee618cfb1! (06 Mar 2022)


/rank info isn't implemented

What do we want this command to do? Just display the permissions?

The permissions and the ppl who have it?

Should it show online players or all players who have the rank?

@twizmwazin I think online players is fine, all players would get a little spammy. It should also show if the rank is a staff rank and if it's a default rank.

@iEli2tyree011 how about adding a /rank list player that displays the list of ranks the player has?


New reloader gremlin, targeted clicker gremlin, modifications to form filler gremlin

Added reloader gremlin that periodically reloads the browser window. Added a targeted clicker gremlin that clicks interesting elements on the page (buttons) more frequently. Modified the form filler gremlin to optionally bubble up input events to trigger angular form validation and watches.

Hey folks. The Marmelab team is attempting to clean up the Issues backlog. With that, we're closing issues that have been open for an one or more years.
We really appreciate the folks have taken the time to open and comment on this issue. Please don't confuse this closure with us not caring or dismissing your issue, feature request, discussion, or report. If the issue pertains to a bug, please re-test for the bug on the latest version of gremlins.js.
The best way to report problems with gremlins.js 2.0 is to file new issues.


Doc: Add more information on split_out argument used in DataFrameGroupBy methods

I ran into a memory problem when running ddf.groupby(...).size(). Coming across a similar StackOverflow problem, using ddf.groupby(...).size(split_out=NUM_PARTITIONS) fixed my problem.
Searching the Dask doc for split_out doesn't yield much information about this argument, nor do the DataFrameGroupBy methods in the API doc (I appreciate that most of these particular methods use the pandas doc).
[I'm unfamiliar with the split_every argument, but this also appears to be in a similar situation regarding documentation.]
Is it worth adding more information on these arguments? Where might the best place for this information be? My attempt at such documentation for split_out would be something like (I'm not actually sure my reasoning behind Dask defaults is correct?):

By default Dask DataFrame groupby methods return an Dask DataFrame object with only 1 partition. This is to optimize performance, assuming the groupby reduction returns a object that is small enough that is able to fit into memory. If your returned object is larger than this, you can increase the number of output partitions using the split_out argument, e.g. ddf.groupby("some_column").size(split_out=8).

Thanks for the feedback @TomAugspurger.
Can you point me to any examples using derived_from? Perhaps this is the cleanest way if it can be used so that the documentation for each groupby method includes the same API description as the dask.dataframe.core.apply_concat_apply you point out.

derived_from is used throughout dask.array and dask.dataframe. See DataFrame.pipe for example. Just to be clear though, this doesn't currently insert parameter descriptions anywhere currently. Updating it to do this might be somewhat difficult. It might be more feasible to have the docstrings include a reference to the prose section describing the parameters.

Thanks for the clarification.
I've created https://github.com/dask/dask/pull/6318 in an attempt to automate the documentation creation for methods containing the split_out argument, guided by your ideas whilst not over-complicating things:

Building the doc locally indicates it has generally worked as a expected:

Some of the generated API documentation have indentation misalignment that I couldn't understand why, hence I created this as a draft pull request (any feedback welcome :) ):

Closed by #6318


Removing Unnecessary Dependencies of gentellela

1 .The theme has various dependencies which we don't need now .
2. The features or all the working with gentellela will be in the form of reusable Angular Directives . For reference  TwitterWall has three directives , singletweet.js , width.js , backgrounddirective.js which are reusable .
@mariobehling , Do you have something more to add here . Please elaborate .

I am not clear on this. We also want to be able to copy/paste newer versions in future. Another point is, as Justin Lee mentioned, we can also use another bootstrap theme in future.
So, I would not really spent a lot of time on this right now. What we hope to see is - code that makes the app work. We can also clean up later.

@mariobehling  ,we have already discussed to make it work in that way , i.e the theme concept . I am clear with it . Let me add it to different Milestone than .


Only write @frozen on public interfaces

Motivation
Fixes a new warning in Xcode 16: @frozen has no effect on non-public enums
Modifications
When rendering enums, first check if the frozen modifier is required, then check if the access modifier is explicitly public. If both conditions are satisfied, render the @frozen modifier
Result
Frozen modifier will be in tact for public enums, but omitted for all other access levels
Test Plan
Integrated into my app, checked the warnings and they were fixed.

Before
After

Thanks @IanHoar! Can you also try with the package modifier? Does it produce a warning and should also be included in this condition or not?

@czechboy0
Since there were no changes in the PR https://github.com/apple/swift-openapi-generator/pull/587, I created a new PR https://github.com/apple/swift-openapi-generator/pull/595. Please let me know if there are any issues!

Already addressed in https://github.com/apple/swift-openapi-generator/pull/595


Add support for

Right now, we don't support the template tag correctly, and there are few issues filed regarding that.
Particularly, the tree builder and states need to be updated to meet the current spec. There's a concept of a template state stack.
As far as an implementation in jsoup goes:

parse -> serialize should be pass through (which is the main gap now - as it's unknown, template in head will boot the tree builder to the body state, and similarly within tables)
following the principle of least surprise, contents of a template element will be visible to selectors on the owner doc (so the contents of a template will not be contained in a document fragment; in script-enabled browsers they are so that script execution and layout doesn't expose them)
easy to use as actual templates via clone, modify, append, unwrap et al

Other than the basics, I'm keen to hear everyone's view on this and what could improve the functionality. And of use cases that you've come across that would be good to optimize for.

(Re-opening - while the core parser changes are implemented, I'm still happy to hear feedback here on how the API might be improved to help related use cases.)


Review adding historical accounts to whitehall

The instructions are still valid, so updating the review date.

Thanks. :star:


Themes

Personally I hate dark-mode, I would like to see the ability to set (at least) a light-mode, but, more customizable, themes, in general, would be nice.

Hi Herman! Thanks for the feedback!
Years ago, I remember I put some effort into creating themes.  Looks like they were buried with 1.x.x versions of WebTerminal. I guess it's not that big thing to do, but given that the project is not actively maintained right now, I can suggest either making PRs or wait until the maintenance (feature request work) is active again.
Thanks for filing this issue anyway!


[fix](load data) decommission replica don't load data when it misses versions

BUG:  when a decommission replica misses versions (last failed version > 0),   it will cause a bug:

It cann't do compaction due to it has miss versions;
It cann't  cloning the missing rowsets because it's decommissioned.

So if it continue to load data, its rowset will increase  util it cause error -235(TOO MANY VERSION).


YACN to do list

Easy stuff
Implement a new data augmentation method, e.g. black out a random block
Auxiliary inputs/outputs for floodfilling
semantics as inputs
affinity map as output
Check if regularization helps
Play with error localization windowing
Support 64-bit segmentation everywhere, and remove assumptions of consecutive seg-ids
streamline visualization process (pre-mesh volumes, activate flood-filling on client side, graph server)
Infrastructure
Reduce number of windows in error detection to speed up development
Make everything easy to use!
document everything!
Queue examples on gpu
Increase frequency of hard examples
Train from sparse/focussed annotations. Switch to precomputed data sources?
Set up a grid search for parameters
Improve diagnostics for go/no-go decisions for error correction inference
Error Detection
Distinguish merge and split errors
More fake errors. In particular, introduce image defects and propagate errors through to segmentation errors
Multiscale error detection
Training error detection seems to take unnecessarily long. Maybe we should pre-train on reconstruction loss and then fine-tuning on error detection loss.
Replace pixel-wise equality with earth mover's distance or Hausdorff distance
Error Correction
Accept flood-filling result only if discriminator approves
Choose flood-filling threshold to optimize discriminator approval
Generative adversarial training
Reduce coverage of error correction
Where possible, speed up CPU operations
Regiongraph
Produce a regiongraph with weighted edges instead of binary. This will be essential for Ran's cell splitting

Tensoboard output have now be visualize with neuroboard.py

train error detection to distinguish different type of errors
train the networks to actually distinguish the three cases:
obj is a superset of something in human_labels,
obj is a subset of something in human_labels
obj is empty

and 2) could happen simultaneously; this means a merge and split error in the same window.

@tartavull yacn do not have blending of output patches?


tiledb:// tuning

There are several places where we need to benchmark & tune tiledb:// (TileDB Cloud) performance and behavior in the Python package.  This issue tracks known issues:

[ ] the SOMAColleciton repr implementation, by default, walks the entire tree of sub-objects.  This makes it very sensitive to latency (e.g., on s3:// or tiledb:// URIs).  Possible fixes:  async or concurrent requests, or don't walk the entire tree.

This will be tracked within TileDB's tracking system, as it's implementation-specific


fix: ensure existing labels are retained for secrets in GCP secrets m…

Problem Statement
When managing existing secrets in GCP Secret Manager using ESO, the original labels on the secret gets removed. This is problematic as the secret here is initially created with KCC (Kubernetes Config Connector) which has the permissions allowed to create secrets. The label managed-by: external-secrets is also added so that ESO can manage secret versions.
With ESO we use PushSecret to add secret versions to an existing secret and it only has IAM role roles/secretmanager.secretVersionManager which allows it to create secret version and not update the secret at all.
The problem occurs when ESO reconciles and then does not take into account the existing labels already present on the secret and removes these labels causing an update to be triggerred, which causes an error as ESO does not have permissions to update the secret, but only to add secret versions.
This fix ensures that the existing labels are taken into account, thus ensuring there is no diferrerence between the existing secret and the operation that the PushSecret controller wants to invoke, ensuring that only a new secret version gets added.
Related Issue
Related to #4016 as the use case that was raised by the user rolandkool.
Proposed Changes
To ensure that the labels of the existing secret are merged with the labels coming from the PushSecret object spec.
Checklist

[x] I have read the contribution guidelines
[x] All commits are signed with git commit --signoff
[x] My changes have reasonable test coverage
[x] All tests pass with make test
[x] I ensured my PR is ready for review with make reviewable

hi @newtondev , thanks for your contribution. This change as is will be a breaking change for users, as we are changing behavior of PushSecret Metadata from Replacing content to Merge content. I'd honestly expect Replace to still be the defaults, as this is what we have everywhere else even on different API Sets 😄
IMO - you should add the proper field within the Metadata Struct:
type Metadata struct {
	Annotations map[string]string `json:"annotations"`
	Labels      map[string]string `json:"labels"`
	Topics      []string          `json:"topics,omitempty"`
	MergePolicy EnumOf<Merge|Replace> # Something Like this
}

And then add proper flow to control if MergePolicy is Merge or Replace. (making replace the default value, as to not cause breaking changes).
Also, please add some tests 😄

@gusfcarvalho Thanks for the suggestion, I do however see the kubernetes provider has a mergeTargetMetadata method that seems to do this - https://github.com/external-secrets/external-secrets/blob/main/pkg/provider/kubernetes/metadata.go#L84

@gusfcarvalho Thanks for the suggestion, I do however see the kubernetes provider has a mergeTargetMetadata method that seems to do this - https://github.com/external-secrets/external-secrets/blob/main/pkg/provider/kubernetes/metadata.go#L84

This is my point exactly. Instead of introducing a breaking change, we should implement something similar to what kubernetes provider does.

Added unit tests and documentation. Ready for review.

/ok-to-test sha=00fcbb6386c47507052c3ea809df0f2a43e87a7b

Thank you 🙏
On Wed, 11 Dec 2024, 06:51 Gergely Brautigam, @.***>
wrote:

Merged #4160
https://github.com/external-secrets/external-secrets/pull/4160 into
main.
—
Reply to this email directly, view it on GitHub
https://github.com/external-secrets/external-secrets/pull/4160#event-15611809506,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQYBEFNGZHNFCQ6WE4RSPT2E7HHPAVCNFSM6AAAAABSVN2LN2VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGYYTCOBQHE2TANQ
.
You are receiving this because you were mentioned.Message ID:
<external-secrets/external-secrets/pull/4160/issue_event/15611809506@
github.com>


ng config --global needs a project

Bug Report or Feature Request (mark with an x)
- [x] bug report
- [ ] feature request

Command (mark with an x)
- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [x] config
- [ ] help
- [ ] version
- [ ] doc

Versions

Node : 9.7.1
Npm : 6.3.0
Ng : 6.2.2
macOS High Sierra : 10.13.6

Repro steps
Install @angular/cli globally and run :
ng config -g cli.packageManager yarn
# or
ng config cli.packageManager yarn -g
# or
ng config cli.packageManager yarn --global

The log given by the failure
The command simply returns :
This command can only be run inside of a CLI project.

Desired functionality
Before, this command doesn't needs a project to apply a global config.

Thanks very much.It's help for me.


NVM  USE 0.10.36 NOT WORKING PLEASE HELP!!!!

when using nvm use 0.10.36 it reports c:/tools\elevate.cmd: file does not exist.
after i changed the root path from the default root which when using nvm list on that its always reporting cannot find pathe c:/users/SB..
Please Help.

oh update
finally fixed it. i had to re-install nvm and change the install path from my users directory to the c drive.
seems my pc name contains space .
thanks all


Publish a request id in a redis event

This is a feature request.
It's not always easy to associate a redis event with a specific request, e.g., when you don't know whether a put would result in a CREATED or UPDATED event.  It would be nice in a request to optionally provide a request id, which would then included in the published redis event to help with this problem.
thanks, Mitchell

Do you mean these events https://redis.io/topics/notifications ? If so I'm afraid I can't do anything here. Since they are generated on Redis side.

Ok, fair enough, that makes sense.


Created getById method in Activities.php

Added new method in Activities.php for returning details about Activity.
I also fixed few spelling mistakes (some methods had details misspelled like detials), and added throw in phpdoc for method add()

+1


Refuses to install

The repository shows up using the custom repository, but can't "download" fails. There is no "install button". A restart of HA just wipes everything for it out. Just loops around all the times I've tried it.
This error originated from a custom integration.
Logger: custom_components.hacs
Source: custom_components/hacs/base.py:730
Integration: HACS (documentation, issues)
First occurred: 3:40:35 PM (1 occurrences)
Last logged: 3:40:35 PM
Download failed - Got status code 404 when trying to download https://github.com/firstof9/ha-gasbuddy/releases/download/1.0.5/gasbuddy.zip
Traceback (most recent call last):
File "/config/custom_components/hacs/base.py", line 730, in async_download_file
raise HacsException(
custom_components.hacs.exceptions.HacsException: Got status code 404 when trying to download https://github.com/firstof9/ha-gasbuddy/releases/download/1.0.5/gasbuddy.zip
Also:
This error originated from a custom integration.
Logger: custom_components.hacs
Source: custom_components/hacs/repositories/base.py:986
Integration: HACS (documentation, issues)
First occurred: 3:40:35 PM (1 occurrences)
Last logged: 3:40:35 PM
<Integration firstof9/ha-gasbuddy> Failed to download https://github.com/firstof9/ha-gasbuddy/releases/download/tags/1.0.5/gasbuddy.zip

What's the error message in home-assistant.log say?

Just added the log messages see top post

Also this from log:
Logger: homeassistant.components.websocket_api.http.connection
Source: custom_components/hacs/repositories/base.py:990
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 3:40:35 PM (1 occurrences)
Last logged: 3:40:35 PM
[547257660736] Error handling message: Unknown error (unknown_error) RA Kish from <IP_ADDRESS> (Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:121.0) Gecko/20100101 Firefox/121.0)
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/websocket_api/decorators.py", line 26, in _handle_async_response
await func(hass, connection, msg)
File "/config/custom_components/hacs/websocket/repository.py", line 234, in hacs_repository_download
await repository.async_install()
File "/config/custom_components/hacs/repositories/base.py", line 893, in async_install
await self.async_install_repository(version=version)
File "/config/custom_components/hacs/repositories/base.py", line 990, in async_install_repository
raise HacsException("Could not download, see log for details")
custom_components.hacs.exceptions.HacsException: Could not download, see log for details

Please verify and if it's working, please close your issue. Thanks!

It is now working correctly from the download, adding integration, and configuring. Thank you


503 error

After all installation when i am running through server it shows after stop the recording.how to remove this error and follow the instructions on the server.

503 Service Unavailable

Service Unavailable
The server is temporarily unable to service your
request due to maintenance downtime or capacity
problems. Please try again later.

Apache/2.4.29 (Ubuntu) Server at localhost Port 80

See your apache error.log to narrow down the problem.

Thank you for your help.


how set moveSpeed and snapSpeed?

Is there a way to set moveSpeed and snapSpeed?

Animations aren't implemented yet


DOC: Include unrendered style notebook in git, execute in build

Did this for statsmodels the other week, works pretty well. Something like
jupyter nbconvert --ExecutePreprocessor.timeout=600 --to html --execute style.ipynb

https://nbconvert.readthedocs.org/en/latest/execute_api.html

yep - even better this actually catches errors
say you have a missing import
the only issue is if you have a deliberate exception (u need to put it in a markdown block instead)

This will have to wait on nbconvert 4.2,  which let's you specify the kernel when executing. I believe it's being released soon.

Closed by https://github.com/pydata/pandas/pull/13225


Applying a Shared Pretrained Model

Reproducibility on the training process is great, but for someone like me just looking for a way to use the model, is there anywhere the pretrained model is hosted? And if not, how would you recommend doing this?
The training process looks quite involved with many models, which makes me doubt whether the process is worth the increase in accuracy. A single training script that does everything might help also.

Hey Joshua,
if you just want to use the models, then you could start with this notebook. You'd need to download the models from Kaggle (links in README). However, I don't think it's very practical to use these models because it is an ensemble of more than 30 large transformer models. It will be way too slow for any real world application.
I'd use the same process but distill the knowledge into a single smaller model. You don't need to have as many teacher models as I used. See this paper for a more recent and streamlined version of the approach: https://arxiv.org/pdf/2208.09243.pdf

Thanks for the response and interesting paper! I've been looking around for a practical solution. Most of the solutions near the top seem to have the same problems in terms of being hard to use, which seems unfortunate given the goals of the competition, and my project is for students. Your success bringing in more data was inspiring nonetheless.
Fine-tuning a pretrained model for the type of data I'm working with seems like a practical solution, especially if I can leverage the same embedding across multiple similar tasks.


Flow indicators can not be part of local or shorthand tags

Spec: https://yaml.org/spec/1.2/spec.html#ns-tag-char
Test: https://github.com/yaml/yaml-test-suite/blob/master/test/WZ62.tml
The WZ62 test is passing now.
Also contained in this branch:
tests/run-parser-test-suite.c: Output error line/column
I will rebase this on #181 when merged.

Note: add a test to yaml-test-suite for block mode:
- !foo, xx


text-align: center with readme decorator

My component inherit text-align from storybook-readme decorator and it brokes my component style :(

Will change default wrapper for story at next release #118

Changed StoryPreview component https://github.com/tuchk4/storybook-readme/blob/master/packages/storybook-readme/src/components/Preview/StoryPreview.js
released at<EMAIL_ADDRESS>But note that right now there are breaking changes with API. I wrote about here #118 (comment).
Big chance that at release I will add support for previous api so no needs to change existing storybooks.

fixed in 5.0.0
Also in is possible to override story decorator to set your own styles.


Registering Faculty of Electrical Engineering in East Sarajevo to the List of Registered Educational Institutions in JetBrains

What this change does?

Registers Faculty of Electrical Engineering in East Sarajevo (Bosnia and Herzegovina) as an educational institution recognized by JetBrains in order for it's students to be able to apply for Student licenses.
Url to the Web Site of the University in East Sarajevo: https://www.ues.rs.ba/en/ues/
Url to the Web Site of the Faculty of Electrical Engineering in East Sarajevo: https://www.etf.ues.rs.ba/eng/
My student email is<EMAIL_ADDRESS>

@divdragana First stage of verification passed, official domain for The Faculty of Electrical Engineering of the University of East Sarajevo is student.etf.ues.rs.ba as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.


@_disfavoredOverload

A forum post reminded me that @_disfavoredOverload exists. I have not used it before, but maybe it can replace the ANKLargeBinaryIntegerWhereDigitIsNotSelf marker protocol that only exists to resolve overload ambiguity. If it applies, then ANKSigned x Self or ANKSigned x Digit may be marked as such.

Other than allowing ANKSigned<UInt> to build, the it may also make it possible to perform arithmetic with integer literals. I think that is an argument for favoring Self x Self methods, because there's few things more annoying than unexpectedly overflowing integer literals.

I chose to disfavor Self x Digit because the only time it matters is for integer literals. This means that integer literals will be interpreted as Self unless coerced as Digit.


Roundtrip for UMI data: fastq -> bam -> fastq -> bam

Do you see an option to allow for a full roundtrip of UMI data? We typically only archive .bam files, and bcbio nicely allows to use .bam and .cram files as input. Molecular barcodes are lost in the conversion to fastq at the moment as far as I can see. biobambam2 bamtofastq does not seem to provide such a feature even though it provides an option for tags. Therefore, reanalysis of .bam files with UMI in the RX tag is not possible - or is it?

Brad,
thanks for you input. So I will have to manipulate the fastq files directly to integrate the RX comment into the read name in the expected form (prefix and all), if I understand you correctly.
I will shelve this for now and probably tackle this once I am faced with re-running an analysis.


run vite:Multiple repeated errors are displayed

When you run vite local debugging and open the same page multiple times.
If a syntax error occurs, the command line error will appear multiple times, making it difficult to view.
Can it just happen once?

is this a real issue? you request the page 3 times s oyou will see the error 3 times

@remorses #987 prevents reporting an error more than once for the same source file + mtime

Fixed in 2.0

Fixed in 2.0


Update review date for PK Accessibility Statement

Changes date to merge date of last PR that changed Accessibility page.
We forgot to change the "last reviewed" date at that time.

@joelanman As requested by my line manager, I've updated the date when the statement was last reviewed. Can I get your review of the change? :)

@joelanman As requested by my line manager, I've updated the date when the statement was last reviewed. Can I get your review of the change? :)


Elaborate Readme for Hidden Functionalities

The readme should probably include info on the helper functions in the GoogleAuthService.js file, e.g., $logout/$login etc.
Let me know if you want me to PR this.

Hi @andrewpeters9,
Yes please. Your help will be much appreciated! :)

#7
Hope this helps or at least gives a starting point.

Yes Lundin, it helped. Thanks

I think we can close this issue.

Keep getting this error while trying to implement a simple solution...?
TypeError: this.$getGapiClient is not a function. (In 'this.$getGapiClient()', 'this.$getGapiClient' is undefined)
Didn't want to open a new issue as this is more of a request for additional information...
I'm finding it difficult to get the plugin to work with my app, do you have an example of an easy implementation you could add to the ReadMe?
methods: {
    login () {
      this.$getGapiClient()
        .then(gapi => {
          // gapi.sheets.spreadsheet.get(...)
          // ...
          gapi.sheets.spreadsheet.get({
          // gapi.client.sheets.spreadsheets.values.get({
            spreadsheetId: '1BxiMVs0XRA5nFMdKvBdBZjgmUUqptlbs74OgvE2upms',
            range: 'Class Data!A2:E',
          }).then(function(response) {
            var range = response.result;
            if (range.values.length > 0) {
              appendPre('Name, Major:');
              for (i = 0; i < range.values.length; i++) {
                var row = range.values[i];
                // Print columns A and E, which correspond to indices 0 and 4.
                appendPre(row[0] + ', ' + row[4]);
              }
            } else {
              appendPre('No data found.');
            }
          }, function(response) {
            appendPre('Error: ' + response.result.error.message);
          });
        })
    }
  },


Wrong link in first tutorial

Thanks for the nice tutorials. Just a small note that the patch based training link in the TorchIO_tutorial.ipynb file links to:
https://torchio.readthedocs.io/data/patch_based.html
Probably meant to link to:
https://torchio.readthedocs.io/patches/index.html

Fixed in https://github.com/fepegar/torchio-notebooks/commit/1e6ed9225bb82fdac62ab330b8b92b6a7d810b7d. Thanks, @sbdoherty!


Outlined variant not applying to Button

Current Behavior
Variant outlined isn't applying to Button like we can see in docs.

Expected Behavior
No response
Tamagui Version
1.74.15

Platform (Web, iOS, Android)
Web

Reproduction
https://stackblitz.com/edit/github-ojykqm

System Info
No response

same for me using 1.74.10

The stackblitz is failing due to missing webpack-serve for me, closing until we get reliable repro.

@natew I've created an repo with repro (with NextJs as base) -> https://github.com/ArekBartnik/tamagui-button-outlined-problem and deployed it here -> https://tamagui-button-outlined-problem.vercel.app
There's also codesanbox if you prefer https://codesandbox.io/p/github/ArekBartnik/tamagui-button-outlined-problem :)
Using Tamagui v1.75.5


[gl] add missing R/Rg/Rgba16Unorm format conversions

Hi gfx team!
In my project I'm using gfx_hal::format::Rg16Unorm, passing it to device.create_image(...) as a format at some point, however as I discovered there is not yet any conversion for that format defined in the gl backend in conv.rs (which later leads to a crash at runtime):
https://github.com/gfx-rs/gfx/blob/fdfe887f5afab7ca057e3e824817d0bcdcf794ba/src/backend/gl/src/conv.rs#L95-L101
I made a naive attempt to add a mapping myself, based on deductions from the other entries in the mapping table and eventually experimenting with different combinations, the last one I arrived at being the one in this draft PR:
 Rg16Unorm => FormatDescription::new(glow::RG16, glow::RG_INTEGER, glow::UNSIGNED_SHORT, 2, Float),

However every variant I tried leads to an Unsupported uniform datatype! panic at a later point, right here:
https://github.com/gfx-rs/gfx/blob/fdfe887f5afab7ca057e3e824817d0bcdcf794ba/src/backend/gl/src/queue.rs#L938-L945
I'd love to make myself useful here and provide a fully working patch but right now I fear I'm stumbling around in the dark a bit, so glad for any pointers how I could continue this!

PR checklist:

[x] make succeeds (on *nix)
[ ] make reftests succeeds

This currently fails elsewhere (in the Vulkan backend) so I can't tell right now:
Testing Vulkan:
	Scene 'vertex-offset':
thread 'main' panicked at 'attempt to shift right with overflow', src/backend/vulkan/src/command.rs:36:15

[ ] tested examples with the following backends:

Not working yet, see above, I'm testing this against a project that uses gfx-hal (https://codeberg.org/suchbliss/good-form), the examples won't likely cover this as this is about unimplemented features.

[x] rustfmt run on changed code

wow the reftests fail for you on Vulkan, please file an issue about this!

Will gladly file an issue on the failing reftests later! :)

I added the other two obvious (R/Rgba16Unorm) conversions to make this PR worth our while :)
I further investigated the "Unsupported uniform datatype" issue and it turns out this is apparently really something else, I would open another issue for that if you concur with that (to me it looks like either a bug or maybe something about the DescriptorType changes in 0.5.0 that's not entirely clear to me yet)

Opened an issue for the reftests failing at #3183

bors r=kvark,msiglreith


Make sure old adorner is removed.

Control enter/leave events seem to sometimes get raised in the wrong order, this needs investigation. However in the meantime, make sure that if there's an existing adorner it's removed before adding a new one.
Fixes #1299

Will that fix "unlimited blue layer works" issue in devtools?

Will that fix "unlimited blue layer works" issue in devtools?

What's that issue? Something like #1299?


Simplified Vendoring

Is your feature request related to a problem? Please describe.
The problem is an increase in the frequency of software supply-chain attacks (example).
Describe the solution you'd like
I'd like to use pipenv to vendor the wheels for the packages in my Pipfile.lock into a folder in my app (to be checked in).
I'd like to use pipenv to install wheels from the vendor folder (during the CI pipeline and after deployment)
Describe alternatives you've considered
A private pypi server that houses "blessed" packages.  I do not prefer this approach due to the overhead of maintaining a blessed list of packages and also of the private server itself.
Additional context
I do not have an opinion on how such a thing could be implemented, but I have an idea in my head of what I imagine.  Something like:
$ pipenv --python 3.7
$ pipenv install --vendor-dir=./vendor requests
$ git add .
$ git commit -m "adds requests to the app"
$ git push

so now I've vendored requests.
$ pipenv sync --vendor-dir=./vendor

and now I've installed from the vendor folder
I don't know.  Perhaps vendoring package-by-package is too complicated (and maybe defeats the purpose of vendoring).  Maybe it should be an all or nothing:
$ pipenv vendor --dir=./vendor on

just straight-up changes the [[source]] in Pipfile to the ./vendor dir
Don't know.  You all would know better.

I feel this is out of the scope of Pipenv and should be implemented in its own tool instead.

I feel this is out of the scope of Pipenv and should be implemented in its own tool instead.

I feel that vendoring is squarely within the packaging lifecycle and that pipenv's goal is to be the one-stop shop for python packaging.  Introducing a new tool would take us back to the days when we were using two programs (pip and virtualenv) for our packaging needs.
However, if it were its own tool (let's call it pipvend) then pipenv would - at a minimum - be asked to install packages from a vendored directory.
$ pipvend init
created wheelhouse in ./vendor
Pipfile updated to use wheelhouse as package source

$ pipvend install requests
downloaded requests-1.2.3.whl to wheelhouse
Pipfile updated
Pipfile.lock updated

$ pipenv sync
# notices that the Pipefile source points to a local directory
# and installs from the .whl files there

In this scenario, is pipenv already able to install from a local directory?

There's a lot of pipenv code that would be re-written in a separate pipvend  tool:

resolving a name to a pypa package
retrieving  and validating packages
resolving and retrieving dependencies
Updating Pipfile and Pipfile.lock

Probably more.  This is the hint that makes me believe that vendoring belongs inside pipenv.
$ pipenv --python 3.7 --vendor ./vendor
created wheelhouse in ./vendor

$ pipenv install  --vendor requests
# updates pipfile, downloads .whl to wheelhouse, installs package into venv, relocks

# check it all in
--
# deploy app somewhere
$ pipenv sync --vendor # installs from `./vendor` subfolder

☝ seems like the right thing.

FWIW I agree with @jar349 here, being able to vendor within pipenv would be excellent, and I'm surprised this isn't a feature already.
I'm currently looking to vendor packages for one of my projects that uses pipenv, and it's looking like the only way to do it at present is to go back to pip, which forces to re-evaluate our choice of using pipenv at all.

I'd have to add that for some newfangled serverless stuff such as AWS Lambda, there is no support for pipenv.
They only support regular pip for dependencies, and they basically use that through vendoring.
They run pip to download the dependencies and then those get bundled into dependency "layers" which are then used by the serverless function.
So far, so bad.
Now, with pipenv, it gets even worse. pipenv is unusable with AWS Lambda, because there is no obvious way to vendor dependencies. I'll probably have to revert to pip (it sucks, but in this case I'll have to choose Lambda over pipenv 🤷♂).

Now, with pipenv, it gets even worse. pipenv is totally unusable with AWS Lambda, from what I see, because there is no obvious way to vendor dependencies. I'll probably have to revert to pip (it sucks, but in this case I'll have to choose Lambda over pipenv ).

Maybe micropipenv can be helpful there?!

@oz123 it's been a few years since I opened this and a year since you closed it. How do you feel about re-opening to take the community's temperature again?

See pdm for a similar functionality.
I am personally not a fan of this approach. Also, pip uses a cache directory so blessed packages aren't downloaded. And finally running a local pypi server is docker pull away, and pipenv has good support for private repos.

I acknowledge your decision, so my comments below are my responses to your reasoning, not a request to reconsider. :-)

running a local pypi server is docker pull away

Running a local pypi sever in the GitHub prod network that is resilient enough to support all of our python deployments is not a docker pull away.  Also, not every part of our network can reach all the other parts due to data locality support, and for other reasons. So it's not one local pypi server, but many.  For us, for a number of reasons, vendoring is our best path.

pipenv has good support for private repos

This is something we wish to explicitly avoid. Going direct to "upstream" is the opposite of vendoring.


Add OSX 10.10 compatible firewall config

it's been a while, but It makes sense to make this change. I'm using pow with the nginx support, so I haven't had the issue with the firewall in yosemite


Utilise the db parameter passed in r.connect() and make overridable

Right now all queries that need to utilise a particular DB must start as r.db("db_name").table("table_name")...


Fix for issue#6

Fix for issue#6
Motivation and Context
This bypasses the normal .send(stream) format of fastify. Instead the template stream gets sent to the raw reply stream directly, fixing the issue described above.
Checklist:

[ ] I have updated/added documentation affected by my changes.
[ ] none of the above apply 🤷♂
[ ] I have added tests to cover my changes.

ok actually hold on before pulling, found another bug that happens somewhat randomly now.
Headers sometimes arent sent before the raw stream. I'll report back when i got a solution

I think the work around we discussed in #6 is good enough for now. If we can come up with some better way to warn or automate this though I'd for sure be interested!


run celery v5

Coverage remained the same at 95.139% when pulling 58e4a13603d5860aa1eaa886cb5c175a533e6673 on celery_v5 into 7d5970d00b0ce29914dbd69e5fe65694ef5ef4c7 on master.

Coverage remained the same at 95.139% when pulling 58e4a13603d5860aa1eaa886cb5c175a533e6673 on celery_v5 into 7d5970d00b0ce29914dbd69e5fe65694ef5ef4c7 on master.


Stop testing for Oracle JDK in CI

The differences between Oracle JDK and OpenJDK are minor starting from version 11. It is probably wasteful to be running both of them in Travis.
More background on the differences: https://www.baeldung.com/oracle-jdk-vs-openjdk
Over the past few years I do not recall encountering any bug in OpenRefine which would only be present with one of OracleJDK and OpenJDK. Running many builds at the same time increases the chances that we run into spurious build failures and can increase the build time if the builds are not run concurrently. So I propose we remove oraclejdk11 from Travis.

I agree it doesn't make sense to test multiple JDK 11s. We currently test oraclejdk11 and openjdk 8, 11, and 12. According to that article the Oracle JDK LTS releases are 8 and 11 and the openjdk policy is to drop support of the previous release when a new release comes out. Oracle JDK 8 LTS support officially ended in January, but commercial customers can buy support through at least December 2012.
Based on that, it seems like a better mix might be:

oraclejdk8 (presuming that we can get it on Travis CI)
openjdk 11
openjdk 12

Certainly there doesn't seem to be any value in testing openjdk8 since it's many versions out of support -- and even the oraclejdk8 is on its last legs.

I think setting up oraclejdk8 is harder than openjdk8 - but I would be happy to have the list you propose in principle.

I was able to find an oraclejd8 on trusty and got rid of the duplicate JDK 11 build, but then I realized that we didn't have JDK 13 or 14 and also didn't have an OS X build, so I ended up adding more builds than I removed. I think it'd be worth adding Early Access Program (ie JDK 16) builds, but jacoco doesn't support it.
You can see an example run here: https://travis-ci.org/github/OpenRefine/OpenRefine/builds/704530492
It currently has 6 jobs, which are, in order:

openjdk14, bionic
oraclejdk11, bionic
bundled java, os x
oraclejdk 8, trusty
openjdk 13, bionic
openjdk 12, xenial

If we can pare it to 5 jobs that will shave a couple of minutes off the build (6.5 min vs 9 min). Dropping the OS X build would get it under 5 minutes.
Having set all this up though, I'm wondering how necessary it is, particularly if we bundle a JRE on all platforms. Perhaps all we need is the Java version that we build our distributions on and future versions (so that we can spot coming compatibility issues).

Trimmed one more job to get it to 5 jobs which finish in 6-7 minutes with the first job finishing in 3-4 minutes.

I don't want to lose track of this question:

Having set all this up though, I'm wondering how necessary it is, particularly if we bundle a JRE on all platforms. Perhaps all we need is the Java version that we build our distributions on and future versions (so that we can spot coming compatibility issues).

@wetneb What are your thoughts? Do we have any Linux distros building their own bundles which need support for older JREs? Is there any other reason to continue testing them or should we focus primarily on current and future JDKs?

Well, that doesn't seem to have improved things. The Mac failure can probably be fixed with a delay to wait for the database services to start up, but I'm not sure what to do about the Java installation failures. I wonder if using the bundled JDK for any given distro would help? I'll do some more investigation...

I am sadly not aware of any linux distros packaging OR (and I feel bad about not having done that yet!). I think it makes sense to keep supporting relatively old JREs as long as this does not impede our work. So far, supporting Java 8 has never been a blocker for me, and there might be people out there who are stuck with it, so I would intuitively keep it in the CI by default.
I am really not an expert on these topics but perhaps we could just test with LTS versions, so Java 8 and 11? At the moment our main need is to reduce spurious build failures IMHO, I would reduce the number of combinations overall.

I've pushed a couple of tweaks. Let's see how they do on my PR CI builds.

Added a sleep for Mac database startup
Tried to use default JDKs for the build platforms
Made some of the problematic builds optional

This will focus on three: Oracle JDK 8 on trusty, OpenJDK 11 on Bionic, Oracle JDK 14 on Mac

Thanks a lot Tom!

@tfmorris I wonder if it might be worth moving all of CI/CD to Github Actions (Azure) instead of Travis which seems flaky at times?


How to control quadruped flat walk task with python bindings

Hi I'm a student and I am looking into the repository to control a unitree go1 robot, but first I am looking into this repository for a controller of an unitree a1 robot because of this video, which looks amazing! This is the code so far just to test out the walk of the robot
import time
from absl.testing import absltest
import grpc
import mujoco
import mujoco.viewer as viewer
from mujoco_mpc import agent as agent_lib
import numpy as np
import pathlib

m = mujoco.MjModel.from_xml_path("mjpc/tasks/quadruped/task_flat.xml")
d = mujoco.MjData(m)
path = pathlib.Path(agent_lib.__file__).parent / "mjpc" / "ui_agent_server"

agent = agent_lib.Agent(
   server_binary_path=pathlib.Path(agent_lib.__file__).parent
        / "mjpc"
        / "ui_agent_server",
   task_id="Quadruped Flat",
   model=m

)

agent.set_mode("Walk")

with mujoco.viewer.launch_passive(m, d) as viewer:

    start = time.time()
    while viewer.is_running() and time.time():
        agent.step()
        mujoco.mj_step(m,d)

With the code above I get this Screencast from 03-11-23 14:12:50.webm. Is there something missing here? I am having a little trouble understanding the documentation.
I installed the python bindings as stated on the read me page, only changing the install for the develop.
python "${MUJOCO_MPC_ROOT}/python/${API}.py" develop

Do you see that vertical orange line in the first two plots on the right?  That line indicates the effective lag between planning and acting.  If the plan is based on a very stale state, the chances are that it won't yield effective actions.  First, try to play with simpler domains (e.g., cart-pole) and verify that in the limit of fast planning everything works as expected.  Then, try to slow down the simulation (with the key "-") and see that the orange line comes closer to the middle of the plot.  This should result in better planning.  If that's where you end up (and there's no other problem at play), the only fix is to use more CPU cores...


Fix high code scanning vulnerabilities

Signed-off-by: Meredith Lancaster<EMAIL_ADDRESS>
Summary

Release Note

Documentation

Codecov Report

Merging #220 (9c4ac67) into main (c9d1b8c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #220   +/-   ##
=======================================
  Coverage   48.05%   48.05%
=======================================
  Files          19       19
  Lines        1130     1130
=======================================
  Hits          543      543
  Misses        529      529
  Partials       58       58

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Uncaught Error: Cannot read property 'object' of undefinedTypeError: Cannot read property 'objec...

Fired randomly today, don't really know why and cannot reproduce unfortunately
I was writing this string: const sprintf = require('sprintf-js').sprintf; and saved, probably my JS linter caused the editor to throw an error.
Atom Version: 1.7.3
System: linux 4.5.1-1-ARCH
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
at ObjectsRegistry.get (/usr/share/atom/resources/atom.asar/browser/objects-registry.js:44:28)
at EventEmitter. (/usr/share/atom/resources/atom.asar/browser/rpc-server.js:318:31)
at emitMany (events.js:108:13)
at EventEmitter.emit (events.js:182:7)
at EventEmitter. (/usr/share/atom/resources/atom.asar/browser/api/web-contents.js:144:25)
at emitTwo (events.js:87:13)
at EventEmitter.emit (events.js:172:7)
At /usr/share/atom/resources/atom.asar/renderer/api/remote.js:170

Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
    at ObjectsRegistry.get (/usr/share/atom/resources/atom.asar/browser/objects-registry.js:44:28)
    at EventEmitter.<anonymous> (/usr/share/atom/resources/atom.asar/browser/rpc-server.js:318:31)
    at emitMany (events.js:108:13)
    at EventEmitter.emit (events.js:182:7)
    at EventEmitter.<anonymous> (/usr/share/atom/resources/atom.asar/browser/api/web-contents.js:144:25)
    at emitTwo (events.js:87:13)
    at EventEmitter.emit (events.js:172:7)
    at metaToValue (/usr/share/atom/resources/atom.asar/renderer/api/remote.js:170:13)
    at BrowserWindow.remoteMemberFunction [as isFullScreen] (/usr/share/atom/resources/atom.asar/renderer/api/remote.js:110:18)
    at ApplicationDelegate.module.exports.ApplicationDelegate.isWindowFullScreen (/usr/share/atom/resources/app.asar/src/application-delegate.js:103:40)
    at AtomEnvironment.module.exports.AtomEnvironment.isFullScreen (/usr/share/atom/resources/app.asar/src/atom-environment.js:588:39)
    at AtomEnvironment.module.exports.AtomEnvironment.serialize (/usr/share/atom/resources/app.asar/src/atom-environment.js:775:26)
    at /usr/share/atom/resources/app.asar/src/atom-environment.js:984:27

Commands
     -2:42.7.0 core:move-up (atom-text-editor.editor.is-focused)
     -2:42.5.0 snippets:next-tab-stop (atom-text-editor.editor.is-focused)
     -2:42.5.0 snippets:expand (atom-text-editor.editor.is-focused)
     -2:42.5.0 editor:indent (atom-text-editor.editor.is-focused)
  4x -2:39.9.0 core:backspace (atom-text-editor.editor.is-focused.autocomplete-active)
     -2:34.5.0 core:paste (atom-text-editor.editor.is-focused)
  2x -2:33.3.0 core:save (atom-text-editor.editor.is-focused)
     -1:32.4.0 core:backspace (atom-text-editor.editor.is-focused)
     -1:31.5.0 core:save (atom-text-editor.editor.is-focused)
     -1:29.5.0 editor:newline (atom-text-editor.editor.is-focused)
     -1:29.1.0 core:save (atom-text-editor.editor.is-focused)
     -0:55.1.0 editor:newline (atom-text-editor.editor.is-focused)
     -0:44.1.0 core:save (atom-text-editor.editor.is-focused)
     -0:13.4.0 editor:newline (atom-text-editor.editor.is-focused)
  9x -0:11.9.0 core:backspace (atom-text-editor.editor.is-focused.autocomplete-active)
     -0:03.2.0 core:save (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "one-light-ui",
      "atom-light-syntax"
    ]
  },
  "editor": {
    "fontSize": 13,
    "invisibles": {}
  }
}

Installed Packages
# User
Quick-JavaScript, v0.15.0
atom-beautify, v0.29.7
atom-jade, v0.3.0
javascript-snippets, v1.2.1
jshint, v1.8.5
markdown-preview-plus, v2.4.0

# Dev
No dev packages

Nevermind, #11692 is the same issue as mine. Closing


update a few gems

we needed to have a new version of net-ldap because of
CVE-2017-17718
this needed an update for devise_ldap_authenticatable
Signed-off-by: Maximilian Meister<EMAIL_ADDRESS>

 (3) [security] update net-ldap because of CVE-2017-17718

lgtm.

looks like either devise-ldap_authenticatable or our way of using the ldap code in velum is slightly incompatible with the new gems. i suspect it's a relation between net-ldap and devise-ldap_authenticatable

looks like either devise-ldap_authenticatable or our way of using the ldap code in velum is slightly incompatible with the new gems. i suspect it's a relation between net-ldap and devise-ldap_authenticatable

after all it's the way we generate the certs, submitted https://github.com/kubic-project/caasp-container-manifests/pull/153 to fix it


⚠ Bofrost.fr has degraded performance

In 3fc6bc2, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1147 ms

Resolved: Bofrost.fr performance has improved in 5b5b96d.


Full backwards compatibility with Vectorize and NamedVariables

Undoes __init__ speedups to implement exactly what we had before, because some models using LinkedConstraintSet rely upon it.
@whoburg this is a priority to merge, please review.

@mayork this is on master now


How can update Configuration

When I modify appsetting.json file,
I have call builder.reload() mathod and Configuration data has updated.
but when use DependencyInjection to get IOptions ,
the value has not update.
so ,how can update Configuration on runtime?

I don't think it's documented yet, but I think you are supposed to to use  IOptionsMonitor<TOptions>. You can see how to use it in this test.
Unfortunately, if you are injecting IOptions<TOptions>, you won't get an updated value after config changes because IOptions<> is registered as a singleton. So, as of right now, you have to use IOptionsMonitor<TOptions> and set up the OnChange callback.
I'm not sure what the recommendation is for getting the most up-to-date TOptions when the OnChange callback isn't already set up. @HaoK thoughts? Did I get anything wrong?

Yeah that's right, you basically have to choose in advance that you want to listen for options changes by using IOptionsMonitor<TOptions>  You won't be able to see any changes via IOptions.


Add StateChangeMetadata to utxo operations for staking rewards

Current dependencies on/for this PR:

feature/proof-of-stake

PR #1022   👈

This stack of pull requests is managed by Graphite.


AKS creation command improvement

[Enter feedback here]
At step https://learn.microsoft.com/en-us/azure/aks/learn/tutorial-kubernetes-workload-identity#create-aks-cluster :

Suggestion: run az upgrade to ensure --enable-workload-identity is recognized
Correction/addition: on ubuntu WSL, this --generate-ssh-keys must be added to the command to create the AKS cluster. (azure-cli 2.47.0)

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

ID: efde8151-ca72-0ce1-271a-57a14a8c7ccd
Version Independent ID: f37efaf0-6573-6813-a3e4-3065e44666e4
Content: Tutorial - Use a workload identity with an application on Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/learn/tutorial-kubernetes-workload-identity.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

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

@gopher194
I'm going to assign this to the document author so they can take a look at it accordingly.
@MGoedtel
Can you please check and add your comments on this doc update request as applicable.

@gopher194 - Thanks for your suggestions. For the first point, we already specify that you need to start with a particular version of CLI or newer that includes support for workload identity, so I'm not sure why it's necessary to go further to suggest running Az upgrade. We've opened a PR to update the tutorial article so that it includes the --generate-ssh-keys parameter to the create cluster command and I'll be opening another PR to update the Deploy and configure cluster with Workload Identity cluster article to do the same.
See PR - https://github.com/MicrosoftDocs/azure-docs-pr/pull/239175.
I'm going to proceed with closing this PR. #please-close.


iOS: Can't exit video from Safari View Controller

"When I open a full screen video from the safari in app browser, I cannot tap the done button and leave the video. Have to force quit the app. Either rotation positions have this problem. "

I can't reproduce this one.  The Done button is always available when expected.  I don't know how it could fail to work, as the Safari View Controller takes over the UI while displayed.
Possible they are referring to the Done button of the Safari View Controller instead of the full-screen video, and don't realize they have to scroll to the top to reveal it?


Sorry, i can't find the tang9k directory, is this exist?

I almost forgot about it. Thank u, I will update it.

I almost forgot about it. Thank u, I will update it.

OK, thanks!

Updated.


fix: scheduling flake due to nodepool readiness

Fixes #N/A
Description
The NodePool readiness controller checks for updates to NodeClasses based on their APIVersion along with the kind and name. Since this NodePool did not set the group, if the nodeclass was not ready before the first NodePool readiness reconciliation it would not be rereconciled and the NodePool would remain unready.
How was this change tested?
make e2etests
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 79.023%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
93.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
5967

Relevant Lines:
7551

💛  - Coveralls


Add stdev function for symbolic dists

Simply adds some stdev bindings for some missing symbolic distributions.

Codecov Report

Merging #1494 (ba0497d) into develop (1e98593) will increase coverage by 14.19%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##           develop    #1494       +/-   ##
============================================
+ Coverage    58.89%   73.09%   +14.19%
============================================
  Files           91       93        +2
  Lines         4968     5017       +49
  Branches       816      817        +1
============================================
+ Hits          2926     3667      +741
+ Misses        2038     1344      -694
- Partials         4        6        +2

Impacted Files
Coverage Δ

__tests__/helpers/distHelpers.ts
96.15% <0.00%> (ø)

__tests__/fixtures/distFixtures.ts
100.00% <0.00%> (ø)

__tests__/helpers/reducerHelpers.ts
94.87% <0.00%> (+2.56%)
:arrow_up:

src/library/registry/helpers.ts
96.38% <0.00%> (+3.61%)
:arrow_up:

src/dist/DistError.ts
45.65% <0.00%> (+4.34%)
:arrow_up:

src/utility/result.ts
90.00% <0.00%> (+5.00%)
:arrow_up:

src/dist/SampleSetDist/SampleSetDist.ts
68.62% <0.00%> (+6.53%)
:arrow_up:

src/dist/SampleSetDist/bandwidth.ts
100.00% <0.00%> (+10.00%)
:arrow_up:

src/PointSet/Mixed.ts
27.21% <0.00%> (+13.92%)
:arrow_up:

src/dist/DistOperations/index.ts
78.90% <0.00%> (+14.84%)
:arrow_up:

... and 16 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.