Copy resolution listeners when a configuration is copied

Context
This PR fixes #1567
Contributor Checklist

[x] Review Contribution Guidelines
[x] Sign Gradle CLA
[ ] Link to Design Spec for changes that affect more than 1 public API (that is, not in an internal package) or updates to > 20 files
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass locally: ./gradlew quickCheck <impacted-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation including proper use of @since and @Incubating annotations for all public APIs
[ ] Recognize contributor in release notes

I've made the requested changes (other than the field being final)

Thanks for making the changes. It has been merged.

@bmuschko this broke our resolution rules plugin. If the copied beforeResolve retains a reference to the original configuration, or resolutionStrategy, for instance:
configurations.all { config ->   config.incoming.beforeResolve {         <do something with config>   } }
Then existing code depending on this behavior fails. If this is the plan going forward, then ResolvableDependencies would need to expose it's reference to the configuration, and this documented as a breaking change.

I don't think ResolveableDependencies should expose a Configuration. It already exposes a DependencySet that you can inspect/mutate. What would be the use case for inspecting the Configuration?

The case is in any way mutating the configuration by either adding dependencies, resolution strategies etc. where the calculation of that logic is expensive, and it's desirable to delay until beforeResolve.
In this case, we're copying the configuration and applying module alignments using resolutionStrategy.eachDependency. We have to do that in beforeResolve so we're not doing that expensive calculation except if the configuration is to be resolved, and we can't mutate the DependencySet because we need to affect transitive dependencies.

In the alignment case, we could catch the recursion differently, but this breaks our dependency lock and recommender plugins too.

@DanielThomas could you check the DependencySet for equality with the one of the configuration you originally registered on? In that case we could just document this as a breaking change.
A long term solution proposed by @adammurdoch would be a new dependency resolution hook that allows plugins to influence which version we pick, based on the whole graph (with potential respins). That would also be a huge performance boost, because these plugins could then stop copying configurations and resolving multiple times.

@oehme that would allow us to short-circuit the alignment, but this still breaks dependency locks and recommender: they'll affect the original, instead of the copy.

but this still breaks dependency locks and recommender: they'll affect the original, instead of the copy.

I'm afraid I don't understand this yet. Would this not solve the problem?
conf.incoming.beforeResolve { incoming ->
  if (incoming == conf.incoming) { //don't let the copy call this again
    maybeApplyLock(conf, extension, overrides, globalLockFilename, lockFilename)
  }
}

That would solve the issue with alignment, but doesn't allow the other plugins to work with copied configurations. We need the beforeResolves in the other plugins to affect the copy, so it can successfully resolve. With this change, when beforeResolve is called on the copy, we're holding a reference to the original.

Maybe I'm missing something, here's what I understand:
The beforeResolve listeners were not copied before. Now they are. The if-condition I proposed makes sure that the copied listener turns into a noop on the copy. That is semantically equivalent to not copying the listener at all. So whatever worked before should work with that.

Let's put the original example aside, as this impacts any use of beforeResolve that uses it as a configuration lifecycle hook, rather than as a way of mutating resolvable dependencies.
The simple pattern being this:
configurations.all { conf ->
	conf.incoming.beforeResolve {
		<do anything with 'conf' here>
	}
}

configurations.compile.copy().resolve()

Previously, these were not copied, which was fine because you simply add the configuration to the ConfigurationContainer to get the add handlers configured correctly. With this change, the beforeResolve handler on the copy will refer to the original configuration, instead of the copy.
That's why I suggest that if you want to move forward with copying these, you differentiate between explicitly configured handlers and those contributed by ConfigurationContainer after add handlers, and exclude the latter from the copy, then you can:

Add the configuration to the ConfigurationContainer to get the handler OR
Implement support to process add handlers in copy (#1555)

@oehme that workaround works a treat! Thanks for the assist.


Documented the order signals get sent

Had a student confused yesterday about the call order of signals, and whether they happened in parallel or serial. Couldn't spot any related wording in the docs so thought a sentence here was most appropriate.


CMAKE_INSTALL_PREFIX not honored for Python files installation on Windows

This is a second fix after #6339. Even after #6371, CMAKE_INSTALL_PREFIX is not honored when building under windows. I just built 2023.03.2, and noticed that the Python wrappers were not installed under the path I had specified.
The cause of the issue is that the "nt" scheme in sysconfig.get_path() uses the base key for the "platlib" path instead of  platbase, as does the "posix_prefix" schema:
In [9]: sysconfig._INSTALL_SCHEMES
Out[9]:
{'posix_prefix': {'stdlib': '{installed_base}/lib/python{py_version_short}',
  'platstdlib': '{platbase}/lib/python{py_version_short}',
  'purelib': '{base}/lib/python{py_version_short}/site-packages',
  'platlib': '{platbase}/lib/python{py_version_short}/site-packages',
  'include': '{installed_base}/include/python{py_version_short}{abiflags}',
  'platinclude': '{installed_platbase}/include/python{py_version_short}{abiflags}',
  'scripts': '{base}/bin',
  'data': '{base}'},
[...]
 'nt': {'stdlib': '{installed_base}/Lib',
  'platstdlib': '{base}/Lib',
  'purelib': '{base}/Lib/site-packages',
  'platlib': '{base}/Lib/site-packages',
  'include': '{installed_base}/Include',
  'platinclude': '{installed_base}/Include',
  'scripts': '{base}/Scripts',
  'data': '{base}'},
[...]
}

Nice inconsisteny, Python...

This breaks the builds for the windows python wheels. Will cherry-pick this fix.


Cranelift: Add heap_load and heap_store instructions

As discussed in https://github.com/bytecodealliance/wasmtime/issues/5283.
I intend to implement option (3) from that issue, but this PR doesn't actually add any legalization or backend lowerings yet, just interpreter support and a suite of tests.

(I will also wait to remove heap_addr until I have everything moved over to heap_{load,store} and feel like we are in a good position)


build/stage1: make target installing units can not remove the ACI dir

This can cause the build to fail for some flavors, as the unit generation targets can delete the stage1/aci rootfs being built after some other make targets responsible for setting it up execute (e.g.: stage1/aci and stage1/usr_from_src/stdlibdirs.mk), effectively undoing everything other targets may have already done.
Here is an example make V=2 output from one build with --with-stage1-flavors=src:
builder[5]:   RM RF        /opt/build-rkt/aci-for-src-flavor/rootfs
builder[5]:   MKDIR        /opt/build-rkt/aci-for-src-flavor/rootfs/lib/x86_64-linux-gnu
builder[5]:   RM RF        /opt/build-rkt/aci-for-src-flavor/rootfs/usr/lib/rkt/plugins/net
builder[5]:   RM RF        /opt/build-rkt/aci-for-src-flavor/rootfs
builder[5]:   STAMP        /opt/build-rkt/stamps/__stage1_units_units_install_mksrc_remove_acirootfs.stamp

... later stage1/usr_from_src fails:
builder[5]:   CP           /lib/x86_64-linux-gnu/libnss_files-2.24.so => /opt/build-rkt/aci-for-src-flavor/rootfs/lib/x86_64-linux-gnu/libnss_files-2.24.so
builder[5]: /usr/bin/install: cannot create regular file '/opt/build-rkt/aci-for-src-flavor/rootfs/lib/x86_64-linux-gnu/libnss_files-2.24.so': No such file or directory

Note that the second RM RF undoes MKDIR .../lib/x86_64-linux-gnu, causing a failure later.

This oddly causes rkt to fail on Fedora, and reverting this fixes it. I'm looking in to why...


UI scale

Steps to reproduce

Increase UI scale using Ctrl +/= (Do this 2-3 times, it depends on display, but 2-3 times should do it)
Open any manga
Open Reader settings

✔ Expected Behavior
Everything works fine and the size increases for fonts, objects and all.
❌ Actual Behavior
Reader settings size also increases, but the bottom settings disappear.
What I mean is that they are not possible to scroll down to.

Fixed, will be released soon.


[SPARK-32818][SQL] Make CONVERT_METASTORE_PARQUET and  CONVERT_METASTORE_ORC session level  configurable

What changes were proposed in this pull request?
There are some cases that we have history hive  table, in the past, it use TEXTFILE serde, later  it's change to ORC serde, since during metastore convert, we will use table level serde to scan all partition, if some old partition's file format is not same , will failed to convert , we need to close metastore convert and restart spark program then run.
But we know that enable hive metastore convert can speed up data r/w, and avoid many minor problem.
It's bad for ad-hoc engine such as long running spark thrift server. so I think it's necessary to make these two config can be changed by use in session level
Why are the changes needed?
Make use range more flexiable
Does this PR introduce any user-facing change?
People can set CONVERT_METASTORE_PARQUET and CONVERT_METASTORE_ORC in session level
How was this patch tested?
WIP

Test build #128389 has started for PR 29672 at commit c2ff589.

Spark can't support SET FILEFORMAT now, so it's hard to construct UT, should we support this?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33013/
Test PASSed.

Test build #128389 has finished for PR 29672 at commit c2ff589.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128389/
Test FAILed.

retest this please

Test build #128440 has started for PR 29672 at commit c2ff589.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33064/
Test PASSed.

Test build #128440 has finished for PR 29672 at commit c2ff589.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128440/
Test PASSed.

Spark can't support SET FILEFORMAT now, so it's hard to construct UT, should we support this?

Not sure about why we cannot add tests; how about just creating a hive table, then checking an optimizing plan (or a physical plan) for scanning it?

Spark can't support SET FILEFORMAT now, so it's hard to construct UT, should we support this?

Not sure about why we cannot add tests; how about creating a hive table, then checking an optimizing plan (or a physical plan) for scanning it?

It's ok to checking plan,. But mostly, I want to reproduce the case I mentioned in desc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33077/
Test PASSed.

Do we need to implement SET FILEFORMAT?

Test build #128454 has started for PR 29672 at commit fc221f5.

Test build #128454 has finished for PR 29672 at commit fc221f5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128454/
Test PASSed.


Updated autonomous group commands: added new paths, modified sequencing, timeouts, and ScoringAction

Added new paths (backwards79, offTarmac, ballToTarmac, and tarmacToBall) and removed old, unused paths
Changed ShooterRun to never finish. Instead end this command externally when shooter is up to speed (in ScoringAction)
Added timeout parameter for ScoringAction to control how long to shoot for (can be set for each ScoringAction instance in each group command)
Added AutoStop in parallel with IntakeDeploy and ScoringAction in group commands that makes sure drive motors are fed at all times in these group commands
Added debug configuration bool for ramsete tuning that sends velocity, position, and heading prints to SmartDashboard during RamseteFollowerExecute when true
Tuned kP for autonomous velocity control
Increased max velocity for auto paths to 2.5 m/s from 1.0 m/s
Removed brake mode setting from RamseteFollowerInit and End

Looks good to me. This pattern you have of making the command group responsible for ending the command is definitely an improvement.


typo 09_ODE

Near the end of the global error for the forward euler example, there is a type in the line:
"In other words the global error is bounded by the original global erro(r) and"

Fixed!


Add support for sequence acquisitions with external triggering

Pycro-manager by default supports hardware triggered acquisitions where the camera is the master device. An alternative to this are situations where where there is another master device and the camera waits to receive triggers from it. Implementing this situation will require a new acquisition hook that runs after the camera has been told to start waiting for triggered acquisitions. This hook will be used to send a custom command to whatever devices is the master of the triggering setup.

For the specific use case that we are hoping to solve right now, there is only one external trigger for each sequence. The external trigger starts the camera, which then generates images at the set frame rate until it receives a software stop signal. The frame rate, ROI, etc... are all configured using software calls before the camera is set to the 'Trigger First' mode.
The more general case you have described will also be useful, but I'm not sure if there will be a practical difference in supporting these cases.

No it should be the same since the hook will only run once for each sequence

@dpshepherd I added support for this that I think should work. Want to give it a test? It should be available in the next nightly build of MM after https://github.com/micro-manager/micro-manager/pull/893 gets merged. New features are available in pycromanager 0.6.0 (which is now on pypi).
You should be able to modify this example to make it work. Let me know if you have questions.

Awesome! Should be able to get to this next week and provide some feedback.

Two questions came up about implementing this for our setup based on your example.

We don't want to setup anything up in the acquisition except the number of images to capture. Would we just set num_time_points equal to the number of images expected, keep time_interval_s=0, and delete the rest of the setup? The external controller will handle the rest.
Is there a reason a we should not pass additional parameters img_process_fn  using a functools.partial object? We would like to move away parsing/saving the MM metadata structure because it really bogs down when we capture millions of images.

Yes, you could do this as just one big timelapse. Though if you plan to use the built in viewer it might be better to create custom acquisition events (rather than using the multi_d_acqusition convenience function) and supply z indices, but nothing related to the hardware.

I think that should work fine. I've set attributes on img_process_fn in order to save some kind of state from call to call, which I suspect might be using the same internal mechanism as partial (though I've not used the latter myself)

if you are trying to save to your own format in real time, you will be limited by the speed with which the transfer layer can send data from Java to Python. I think this is around 100 MB/s. The native Java saving can go a lot faster than this if you have the right hardware and don't use an image processor. Also, this saving uses a different storage library than the regular MM one. Though they are similar, this one is optimized for speed and scale. So you may not run into the same problems as before.

Ah, I didn't realize the max data transfer rate of the bridge. We are well in excess of that (order of magnitude). We will try it out but will probably have to find another path forward for the particular scope I had in mind.
We have another rig where we don't generate data as rapidly that this should be useful for.

It's possible if your RAM is big enough to hold all or most of a full acquisition you could cache the images as they are sent across the bridge.
Another better possibility would be that you can just read the data as soon as the acquisition ends and resave in a different format. Could be in the same script so you wouldnt need to run a separate step.
Out of curiosity, what is it about the tiff-based format that doesn't work for your analysis?

Holding into RAM isn't an option, because we are acquiring at minimum > 7 TB of raw imaging data and need to stream it to disk as fast as possible.
We are already doing your second suggestion with our micromanager 2.0 based acquisition.
We are acquiring tilted planes with a high NA stage scanning oblique plane microscope. We have to deskew these somehow. This can be done by directly writing to a bigdataviewer H5 and baking in the needed transforms OR do orthogonal interpolation once a given strip scan is done. It is much more efficient to save as the H5 format as a first step for these massive datasets.

Makes sense. I tend to think in general that it is better to not do any intensive processing on the fly if you dont have to and you're really trying to push data throughput, and it's not clear that writing directly to hdf would be able to keep up given the performance issues with h5py.
There are other things you could try, such as writing a Java-based image processor to avoid going through the bridge, but there are not guarantees these would work at the performance level you need. Might not be worth the trouble compared to the current strategy

Yes, I agree. We had some other reasons (fluidics control) that we were
interested in pycro-manager.
There are performant ways to write from numpy to h5 within Python. For
example,  https://github.com/nvladimus/npy2bdv. One can also set up a
custom solution and compress on the fly using something like B3D
https://github.com/balintbalazs/B3D. We have a pure Python based control
code for this, but our current camera's Python SDK is a bit annoying and
has nice MM drivers.
On Mon, Aug 3, 2020 at 11:29 AM Henry Pinkard<EMAIL_ADDRESS>wrote:

Makes sense. I tend to think in general that it is better to not do any
intensive processing on the fly if you dont have to and you're really
trying to push data throughput, and it's not clear that writing directly to
hdf would be able to keep up given the performance issues with h5py
https://cyrille.rossant.net/moving-away-hdf5/.
There are other things you could try, such as writing a Java-based image
processor to avoid going through the bridge, but there are not guarantees
these would work at the performance level you need. Might not be worth the
trouble compared to the current strategy
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/micro-manager/pycro-manager/issues/60#issuecomment-668175361,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMK4VVXGT4V6UF6DHBZ2A3R636Z7ANCNFSM4OHLVC6Q
.

Good to know. Maybe in the future a better transfer layer can make this possible, but it's not on my immediate todo list
Let me know if the triggering stuff works as expected


Remove macOS menu bar icon

Can we please get the ability to turn off the macOS menu bar icon?

I was looking for the method to hide it, I'm surprised there's none yet

+1!


visual cue of pointer depthness

William suggested: "i think the UI could use some upgrades to make it easier to navigate pointer-hell, like maybe giving unique colors to each pointer onscreen, putting each pointer on a new line as opposed to inline expansion, etc"
Liked by 3 people.
https://ought-inc.slack.com/archives/CMVHF5HJ8/p1567717358030000

@DerekElkinsOught err sorry what I meant was -- are we sharing code between Mosaic 1 and 2 in such a way that implementing this for one would effectively implement it for both?

@DerekElkinsOught err sorry what I meant was -- are we sharing code between Mosaic 1 and 2 in such a way that implementing this for one would effectively implement it for both?

No.


Asymmetric offsets for double bonds

To draw extra cylinders/lines inside a ring

I've got an initial commit for this here: fredludlow/ngl@11faf5d3295ca3deb314803f87e5d180079cf313
Looks nice for lines, looks okay for licorice (the cylinders are uncapped - might need to place extra spheres at the ends to round them off?) and looks a bit sketchy for ball+stick mode.

Looks great!
Does the amount of "shortening" need to be adjusted for different values of bondSpacing?

the cylinders are uncapped - might need to place extra spheres at the ends to round them off?

Yeah would be nice, but can be done in the future. Either with extra spheres or rounded caps (https://github.com/arose/ngl/issues/187). In both cases it requires some refactoring as the placement of the spheres and the "identity" of the shifted bonds is only known within the getBondData method.

looks a bit sketchy for ball+stick mode.

I guess that is okay. One could change the aspectRatio param for bigger spheres.

Thanks!

Does the amount of "shortening" need to be adjusted for different values of bondSpacing?

Probably - actually the parameters need a bit of a think through (I'm currently abusing the bondSpacing parameter because I just wanted a way to play with relative radii in the UI easily without wiring up another parameter).
I think for Balll+Stick the current (symmetric) rendering looks better anyway (it looks a bit more like the plastic molecular modelling kits you can buy) so maybe for that mode at least the current (two smaller symmetric cylinder) mode should be default.
I wasn't sure what the most intuitive set of parameters would be to expose the various options, multipleBond could take values null, symmetric, offset? Or there could be a separate param for offset/asymm? Then the parameters you need internally are a radius scaling factor, a shortening factor and an offset, which of those are independent and exposed to the user and which are calculated from others I don't really have a strong opinion on as long as the defaults look good. Do you have a preference?

multipleBond could take values null, symmetric, offset

I like this approach

Do you have a preference?

not really, just not too many :-)

Okay, nearly there I think. What I've gone for in terms of parameters is:

multipleBond is a select with 'off', 'symmetric', 'offset'

There are two parameters, bondScale and bondSpacing:

bondScale - ranges from 0-1.0 - relative radius of the second bond for double bonds and (in the offset case) triple bonds. In the symmetric case there's another adjustment that takes bond order into account. Default of 0.4 seems okay
bondSpacing - In symmetric mode a value of 1 means the outside of the cylinders will be placed at the standard radius distance:

This can vary between 0.5 and 2.0 to push the cylinders in and out
In offset mode a value of 1 corresponds to the center of the offset cylinder being 2*radius offset:

Bond shortening is calculated according to offset but there's a hard limit at 80% of the original bond length (otherwise you end up with weird effects with unusual combinations of other parameters).

When loading a PDB through the webapp menu the default representation for het groups is ball+stick with scale=2.0, I've added values for bondScale and bondSpacing so that if the user subsequently selects either of the multiple-bond modes it looks a bit better.
I'm afraid I've ended up with a fairly monolithic PR with both the ring stuff and this sorry.

This is really great!
One little thing regarding bond shortening. Would look a bit better to shorten only one side of the bond for e.g. carbonyls?

Yes - I guess a quick "is it terminal" check would do it. What's the cleanest way to deduce this inside the Structure.getBondData method? (residue type and bondgraph, atom/bond proxy and bondHash, or something else...?)
Even better would be to be aware of the geometry and adjust cutoff accordingly (linear alkynes also look a bit funny at the moment) but that sounds like another level of perception of the molecule

Yes - I guess a quick "is it terminal" check would do it. What's the cleanest way to deduce this inside the Structure.getBondData method? (residue type and bondgraph, atom/bond proxy and bondHash, or something else...?)

var isTerminal = bondHash.countArray[ atomIndex ] === 1;

Hmm, actually if you have different shortenings on both sides: fredludlow/ngl@39ce713 you end up with with color-changes not quite lining up:

I'm inclined to leave this as is for now, what do you think?

... color-changes not quite lining up:

yeah fixing that will cause some headaches, not sure if it is worth it, I would say no. What do you think.

Agreed, I'll close this for now.


Add range_qname and range_value methods to Attribute.

To save memory this only uses a u16 and u8 per Attribute, but the tradeoff is that the qname and value ranges will be incorrect in a few extreme data cases, specifically if the attribute's qname has len > 65535 or there are >254 spaces surrounding its equal sign.  Those cases seem unlikely enough in practice to justify not using more memory for this.

Thanks! Looks good to me. Is that all or do you plan any other changes? If no then I will publish a new release.

Nope, not planning any other changes right now.  Thanks!


Catapult doesn't reload after shooting.

Maybe never or maybe sometimes. Needs more testing and debugging.

If we use m_waiting to determine whether the button was released before the waiting period expires then m_waiting should be reset when the period expires.
The side effect can actually be useful: if the "fire" button was held for long enough (more than 3 secs) then it could mean that the catapult is released for maintenance, so no reloading is needed.
Done.

At the bag-n-tag night the catapult was not behaving again. Sometimes it reloaded automatically after the "fire" sometimes not. Still needs to be debugged and fixed.


Confusing error message on incorrect command

Problem Statement
The error message for an incorrect command to turnkey COMMAND is too confusing right now. This happens because the first positional argument can be either a file (in which case COMMAND=benchmark is assumed, ie turnkey INPUT_FILES == turnkey benchmark INPUT_FILES) or an explicit command (ie, turnkey COMMAND INPUT_FILES`).
The error message presents both of these possibilities, which is too broad of a decision space for new users.
Example
An attempt to call turnkey build INPUT_FILE (which is not currently supported) results in this error message:
(lightning_py310_231210) user1@hitssv565:/local_data/user1/models$ turnkey build rotational_power.py --sequence optimize-fp16

Error: Unexpected positional argument `turnkey build`. The first positional argument must either be an input file with the .py or .onnx file extension or one of the following commands: ['benchmark', 'cache', 'models', 'version'].

▄██████████████▄▐█▄▄▄▄█▌
██████▌▄▌▄▐▐▌███▌▀▀██▀▀
████▄█▌▄▌▄▐▐▌▀███▄▄█▌
▄▄▄▄▄██████████████

Traceback (most recent call last):
  File "/local_data/user1/miniforge3/envs/lightning_py310_231210/bin/turnkey", line 8, in <module>
    sys.exit(turnkeycli())
  File "/local_data/user1/miniforge3/envs/lightning_py310_231210/lib/python3.10/site-packages/turnkeyml/cli/cli.py", line 493, in main
    raise exceptions.ArgError(error_msg)
turnkeyml.common.exceptions.ArgError: Unexpected positional argument `turnkey build`. The first positional argument must either be an input file with the .py or .onnx file extension or one of the following commands: ['benchmark', 'cache', 'models', 'version'].

Proposed Solution
An arguments error in the first positional argument to turnkey should only ever present help for either an incorrect command OR an incorrect file path.
UI/UX:

The error for turnkey banchmerk model.py should be: "command banchmerk is not a valid command, try CHOICES" (no mention of file inputs)
The error for turnkey cant_find_me.py should be specifically about how a file with the supported file extensions could not be found (no mention of command choices)

Implementation: we will look for an indication that the first positional argument is either a command or a file path, and provide an appropriate error accordingly.

If the special characters *, \, /, or . is present, we assume the argument is INPUT_FILES
Otherwise, we assume the argument is COMMAND

cc @andife with regards to #69

@danielholanda can you review the implementation proposal and let us know if you are good with it?

The proposed implementation is definitely a great improvement!


Better Session Provider Documentation

Gitea version (or commit ref): 1.13
Git version:
Operating system: Docker / Kubernetes

Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No

Log gist:

Description
It took me litterally 1 hour to figure out how to configure the sessions properly with Postgres. The offical Gitea Documentation is really lite on that topic as you can see here https://docs.gitea.io/en-us/config-cheat-sheet/#session-session
There are only 2 lines that describe how to configure it but doesn't mention how to do it exactly. A example would be great here.
After searching in the code I found out that Gitea uses a library for that and the library has a wonderfull documentation. Maybe it make sense to a) either link their documentation in the Gitea documentation or b) copy the important parts out of it and further reference them.
https://go-macaron.com/middlewares/session#postgresql
What do you think?
...
Screenshots

macaron has been replaced by go-chi and the session middleware has been replaced by https://gitea.com/go-chi/session

macaron has been replaced by go-chi and the session middleware has been replaced by https://gitea.com/go-chi/session

@Skaronator if you like you could enhance the docu ... :)

@Skaronator if you like you could enhance the docu ... :)

The new db session provider is even simpler than any of the rest - it simply uses the gitea db so doesn't need any configuration.

@zeripath we should at least mention this option for clustering ...

@Skaronator @lunny we actually need some lines about how to store session within postgres. I've tried to change PROVIDER to postgres and PROVIDER_CONFIG to postgrest connection string, but Gitea throws error:
Apr 22 09:44:52 gs gitea[8184]: panic: pq: SSL is not enabled on the server
Apr 22 09:44:52 gs gitea[8184]: goroutine 1 [running]:
Apr 22 09:44:52 gs gitea[8184]: gitea.com/go-chi/session.Sessioner(0xc002f5d3f0, 0x1, 0x1, 0x6)
Apr 22 09:44:52 gs gitea[8184]:         /workspace/gitea/vendor/gitea.com/go-chi/session/session.go:238 +0x1a5
Apr 22 09:44:52 gs gitea[8184]: code.gitea.io/gitea/routers/routes.WebRoutes(0xc00249cf80)
Apr 22 09:44:52 gs gitea[8184]:         /workspace/gitea/routers/routes/web.go:133 +0x1b8
Apr 22 09:44:52 gs gitea[8184]: code.gitea.io/gitea/routers/routes.NormalRoutes(0xc00133be40)
Apr 22 09:44:52 gs gitea[8184]:         /workspace/gitea/routers/routes/web.go:123 +0x11b
Apr 22 09:44:52 gs gitea[8184]: code.gitea.io/gitea/cmd.runWeb(0xc00133be40, 0x0, 0x0)
Apr 22 09:44:52 gs gitea[8184]:         /workspace/gitea/cmd/web.go:138 +0x4ae
Apr 22 09:44:52 gs gitea[8184]: github.com/urfave/cli.HandleAction(0x24c10c0, 0x3728000, 0xc00133be40, 0xc00133be40, 0x0)
Apr 22 09:44:52 gs gitea[8184]:         /workspace/gitea/vendor/github.com/urfave/cli/app.go:524 +0x105
Apr 22 09:44:52 gs gitea[8184]: github.com/urfave/cli.Command.Run(0x28c963a, 0x3, 0x0, 0x0, 0x0, 0x0, 0x0, 0x2a2d0ae, 0x16, 0x0, ...)
Apr 22 09:44:52 gs gitea[8184]:         /workspace/gitea/vendor/github.com/urfave/cli/command.go:173 +0x579
Apr 22 09:44:52 gs gitea[8184]: github.com/urfave/cli.(*App).Run(0xc0011bddc0, 0xc00003c040, 0x2, 0x2, 0x0, 0x0)
Apr 22 09:44:52 gs gitea[8184]:         /workspace/gitea/vendor/github.com/urfave/cli/app.go:277 +0x808
Apr 22 09:44:52 gs gitea[8184]: main.main()
Apr 22 09:44:52 gs gitea[8184]:         /workspace/gitea/main.go:115 +0xa6a
Apr 22 09:44:52 gs systemd[1]: gitea.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Apr 22 09:44:52 gs systemd[1]: gitea.service: Failed with result 'exit-code'.

@ndaidong what is the :-1: for?
How can I be clearer:
If you set PROVIDER=db then you don't need to set anything in PROVIDER_CONFIG and the session will just use the DB configuration.

@zeripath thank you, it works for me now, even the log show:
Apr 23 09:32:22 gs gitea[10421]: 2021/04/23 09:32:22 ...odules/session/db.go:166:GC() [I] session/DB: error garbage collecting: pq: column "created_unix" does not exist
Apr 23 09:32:23 gs gitea[10421]: 2021/04/23 09:32:23 cmd/web.go:189:listen() [I] Listen: http://<IP_ADDRESS>:18081
Apr 23 09:32:23 gs gitea[10421]: 2021/04/23 09:32:23 cmd/web.go:192:listen() [I] LFS server enabled
Apr 23 09:32:23 gs gitea[10421]: 2021/04/23 09:32:23 ...s/graceful/server.go:55:NewServer() [I] Starting new Web server: tcp:<IP_ADDRESS>:18081 on PID: 10421
Apr 23 09:32:23 gs gitea[10421]: 2021/04/23 09:32:23 ...odules/session/db.go:166:GC() [I] session/DB: error garbage collecting: pq: column "created_unix" does not exist

Anyway, it's neccessary to update the docs and example, currently there is no word mentioning to db or posgres.
[session]
; Either "memory", "file", or "redis", default is "memory"
PROVIDER = memory
; Provider config options
; memory: doesn't have any config yet
; file: session file path, e.g. `data/sessions`
; redis: network=tcp,addr=:6379,password=macaron,db=0,pool_size=100,idle_timeout=180
; mysql: go-sql-driver/mysql dsn config string, e.g. `root:password@/session_table`
PROVIDER_CONFIG = data/sessions

@ndaidong Could you help to send a PR to improve that?

@lunny sure, my pleasure. I will send a PR this weekend.


add  support for llama cpp and rag

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I will add LlamaCpp to the roadmap, RAG is already on the roadmap and should be available in the next release which I hope to have out in the next month.

Just relesed parllama v0.3.11 with support for llamacpp running in openai compatibility mode.
start llamacpp in openai compat mode with your desired model loaded
Go to options area in parllama to change base url if needed, go to chat tab seled LlamaCpp as provider and leave model as "default"


[🐛 BUG]: Panic in grpc plugin

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

What happened?
Sometimes RR doesn't start and ends with an error with exit code: 1
Version (rr --version)
2023.1.0-rc.2
How to reproduce the issue?
I don't have a clear idea how to reproduce the bug since it doesn't always appear
Relevant log output
2023-04-10T13:17:01+0000	DEBUG	server      	worker is allocated	{"pid": 17, "internal_event_name": "EventWorkerConstruct"}
2023-04-10T13:17:01+0000	DEBUG	server      	worker is allocated	{"pid": 13, "internal_event_name": "EventWorkerConstruct"}
2023-04-10T13:17:01+0000	INFO	grpc        	grpc server was started	{"address": "tcp://<IP_ADDRESS>:22164"}
handle_serve_command: Function call error:
	got initial serve error from the Vertex *metrics.Plugin, stopping execution, error: duplicate metrics collector registration attempted
2023-04-10T13:18:11+0000	DEBUG	server      	worker is allocated	{"pid": 15, "internal_event_name": "EventWorkerConstruct"}
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x30 pc=0xdb9641]

goroutine 97 [running]:
github.com/roadrunner-server/grpc/v4.(*Plugin).Workers(0xc000c44080?)
	github.com/roadrunner-server/grpc/v4@v4.1.6/plugin.go:222 +0xc1
github.com/roadrunner-server/sdk/v4/metrics.(*StatsExporter).Collect(0xc0000a7180, 0xc00059a760?)
	github.com/roadrunner-server/sdk/v4@v4.2.0/metrics/metrics.go:42 +0x4a
github.com/prometheus/client_golang/prometheus.(*Registry).Gather.func1()
	github.com/prometheus/client_golang@v1.14.0/prometheus/registry.go:456 +0x10d
created by github.com/prometheus/client_golang/prometheus.(*Registry).Gather
	github.com/prometheus/client_golang@v1.14.0/prometheus/registry.go:548 +0xc09

Hey @genhoi 👋🏻
Do you use metrics? Could you please share your configuration as well?

@rustatian yes )
version: "3"

server:
  command: "php /app/public/index.php"
  env:
    - APP_RUNTIME: 'Baldinof\RoadRunnerBundle\Runtime\Runtime'
    - RR_MODE: 'grpc'

rpc:
  listen: ${RR_RPC}

metrics:
  address: "<IP_ADDRESS>:${RR_METRICS_PORT}"

grpc:
  listen: "tcp://<IP_ADDRESS>:${RR_GRPC_PORT}"
  proto:
    - "/app/proto/services.proto"
  pool:
    num_workers: ${RR_GRPC_WORKERS_COUNT}
    debug: ${RR_GRPC_WORKERS_DEBUG}

service:
  update-automatic-collection-items:
    command: "php /app/bin/console app:update-automatic-collection-items"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 900
  update-section-items:
    command: "php /app/bin/console app:update-section-items"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 1800
  update-collection-sections:
    command: "php /app/bin/console app:update-collection-sections"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 7200
  update-is-draft-items:
    command: "php /app/bin/console app:update-is-draft-items"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 7200
  sync-collection-item-consume:
    command: "php /app/bin/console messenger:consume sync-collection-item-consume"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 30
  sync-item-size-data-consume:
    command: "php /app/bin/console messenger:consume sync-item-size-data-consume"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 30
  fill-indexing-filters:
    command: "php /app/bin/console app:fill-indexing-filters"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 1200
  messenger-setup-transports:
    command: "php /app/bin/console messenger:setup-transports"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 30
  collect-metrics:
    command: "php /app/bin/console app:collect-metrics"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 3600
  collect-item-metrics:
    command: "php /app/bin/console app:collect-item-metrics"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 1800
  sync-category-item-attribute-values:
    command: "php /app/bin/console app:sync-category-item-attribute-values"
    process_num: 1
    exec_timeout: 0
    remain_after_exit: true
    restart_sec: 60

@genhoi Thanks 👍🏻 . I understand the reason for panic, but if you're using metrics, you should avoid duplicate metric registrations. Prometheus does not allow this.
You may try the new lock plugin to register metrics only once using.


Compress all binding resource packages.

Compress all binding resource packages, to try to avoid MAX_PATH problems on
Windows.

Hi @rolfbjarne,
when will this fix merged and released? I hope this will fix the nuget and build errors.
Thank you.


Upgrade dependencies

Same issue as with Intercept, the outdated dependencies are screwing up Webpack builds.

PRs welcome!
:stuck_out_tongue:

Will probably be on sunday - let me know if you did it yourself before then. :)

Released as 1.1.0!
Thanks @jeffijoe !


feature: whisper api

https://platform.openai.com/docs/api-reference/audio/create
Needed:

[x] Basic wirings to whisper.cpp #211
[ ] Support for prompt
[ ] Support for response_format
[ ] Separate container images with ffmpeg (licensing)
[ ] Add tests (as models are free, and audios can be found online, should be pretty straightforward!)

Hah nice ;) I'll take a crack at this, this week

argh, started playing with this after seeing the message on twitter and just saw your message, sorry about this :/, however I've just added some skeleton and wiring to the lib, it's far for complete yet

I've just added the skeleton now - I will take a look at adding basic tests over the weekend, everything else is open to being picked up :+1:

Awesome I'll try it out and see if I can use it in my project. Feel free to drop some issues if there is stuff still to be done

Transcription stops at around 3min of the input audio. The mp3 file is 10MB and 7min.


[Hotfix][e2e] Fix e2e error

Purpose of this pull request
[Hotfix][e2e] Fix e2e error
fix https://github.com/apache/seatunnel/pull/5957
Does this PR introduce any user-facing change?
No
How was this patch tested?
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

@ic4y @Hisoka-X @EricJoy2048


[service-bus] Make close() work when a connection is in process

Currently it's possible for a user to call close() while a connection is in process. When this happens you can end up in a bad state where the object thinks it's closed but it still has open resources.
This commit changes it so:

open() and close() now use the same lock to prevent them from overlapping
open() respects an "is closed" flag.

Can we pull this PR out of the draft stage?

I found some issues and I think I want to look at it with fresh eyes tomorrow morning:

BatchingReceiver works ok with the way I've got it now - as far as I can tell there isn't a way to start initializing a connection without having set it up in some way so close() blocks on it properly.
SessionReceiver also had an issue but the fix there was just to move the lock logic into the SessionReceiver and out of MessageSession.
StreamingReceiver doesn't work with how it's coded right now. StreamingReceiver.create is async so there's a chance the user can call close(), complete, and then we'll resume creating the streaming receiver which is now "uncloseable" again. We don't store the context.streamingReceiver in time for close() to even know that open() is happening. I can fix this by just moving the lock logic into Receiver, which is what I basically did for SessionReceiver.

As it is my tests are failing but I think the ideas make sense. Just need to take a longer look at it.

@ramya-rao-a  - okay, it looks correct now.
The basic pattern was:

When we create the underlying MessageReceiver/MessageSession make sure the outer object (Receiver/SessionReceiver) has a reference to it. Otherwise, close() doesn't actually get routed and all of our locking gets bypased, resulting in an orphaned receiver.
In the lock check inside to make sure that the user hasn't tried to close the object already and that the connection isn't open. There's a "connectionClosing" variable that we don't need to check anymore since the lock takes care of it.

/azp run js - servicebus - tests

/azp run js - servicebus - tests

/azp run js - servicebus - tests

There are certain parts of this PR that are straight-forward and can be pulled out into a separate PR while we keep this PR open to think more on the ramifications of the changes being done. I recommend creating a separate PR for the below 2 changes

Batching Receiver: Resolve the promise with empty array after init() completes, but batching receiver has been closed. This avoids the TypeError from being thrown
Streaming Receiver: If close was initiated while link recovery is in progress, then init() should gracefully exit i.e check for this. wasCloseInitiated inside init()

Closing this PR for now:

The simple work that @ramya-rao-a  mentioned above closes most of the gaps that we were concerned about. close() can still return early (the locks are needed to make that not an issue) but the subsequent init() will make sure the created receiver ultimately gets cleaned up.
The work to add in the locking makes sense for track 2 (maybe even in a different form) but will probably be a bit too complicated for just delivering a hotfix.


New TeamCard 2

Design a new TeamCard for Head Coordinators.
The card should have social media icons in SVG format.
Map the card component to appropriate data set.
This card will have picture.
Use this card component only for Year 2023 Team. Name the component accordingly.
Submit SS/Video of the component in working condition.
Use good naming convention for CSS classes.
Avoid usage of any external library.
The card should symbolise our theme of Halloween. Idea - Stick spiderwebs in top corners of a pumpkin near the bottom , maybe a picture of a small graveyard tablet. Use the smallest size assets available for this, preferred SVG.

I want to work on this.

Only work on making the card as of now with some dummy data. Use hardcoded values, don't connect it with any API.
Import that card as a separate component with the name TeamCard2_2023.
I will let you know when you use the api for data.

okay sir,
Which social media icons it should contain, only linkedIn or all others

only linkedin

This is the design. If any further cahnges are required please inform.

https://github.com/TCP-Tech/CodeUtsava7.0/assets/104003078/0d9bf0d1-7e2c-4173-a5cb-da115d5db920


OpenAPI3 emitter (Same with autorest) produce 204 for record used as response type

import "@cadl-lang/rest";

using Cadl.Http;

op test(): Record<int32>;

"get": {
        "operationId": "Int32Value_get",
        "parameters": [],
        "responses": {
          "204": {
            "description": "There is no content to send for this request, but the headers may be useful. "
          }
        }
      }

Playground Link
Probably a bug with deciding that empty object should produce 204 and not checking for indexer #1015

Tracking this as relkated to #1015


Remove repositoryPath from template list contract

Remove repositoryPath from template list contract once we know it's safe to do, i.e. our IDE partners: Visual Studio, and Visual Studio Code, confirm that it has started using path instead of repositoryPath
Example contract from current azd template list --output json output:
  {
    "name": "Static React Web App + Functions with C# API and SQL Database",
    "description": "A blueprint for getting a React web app with a C# API and a SQL database on Azure. The blueprint includes sample application code (a ToDo web app) which can be removed and replaced with your own application code. Add your own source code and leverage the Infrastructure as Code assets (written in Bicep) to get up and running quickly. This architecture is for hosting static web apps with serverless logic and functionality.",
    "path": "todo-csharp-sql-swa-func",
    "repositoryPath": "todo-csharp-sql-swa-func"
  },

No longer applicable since we forwent the change in #2093


Fix broken links

Address most of the issues in #91.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Included in #96


fix(ui): node and activity modals overhaul

Description:
Bring node details and activity modals under the common modal system (modal stack)

Motivation and Context:
Currently node and activity modals are standalone and thus are not benefiting from the existing modal system (transition animation, proper escape button handling etc), as well as causing rendering issues.
Modals are now part of the common stack and are also bound closer to a redux state source resulting in a minor perf improvements.
This PR also does some cleanup and refactoring of the top level App container
Fix #1614

How Has This Been Tested?
Manually

Types of changes:
Bugfix and refactoring

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage increased (+0.03%) to 18.779% when pulling 4794a05018fbf36e235f97d3978a5f4297c4ff6f on korhaliv:fix/node-details-modal into 6c73c6d96c4e28cb179f894deb03485658865e13 on LN-Zap:next.


Pip install topohack directly in environment.yml using egg-name topolib

Fixes problem with installing the dependency topohack using pip after the merge of #4. Looks like the repository name is topohack but the package name is topolib in the setup.py file (see https://github.com/ICESAT-2HackWeek/topohack/blob/7194c6ea87bb41cc9dad38cbbef230391fbf7877/setup.py#L5).
For reference, the error messages at https://github.com/icesat2py/icepyx-examples-image/runs/2178186854#step:4:2455 were as follows:
  Installing pip dependencies: ...working... Ran pip subprocess with arguments:
  ['/srv/conda/envs/notebook/bin/python', '-m', 'pip', 'install', '-U', '-r', '/home/jovyan/condaenv.q6k_4xg4.requirements.txt']
  Pip subprocess output:
  Collecting topohack
    Cloning https://github.com/ICESAT-2HackWeek/topohack.git to /tmp/pip-install-mwukn_pq/topohack_6fab7f0621fb4c2ba166156d493385c2

  failed
  Pip subprocess error:
    Running command git clone -q https://github.com/ICESAT-2HackWeek/topohack.git /tmp/pip-install-mwukn_pq/topohack_6fab7f0621fb4c2ba166156d493385c2
    WARNING: Generating metadata for package topohack produced metadata for project name topolib. Fix your #egg=topohack fragments.
  WARNING: Discarding git+https://github.com/ICESAT-2HackWeek/topohack.git#egg=topohack. Requested topolib from git+https://github.com/ICESAT-2HackWeek/topohack.git#egg=topohack (from -r file:requirements.txt (line 1)) has inconsistent name: filename has 'topohack', but metadata has 'topolib'
  ERROR: Could not find a version that satisfies the requirement topohack (unavailable)
  ERROR: No matching distribution found for topohack (unavailable)

I've tested the install properly this time, and it should report the following output instead:
Installing pip dependencies: | Ran pip subprocess with arguments:
['/srv/conda/envs/icepyx-examples-image/bin/python', '-m', 'pip', 'install', '-U', '-r', '/home/jovyan/condaenv.wkyv1ae8.requirements.txt']
Pip subprocess output:
Collecting topolib
  Cloning https://github.com/ICESAT-2HackWeek/topohack.git to /tmp/pip-install-eibdfbem/topolib_993ba1be67394fec8b4c1f78af699113
Requirement already satisfied: requests in /srv/conda/envs/icepyx-examples-image/lib/python3.9/site-packages (from topolib->-r /home/jovyan/condaenv.wkyv1ae8.requirements.txt (line 1)) (2.25.1)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in /srv/conda/envs/icepyx-examples-image/lib/python3.9/site-packages (from requests->topolib->-r /home/jovyan/condaenv.wkyv1ae8.requirements.txt (line 1)) (1.26.4)
Requirement already satisfied: chardet<5,>=3.0.2 in /srv/conda/envs/icepyx-examples-image/lib/python3.9/site-packages (from requests->topolib->-r /home/jovyan/condaenv.wkyv1ae8.requirements.txt (line 1)) (4.0.0)
Requirement already satisfied: certifi>=2017.4.17 in /srv/conda/envs/icepyx-examples-image/lib/python3.9/site-packages (from requests->topolib->-r /home/jovyan/condaenv.wkyv1ae8.requirements.txt (line 1)) (2020.12.5)
Requirement already satisfied: idna<3,>=2.5 in /srv/conda/envs/icepyx-examples-image/lib/python3.9/site-packages (from requests->topolib->-r /home/jovyan/condaenv.wkyv1ae8.requirements.txt (line 1)) (2.10)
Building wheels for collected packages: topolib
  Building wheel for topolib (setup.py): started
  Building wheel for topolib (setup.py): finished with status 'done'
  Created wheel for topolib: filename=topolib-0.1-py3-none-any.whl size=13497 sha256=d77c2441299836131acf9e71b7ea573305ebed8e0bef19dfe7fe922724862c18
  Stored in directory: /tmp/pip-ephem-wheel-cache-dq0tcxkj/wheels/7e/66/16/bf4f6feec1b2cdde0605cfb36fde1cc69e5dcf0d44ecd6d529
Successfully built topolib
Installing collected packages: topolib
Successfully installed topolib-0.1

done

Oh, and I've also consolidated all the dependencies to be in the conda environment.yml file (i.e. removed the requirements.txt).

Closing as the community has moved to maintaining a dedicated Cryo docker image at https://github.com/CryoInTheCloud/hub-image


Hyperlink DOIs against preferred resolver

Same as https://github.com/Sage-Bionetworks/Synapse-Markdown/pull/21 ;-)

We no longer poll the DOI resolver to verify that a DOI is created, so this isn't an issue anymore. We now just make requests to DataCite, our DOI provider (using HTTPS here).
Thank you for your contribution!


Error when trying to publish

Description
I'm trying to publish a fix to my Raycast extension using the npm run publish command, but I'm receiving an error related to needing to commit or discard local changes before publishing. The full error message is as follows:
> npx @raycast/api@latest publish
wait  - validate package.json file ..  /Users/ralvarez/Documents/Raycast/Extensions/raycast-servicenow/package.json
  4:11  warning  Extension's title has to be Title Cased. Expected "Servicenow"
ready  - validate package.json file
ready  - validate package-lock.json
ready  - validate other lock files
ready  - validate extension icons
ready  - validate extension metadata
ready  - run ESLint
/Users/ralvarez/Documents/Raycast/Extensions/raycast-servicenow/src/components/Tables.tsx
  129:31  warning  Prefer Title Case naming convention for action titles (e.g Copy to Clipboard)  @raycast/prefer-title-case
  134:31  warning  Prefer Title Case naming convention for action titles (e.g Copy to Clipboard)  @raycast/prefer-title-case

✖ 2 problems (0 errors, 2 warnings)
  0 errors and 2 warnings potentially fixable with the `--fix` option.
ready  - run Prettier 3.3.3
ready  - getting fork
error  - preparing clone
    Error: failed running git Command failed: git checkout main
    error: Your local changes to the following files would be overwritten by checkout:
        extensions/servicenow/metadata/servicenow-2.png
        extensions/servicenow/metadata/servicenow-4.png
        extensions/servicenow/src/components/FavoriteItem.tsx
        extensions/servicenow/src/components/NavigationHistory.tsx
        extensions/servicenow/src/components/NavigationHistoryFull.tsx
    Please commit your changes or stash them before you switch branches.
    Aborting

Steps To Reproduce

Make local changes to files within the repository.
Run the command npm run publish.
The system shows an error that prevents switching branches.

Current Behaviour
No response
Expected Behaviour
No response

Did a previous publish fail?

Hi @mathieudutour,
Yes, a previous publish attempt did fail.

Alright, can you run cd ~/.config/raycast/public-extensions-fork && git checkout .? Then you should be able to publish the extension

Thanks! That solved the error when preparing the clone.
I don't have anything to publish at the moment, but I still tried the command and with the latest version I'm getting this:
❯ npm run publish

> publish
> npx @raycast/api@latest publish

wait  - validate package.json file ..  /Users/ralvarez/Documents/Raycast/Extensions/raycast-servicenow/package.json
  4:11  warning  Extension's title has to be Title Cased. Expected "Servicenow"
ready  - validate package.json file
ready  - validate package-lock.json
ready  - validate other lock files
ready  - validate extension icons
ready  - validate extension metadata
ready  - run ESLint
/Users/ralvarez/Documents/Raycast/Extensions/raycast-servicenow/src/components/Tables.tsx
  129:31  warning  Prefer Title Case naming convention for action titles (e.g Copy to Clipboard)  @raycast/prefer-title-case
  134:31  warning  Prefer Title Case naming convention for action titles (e.g Copy to Clipboard)  @raycast/prefer-title-case

✖ 2 problems (0 errors, 2 warnings)
  0 errors and 2 warnings potentially fixable with the `--fix` option.
ready  - run Prettier 3.3.3
ready  - getting fork
ready  - preparing clone
ready  - checking for new contributions
wait  - preparing extension .   /Users/ralvarez/.npm/_npx/daafed3b81e85078/node_modules/@raycast/api/dist/commands/publish/index.js:815
`))}`)}i.printExtractedSchemas&&(E_("info","extracted tool schemas:"),console.log(k5.ux.colorizeJson(pe,{theme:{null:"red",string:"green",number:"magenta",boolean:"cyan",key:"white",comma:"yellow",brace:"yellow",bracket:"yellow"},pretty:!0}))),c.tools=pe;try{let X=k6t();X&&(c.ai={...c.ai||{},...X})}catch(X){throw X instanceof Error&&(X.message=`reading AI file failed (${X.message})`),X}}delete c.type,t1.writeFileSync(ag.join(i.outputDir,"package.json"),JSON.stringify(c,null,"  ")),t1.rmSync(ag.join(i.outputDir,"assets"),{recursive:!0,force:!0}),t1.existsSync("assets")&&t1.cpSync("assets",ag.join(i.outputDir,"assets"),{recursive:!0}),P6t(i.outputDir,y),P6t(ag.join(i.outputDir,"tools"),x),E_("success","built extension successfully")}async function A6t(i,s,c){let p=i.environment=="dist",y=await I6t.build({entryPoints:i.entryPoints,entryNames:i.entryNames,sourcemap:i.sourceMap,outdir:i.outputDir,platform:"node",external:i.external,target:"node20.15.1",jsxFactory:"_jsx",jsxFragment:"_jsxFragment",bundle:!0,minifyWhitespace:p,minifyIdentifiers:p,minifySyntax:p,write:!0,logLevel:"silent",metafile:i.metaOutputDir!="",banner:i.banner,alias:i.alias,loader:{".wasm":"binary"},plugins:[Cwt,kwt(p,s,c)]});if(y.errors.length>0){let x=y.errors[0];throw new Error(`build failed (${x.location?.file}:${x.location?.line}:${x.location?.column} ${x.location?.lineText}): ${x.text}`)}i.metaOutputDir&&y.metafile&&t1.writeFileSync(ag.join(i.metaOutputDir,"meta.json"),JSON.stringify(y.metafile,null,"  "),"utf-8")}function P6t(i,s){try{t1.readdirSync(i).forEach(p=>{ag.extname(p)===".js"&&(s.find(y=>ag.basename(y,ag.extname(y))===ag.basename(p,ag.extname(p)))||t1.unlinkSync(ag.join(i,p)))})}catch{}}async function D7r(){let i="./node_modules/.bin/tsc";if(!t1.existsSync(i))throw new Error("please install the TypeScript compiler (npm install typescript)");try{await p3(i,["-p","tsconfig.json","--noEmit"],{preferLocal:!0,stderr:"inherit",stdout:"inherit"})}catch(s){throw new Error(`TypeScript check failed (${s.message})`)}}var T9e=require("child_process");var O6t=Da(require("path")),R6t=".",L6t="-",M6t="`git rev-list --max-parents=0 HEAD | tail -n 1`",x9e;function Z2(i){x9e=i}function G_(i,s){return new Promise((c,p)=>{(0,T9e.exec)(i,{cwd:x9e},(y,x,w)=>{if(s.throwOnError&&y){p(new Error(`failed running git ${y.message.replace(/oauth2:gho_[a-zA-Z0-9]+@/g,"oauth2:gho_xxxxx")}

Error: failed running git Command failed: git checkout ext/servicenow
fatal: Unable to create '/Users/ralvarez/.config/raycast/public-extensions-fork/.git/index.lock': File exists.

Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.

fatal: Unable to create '/Users/ralvarez/.config/raycast/public-extensions-fork/.git/index.lock': File exists.

Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.
    at /Users/ralvarez/.npm/_npx/daafed3b81e85078/node_modules/@raycast/api/dist/commands/publish/index.js:815:2283
    at ChildProcess.exithandler (node:child_process:430:5)
    at ChildProcess.emit (node:events:517:28)
    at maybeClose (node:internal/child_process:1098:16)
    at Socket.<anonymous> (node:internal/child_process:450:11)
    at Socket.emit (node:events:517:28)
    at Pipe.<anonymous> (node:net:350:12)

There is no index.lock file in the mentioned directory.
I killed the git process just in case and run again, but same error.
Then I tried ❯ npx<EMAIL_ADDRESS>publish and it worked fine.

Ah I think I found the issue, will fix shortly


check if suite is a DOMElement

If the phpunit.xml file is incorrectly/differently configured, $suite1 is  going to be null.
If there is no <testsuites> tag around the <testsuite> tag in the phpunit.xml file, the xpath query will find nothing.

Thanks @robertbasic!


AZ-204_lab_06.md

Image on page 4 is not translated
Image on page 11 is not translated
AZ-204_lab_06.md.pdf

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

fixed.
https://github.com/MicrosoftLearning/AZ-204-DevelopingSolutionsforMicrosoftAzure.es-es/blob/OLPRODLOC/Instructions/Labs/media/l06_aad_autentication_webapp.png
https://github.com/MicrosoftLearning/AZ-204-DevelopingSolutionsforMicrosoftAzure.es-es/blob/OLPRODLOC/Instructions/Labs/media/l06_aad_register_mt_webapp.png


Don't remove spaces when printing a new bottom line with a background color

Turns out we're still being a bit too aggressive when removing spaces.
If there are spaces at the end of the first run painted to a bottom
line, and the bottom line was a different color than the previous,
then we can't trim those spaces off the string. We still need to emit
those to make sure the terminal has colored spaces in it as well.
References

there's like 80 PRs in the last month for this function

PR Checklist

[x] Closes #5502
[x] I work here
[x] Tests added/passed

Validation Steps

[x] ran the tests
[x] checked that vtpipeterm still worked
[x] Checked that the bug was fixed in the Terminal

So in this case, the row we're printing is separated into two runs:

"    #         "

This run is in green-on-blue

"#             "...

This run is in the default attrs

It's that first run's 9 spaces that we tried to optimize with the ECH.
I could certainly see us needing to cache more info, but maybe that's just something we should handle in #2661 when we have all the info in one nice blob

Wow -- merging master actually broke the test (???) Sorry :(


Horn branch

See Comments from Commit. Added theme imp+ Menu Bar

I will look into this and get back to you.

I fixed #1 by commenting it out since there is nothing to do there atm.
#2 i have no idea what is causing that problem. I am using 26, and no there are no local assets.

the findOnPagePreviousDrawable i dont know where that exists, what file is it in.

Thanks, compiles and looks good.


Add Task List syntax

Is your feature request related to a problem? Please describe.
Add Task list syntax

@atapas Can you assigned to me


Cannot use Ollama at all on Windows

On Windows, with the preview build, I installed Ollama, but as soon as I entered “ollama run llama2”, I got this:
Error: max retries exceeded: 400: InvalidArgumentInvalid Argument: range must be positive.
Not sure what to do about this

Can you try again? We had a brief outage yesterday where non-authenticated clients weren't pulling correctly.

This should be resolved now.  If you're still having problems, please let us know and I'll re-open the issue.


Add black and run on luigi_pipeline/tests.

Running only on the tests dir to keep the diff (relatively) small.
Also, don't think it matters too much in this repo but the git blame section was worth a call out.  If we wanted to preserve git blame info it's doable!

I'm gonna go ahead and merge this and fix merge conflicts on https://github.com/broadinstitute/seqr-loading-pipelines/pull/427 manually.


bump TCGC version to 0.48.4

Fixes https://github.com/microsoft/typespec/issues/5261

API change check
API changes are not detected in this pull request.


npm clean-install: Unable to save binary

This issue concerns the installation step 22/28.
When npm clean-install is performed, the execution results in an error since the process doesn't have the required permissions for /app/platform/src/Administration/Resources/administration/node_modules/node-sass/vendor.

I attempted to create the directory manually, but running it again will still result in that error.
Any ideas?

Are you using a local setup or do you use docker?
What permissions are set on the node_modules folder? And are there any other node modules already installed when it crashes?
You definitley need write permissions to the node_modules folder, do you use a different user for running the install command that doesn't has the write permissions?

Some more precise information:

I'm running the installation with Docker
The node_modules directory has the permissions rwx/r-x/r-x and is owned by root
There are modules in the folder, but I'm not sure if they were created during the install
I'm running ./psh.phar install as root in the container (after accessing it with ./psh.phar docker:ssh)
I started the Docker daemon via sudo dockerd

node_modules should not be owned by root. Did you run ./psh.phar docker:ssh as root?
To fix the permissions for your current user, run this with your development directory:
sudo find $DEVELOPMENT_DIR -not -user "$(id -u)" -and -not -user 999 -exec chown "$(id -u):$(id -g)" {} \;
This command changes the ownership of all files and directories that are not owned by your current user (id -u) or 999 (uid of the mysql user in the container) to your current user.

Did you run ./psh.phar docker:ssh as root?

Yes. The reason is that otherwise (logged in as application), I don't have permissions to some composer directories:
[ErrorException] copy(./composer/cache/files/composer/xdebug-handler/...): failed to open stream: Permission denied

It seems like your ~/.composer and maybe your ~/.npm directory is owned by root. ./psh.phar docker:start should check that and fail if that's not the case.

The error is fixed now.
Don't run any ./psh.phar scripts using sudo. Otherwise the permissions for node_modules are invalid.
All scripts must work without sudo. If  docker:start does not work without sudo, make sure your user is in the system's docker group.


Context menu missing for drives

The context menu is available on directories only, not on drives. This is easy to fix in the InnoSetup sections. The net result should look like this:
Windows Registry Editor Version 5.00

[HKEY_CLASSES_ROOT\Drive\shell\WinThumbsPreloader]
"MUIVerb"="WinThumbsPreloader"
"Icon"="\"C:\\Program Files\\WinThumbsPreloader\\WinThumbsPreloader.exe\",0"
"ExtendedSubCommandsKey"="Drive\\shell\\WinThumbsPreloader"

[HKEY_CLASSES_ROOT\Drive\shell\WinThumbsPreloader\Shell]

[HKEY_CLASSES_ROOT\Drive\shell\WinThumbsPreloader\Shell\Preload]
"MUIVerb"="Preload thumbnails"
"Icon"="\"C:\\Program Files\\WinThumbsPreloader\\WinThumbsPreloader.exe\",0"

[HKEY_CLASSES_ROOT\Drive\shell\WinThumbsPreloader\Shell\Preload\command]
@="\"C:\\Program Files\\WinThumbsPreloader\\WinThumbsPreloader.exe\" \"%1\\\""

[HKEY_CLASSES_ROOT\Drive\shell\WinThumbsPreloader\Shell\PreloadRecursively]
"MUIVerb"="Preload thumbnails recursively"
"Icon"="\"C:\\Program Files\\WinThumbsPreloader\\WinThumbsPreloader.exe\",0"

[HKEY_CLASSES_ROOT\Drive\shell\WinThumbsPreloader\Shell\PreloadRecursively\command]
@="\"C:\\Program Files\\WinThumbsPreloader\\WinThumbsPreloader.exe\" -r \"%1\\\""

Yes, it would be very nice if this worked on drives. I find that it does not work on them whether a) you right-click on the drive icon or b) open the drive's top-level folder and right-click on a blank area within the directory listing.  It's not a "fatal" problem, since even if you have a media storage drive that consists of hundreds of folders at the top level (e.g. by date of photos), you can mass-select all of them and the app will appear in the context menu when you right-click the entire selection, as long as it consists entirely of folders.


javascript popovers require page refresh when coming from editng or visiting a listing

Javascript popovers for front page gallery images, show popover when the user hovers over them on the first visit to the root page. However, after visiting an article/show page, if the user returns to the home page, the popovers will not work without refreshing the page again.

fixed by praveen


Implemented Block equivocation without unit tests

Implemented Block equivocation via creating map per slot to keep information about produced blocks buy authorities in given slot.
Checklist:

[x] I have read the contributing guidelines.
[x] My PR title matches the Conventional Commits spec.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Resolve conflicts from dev branch.


Transpose Matrix Calculator

Project title
Transpose Matrix Calculator
Project Description
A web-app based Transpose Matrix Calculator
Stack:

[x] Html
[x] CSS
[x] JavaScript

/assign


Argmax giving the same result on each pixel

When applying argmax on logits, the same result is for every pixel in the image.
Does anyone know how to solve this?

I also mention that the accuracy is very high during the training and the masks are prepared accordingly(tested with other neural network).

how many classes are you using, are you applying softmax or sigmoid to those logits?

30 classes and applying softmax not sigmoid; to the best of my knowledge sigmoid can be applied in case of overlapping classes, just like in multilabel classification.

Will test again with new testing configuration.


Unable to update color for current/active pagination button

I'm using the AdminLTE red theme but the current/active pagination button is blue, so I'm trying to update it to mesh with the red theme a bit more. I have found the CSS where the datatables site says this is to be updated, however changes don't seem to do anything. I would imagine this is because the AdminLTE implementation is more customized, and this colour is getting picked up from somewhere else, however I just can't find where to update it.
The code that the datatables site says to update is, but changing the colors doesn't change the current/active button color:
.dataTables_wrapper .dataTables_paginate .paginate_button.current, .dataTables_wrapper .dataTables_paginate .paginate_button.current:hover {
  color: #333 !important;
  border: 1px solid #cacaca;
  background-color: white;
  background: -webkit-gradient(linear, left top, left bottom, color-stop(0%, white), color-stop(100%, #dcdcdc));
  /* Chrome,Safari4+ */
  background: -webkit-linear-gradient(top, white 0%, #dcdcdc 100%);
  /* Chrome10+,Safari5.1+ */
  background: -moz-linear-gradient(top, white 0%, #dcdcdc 100%);
  /* FF3.6+ */
  background: -ms-linear-gradient(top, white 0%, #dcdcdc 100%);
  /* IE10+ */
  background: -o-linear-gradient(top, white 0%, #dcdcdc 100%);
  /* Opera 11.10+ */
  background: linear-gradient(to bottom, white 0%, #dcdcdc 100%);
  /* W3C */
}

The button I'm referring to/trying to change from blue is: http://i.imgur.com/bMtoBdp.png
Does anyone have any idea where to update this?

For anyone else trying to figure this out, it's located in bootstrap.min.css. Just search for the following code:
.pagination>.active>span:hover{z-index:3;color:#fff;cursor:default;background-color:#337ab7

I think this feature need be added  @almasaeed2010


Add GTFS frequent stop analysis folder

The core idea of current frequency analysis is adopted from the average headway approach that Carl has been working on for Australian cities, and generalize to work in the global context. Updates are to be expected as required.
TODOs (updated):

[x]  need to think about ways to refactor the code to work with cities with more than one transit feed

bring in frequencies.txt (urban access doesn't do this it seems)
simplified some of the datetime parsing (allowed some things to be written more concisely)
added direction_id, and ensured there were no nulls (to allow grouping over this, avoid logic loop - less code)
comment out the 'representative week' code -- this code caused Mexico City trains not to be processed. We average over dates anyway, so I think this is fair.
weight hours in frequency.txt (for taking weighted average of headway for hours within analysis window; hours outside analysis window are weighted zero --- but before then, any zero weight rows are dropped anyway)
if there are stops with valid trip frequencies and positive weight defined, these will be used; any other stops are defined using stop_times if possible
Updated GTFS configuration with new 'validation':False parameter.  It happens to be set False for all layers, however the main impact is only observable for Odense and Bern.  By setting validation to False, the code now creates a 'cleaned' version of stop_times with stops restricted to those within the limits of the current city's bounding box.

[x] need to discuss transit mode specifications used for the global context

amended modes to reflect GTFS standard route types by default (modified where exceptions occur)

[x] need to validate correct school terms for each study region

[ ] clean and test the updated code with available GTFS cities, further changes may be expected to make

@gboeing could you update the docker with UrbanAccess package if you get a chance? I use some UrbanAccess functions to load the GTFS data in this workflow. It seems like now I need to reinstall the package every time I rerun the docker container.
Thanks!

could you update the docker with UrbanAccess package if you get a chance?

@shiqin-liu yes I can. But I might suggest we see if anything else needs to change in the docker image, so we update everything at once. For instance, I'm planning to release the new version of OSMnx in about 10 days: should we wait for that to incorporate the latest in the image? Any other changes to make?
cc @carlhiggs

Thanks, @gboeing I agree it is good to wait to incorporate more updates. On my end, I cannot think of other changes other than UrbanAccess for now, but that's is not urgent. Excited to know about the new version OSMnx release, should we expect any significant changes in our workflow if incorporating this new version (other than the POIs and clean intersections we know of)?

Excited to know about the new version OSMnx release, should we expect any significant changes in our workflow if incorporating this new version (other than the POIs and clean intersections we know of)?

@shiqin-liu you can see the draft change log for what's new in the release as currently planned. It's a major refactor, but most of it should be seamless for the end user. A couple function names have changed, and file save/load filepath parameters have changed. You can see the proposed release functionality in the latest docs.

Per my action item, I spent some time this morning updating and rebuilding the docker image. It is available now to pull via the latest tag. See commit 0001e71b0a1a15c8f8bc088a6b777aff40cc8132 for changes.
cc @carlhiggs @nicholas-david @duelran @DRosasMoctezuma

@shiqin-liu see comment: https://github.com/shiqin-liu/global-indicators/pull/7#issuecomment-682067842
It looks like this is an issue with this PR's diff.

Thanks, Geoff. I must have missed the notification for the https://github.com/shiqin-liu/global-indicators/pull/7. I have reviewed and merged the updated PR Carlhiggs gtfs frequency patch 1 by Carl. They look good to me on my branch. Is there still a problem with this PR's diff? should I close this for now?

@shiqin-liu yes, see the diff at the top of the PR's page: +2,444,647 −0
It's adding millions of lines to the repo because all the data files are included in the commit history.

OK, I see... I will close this for now then, I am pretty sure I ignore all the .txt data file, but I will check on my branch again, and make sure all are good before making a new PR.

Note that you'll need to remove it from the commit history, not just ignore it, because it's already in there.


Additional methods for ObservableList to add elements

🆕🐥:dog: First Timers Only
This issue is reserved for people who never contributed to Open Source before. We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you 💝
:space_invader: Description of the issue
The JavaFX ObservableList interface adds some nice methods to the list interface like
add(int index, E... elements) that we should add to out ObservableList interface and implementation.
📋 Step by Step

🙋 Claim this issue: Comment below.
🔄 replace the up for grabs label with in progress.
:fork_and_knife: fork the repository in github by simply clicking the 'fork' button.
:arrow_heading_down: check out the forked repository
:twisted_rightwards_arrows: create a feature branch for the issue. We do not have any naming definition for branches.
💾 Commit your changes.
🔀 Start a Pull Request.
🏁 Done 👍 Ask in comments for a review :)
🔬 If the reviewer find some missing peaces or a problem he will start a discussion with you and describe the next steps how the problem can be solved.
:boom: you did it! We will merge the fix in the master of the Dolphin Platform project.
:bouquet: Thanks, thanks, thanks for being part as an open source contributor for Dolphin Platform

🤔❓ Questions
You can join our community chat at Slack

I'll try working on this guy.

Though, I am a bit puzzled, as there is no method with a signature of add(int index, E... elements)  in JavaFX ObservableList: https://docs.oracle.com/javase/8/javafx/api/javafx/collections/ObservableList.html

lol, you are right. So the methods that are part of the JFX interface should be added ;)

see https://github.com/canoo/dolphin-platform/pull/682


strange behavior of include and theninclude funtions

I have a query like belove on my database context:
             post.Include(p => p.User)
                        .ThenInclude(p => p.UserProfile)

                        .Include(p => p.User)
                        .ThenInclude(p => p.UserTeams)
                        .ThenInclude(p => p.Team)

                        .Include(p => p.WarWinner)
                        .Include(p => p.WarDeclarer)

                        .Include(p => p.SharePosts)

                        .Include(p => p.PostDisableCoins)
                        .ThenInclude(p => p.User)
                        .ThenInclude(p => p.UserTeams)

                        .Include(p => p.Likes)

                        .Include(p => p.Comments)
                        .ThenInclude(p => p.CommentLikes)

                        .Include(p => p.Comments)
                        .ThenInclude(p => p.User)
                        .ThenInclude(p => p.UserTeams)

                        .ThenInclude(c => c.Team)
                        .Include(p => p.Comments)
                        .ThenInclude(p => p.User)
                        .ThenInclude(p => p.UserProfile);

until now the is nothing special things in code, but after running query post.User.UserTeam is null, actualy query doesnt load data, and in sql provider I have seprated queries for loading commentlikes that I expect just one query when I use include.

EF Team Triage: This issue is lacking enough information for us to be able to effectively triage it. In particular, it is missing the following information requested in the new issue template. Can you please provide this information?

Steps to reproduce

Ideally include a complete code listing that we can run to reproduce the issue.
Alternatively, you can provide a project/solution that we can run.
BTW we're not just doing this to be mean :smile:... we get a lot traffic on this project and it takes time to attempt to reproduce an issue based on fragments of information. In addition, our attempt is often unsuccessful as the exact conditions required to hit the issue are often not explicitly included in the code provided. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we ask that folks give us a self-contained way to reproduce an issue.
For a guide on submitting good bug reports, read Painless Bug Tracking.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Register

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/32923
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" f948ec122662ea77602e5e8123cc46d6573e6b55
git push origin v0.1.0

@JuliaRegistrator register

Registration pull request updated: JuliaRegistries/General/32923
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 6b39f38e00b8bb4fd8a4b1379c5e2a5f6e079b93
git push origin v0.1.0


[on-chain config] test for e2e pub/sub flow

Summary

Tests for on-chain config e2e pub/sub flow (#3108), and executor's commit returning events for reconfiguration
Extracted shared test code with executor as test helpers

Once we have more on-chain configs defined in Rust-side, we should add more test-cases

@bors-libra r=runtianz

:pushpin: Commit 5888c0d has been approved by runtianz

:bulb: This pull request was already approved, no need to approve it again.

There's another pull request that is currently being tested, blocking this pull request: #3237

:pushpin: Commit 5888c0d has been approved by runtian-zhou

@bors-libra r=runtian-zhou

:bulb: This pull request was already approved, no need to approve it again.

There's another pull request that is currently being tested, blocking this pull request: #3237

:pushpin: Commit 5888c0d has been approved by runtian-zhou

:hourglass: Testing commit 5888c0df6d342990db0e10844c79014b0adcfa01 with merge bcf18904d6e739adc5a36877d286fb6cbf219de1...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: runtian-zhou
Pushing bcf18904d6e739adc5a36877d286fb6cbf219de1 to master...


[FLINK-16011] fix the bug that within will not effect if not in the end of a pattern

What is the purpose of the change
This pull request fixes the bug that In a CEP Pattern, if there is no within() in the end of a pattern, the within() will not effect.
Verifying this change
This change added tests and can be verified as follows:

Added test in NFACompilerTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Hi @tillrohrmann , thank you for the review and improvements, I'm quite OK with your solution.


provide custom library path

I have installed TensorFlow using "pip install tensorflow==1.13.1"
I am trying to run the TensorFlow with specific version 1.13.1
but it failed due to incompatible due to "ImportError: /lib64/libc.so.6: version `GLIBC_2.16' not found".
I have already installed the glibc on path /opt/glibc_2.16/
If, I gave custom path using "export LD_LIBRARY_PATH=/opt/glibc_2.16/lib:$LD_LIBRARY_PATH"
Then "Segmentation fault (core dumped)" is shown.
How can I provide this path to tensorflow?
Please help.
(base) [rakesh@localhost rasax]$ ls /opt/glibc_2.16/
bin  etc  include  lib  libexec  sbin  share  var
For error reporting during run is given below:-
(base) [rakesh@localhost rasax]$ python -m tensorflow
Traceback (most recent call last):
File "/home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "/home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "/home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "/home/rakesh/anaconda3/lib/python3.7/imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "/home/rakesh/anaconda3/lib/python3.7/imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: /lib64/libc.so.6: version `GLIBC_2.16' not found (required by /home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/rakesh/anaconda3/lib/python3.7/runpy.py", line 183, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "/home/rakesh/anaconda3/lib/python3.7/runpy.py", line 142, in _get_module_details
return _get_module_details(pkg_main_name, error)
File "/home/rakesh/anaconda3/lib/python3.7/runpy.py", line 109, in _get_module_details
import(pkg_name)
File "/home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/init.py", line 24, in
from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
File "/home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/python/init.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "/home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "/home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "/home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "/home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "/home/rakesh/anaconda3/lib/python3.7/imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "/home/rakesh/anaconda3/lib/python3.7/imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: /lib64/libc.so.6: version `GLIBC_2.16' not found (required by /home/rakesh/anaconda3/lib/python3.7/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so)
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

It looks like you are installing TensorFlow (TF) prebuilt binaries.
TensorFlow release binaries version 1.6 and higher are prebuilt with AVX instruction sets.
Therefore on any CPU that does not have these instruction sets, either CPU or GPU version of TF will fail to load.
Apparently, your CPU model does not support AVX instruction sets. You can still use TensorFlow with the alternatives given below:

Try Google Colab to use TensorFlow.

The easiest way to use TF will be to switch to google colab. You get pre-installed latest stable TF version. Also you can use pip install to install any other preferred TF version.
It has an added advantage since you can you easily switch to different hardware accelerators
(cpu, gpu, tpu) as per the task.
All you need is a good internet connection and you are all set.

Try to build TF from sources by changing CPU optimization flags.

Please let us know if this helps.


Script for training embeddings

Hi there,
Thanks for uploading the NER Tagger! I'm trying to build on the performance of your model for German. You already provided the pre-trained embeddings in issue #44 , however, I want to extend your corpus with some more text. Is it possible for you to upload the script with which the embeddings were produced?
Thank your very much!
@glample
@pvcastro
@julien-c

Sorry, I'm just working with Portuguese language, can't help you with scripts for German!

Ok!
I'm actually looking for the original script with which the embeddings were trained on the Leipzig corpora collection & German monolingual training data from 2010 Machine Translation (according to the paper).

Hi,
We trained our embeddings using the wang2vec model, you can find it here:
https://github.com/wlin12/wang2vec

Thank you! And do you have your preprocessing script with which you produced the texts for wang2vec? I want to exactly reproduce the GER64 embeddings (and therefore the results) for the NER tagger.

Sorry I don't remember about the preprocessing :/
But I think we only used the Moses tokenizer: https://github.com/moses-smt/mosesdecoder/

Ok, thank you! And what about the the param settings for wang2vec, including the window size (should be different for Geman than for, say, English).
./word2vec -train input_file -output embedding_file -type 0 -size 50 -window 5 -negative 10 -nce 0 -hs 0 -sample 1e-4 -threads 1 -binary 1 -iter 5 -cap 0
Do you have them?

Parameters can be same for all languages. You should use -type 3 and also -size 50 is the dimension of your embeddings so you probably want more than that. GER64 uses 64 typically, but higher might be better.

Which versions of the Leipzig corpora collections have you used? Minus "web", there are 4 text sources ("wiki,news,newscrawl,mixed") each consisting 30k to 1M sentences. Have you by chance used only the 1M variants and merged all 4 documents?


[Feature Request]: Open Parties and Claims integration

Is your feature request related to a problem?
https://www.curseforge.com/minecraft/mc-mods/open-parties-and-claims
Solution(s)
No response
Describe alternatives you've considered
No response
Mod Version

Mod Loader Version

Mod Loader
Forge
Additional context
No response

Open Parties and Claims already has build-in support for argonauts


fix for #41

This change is

tks seems to be working

Comments from Reviewable

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

Comments from Reviewable


User without permission can see Edit button on organization profile

On the Organization Profile, the Edit button is visible to anonymous users:

Steps

As an anonymous user, navigate to an organization
Click on Edit button (place is near organization name)
Try to edit data

Found

Form is opened
After editing and saving nothing is happened
These is error on browser console about "Access denied [403]"

Expected

Don't show Edit button to any users, only for limited audience

Task
Make sure the Edit button only displays for Organization Managers or Admin users. there may already be an organization.currentUserCanEdit helper, otherwise we may need to create it.

@bajiat interested

@bajiat , verified this resolved issue on https://staging.apinf.io version 0.39.0 and it is not reproducible.


⚠ Bofrost.lu has degraded performance

In 96e7908, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1032 ms

Resolved: Bofrost.lu performance has improved in 6a12418 after 15 minutes.


Add RAI QA RAG and non-RAG flows

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
All Promptflow Contribution checklist:

[x ] The pull request does not introduce [breaking changes].
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

for your convenience, added some request changes following Clement's comment


Fixing typos in readme

Hi @fussinatto, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Hi @fussinatto, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


All competencies

Popular competencies component, All competencies component, breadcrumbs, navigation from explore page

Don't merge now, need to do some changes

Please review and merge now..changes are pushed.


Update SecurityGroupPolicy.spec.securityGroups Interface to accept Security Group Names

What would you like to be enhanced:
Currently the SecurityGroupPolicy only supports hardcoded Security Group IDs.
apiVersion: vpcresources.k8s.aws/v1beta1
kind: SecurityGroupPolicy
metadata:
spec:
  securityGroups:
    groupIds:
      - sg-0123456789

It would be handy if SecurityGroupPolicy.spec.securityGroups exposed a way to look up a security group by tags or by security group name, rather than just by ID.
Why is the change needed and what use case will it solve:
I am not familiar with an easy way to programmatically get the Security Group ID into this resource definition. Anything I come up with is fairly convoluted.
Additionally, an actual security group name would be more explicit than an arbitrary ID.
Finally, if sg-0123456789 is destroyed and re-created, I need to update the Security Group ID. However, if the Security Group is created with the same name, this resource would not need to be updated.

Thanks for the report!
This is being tracked in #20.


Synthesize Results from CSV Uploader Pilot

Problem statement
As the Experience Team, we need to synthesize the results collected from the piloting of the CSV Uploader feature for our Cohort 2 group so that we can guide direction of the ReportStream product.
What you need to know

Lots to look into as we have thus far not tested any of the uploader with users.
We should gather feedback and metrics from not only the Sender's interaction with the CSV Uploader but also the onboarding process and the delivery of results to PHDs

Ideas to start with:

Are users able to upload a csv?
Do error messages make sense?
What is working well about the csv upload?
What could be improved about the csv upload process?

Acceptance criteria

[ ] Synthesize results

@BerniXiongA6 tagging you here so you can see the original ticket on this
https://cdc.sharepoint.com/:p:/r/teams/USDSatCDC/_layouts/15/Doc.aspx?sourcedoc={3883912A-317A-4E99-8A2E-187C105C02BF}&file=A1H Pilot Synthesis_1.27.22_Internal.pptx&action=edit&mobileredirect=true