nixos/urxvtd: remove socket activation

This fixes #23193, urxvtd service doesn't work first time by removing systemd socket activation.
Motivation for this change
urxvtd is not compatible with systemd's socket activation feature. When it starts up, it replaces the systemd-created socket with its own, causing a connected urxvtc client to hang. See this comment for a more detailed explanation.
Things done

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

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

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

@rnhmjoj I will inquire with the upstream project. It seems like a pretty easy thing to fix.

@fishyfriend Nice catch, thank you! Could you consider reporting the issue upstream, and opening a backport PR with a cherry-pick of this PR's commit against release-18.09? The problem appears to be present there too.
The procedure for cherry-picking is git cherry-pick -x [your commit], so that the reference to the cherry-picked commit is kept in the new commit.

@Ekleog Thanks for the suggestion, done: https://github.com/NixOS/nixpkgs/pull/51299

I have submitted a patch on the rxvt-unicode mailing list to add socket activation to urxvtd.

Great, thanks! :)


Compatibility with JupyterLab 3.x

Hi @ocoudray @PierreMarion23
I like the idea of this Jupyter plugin. Unfortunately I haven't got it running with Jupyter Lab 3.0.16. Seen that this plugin may be outdated (since the last activity was 4 years ago) I would like to ask:

Is this plugin somehow compatible with newer Jupyter Lab versions, e.g. 2.x or 3.x?
If not: How complex do you assume adding compatibility to these newer versions would be?
Would you be willing to have a look and maybe add this compatibility?

Unfortunately I'm just a Jupyter Lab user and have no clue of the inner workings so I cannot judge this.
Thanks for your work. :)

I'm using ipycanvas now instead.


EnableContentHashing not being considered from appsettings.json

Fixes / New Feature #
Proposed Changes
This pullrequest is to fix the EnableContentHashing not being considered on appsettings.json.
FileCacheOptions.cs didn't have the option to fetch from the config and also the RoutesCreator.cs didn't called the overload with the settings.

Thank you, Thiago!
What issue is it related to?

It is related to the following issue: https://github.com/ThreeMammals/Ocelot/issues/2059

@raman-m @thiagoloureiro I have added a global configuration for the cache options. Since we induced a breaking change by adding the property EnableContentHashing and setting it to false by default, I think the users shouldn't bear the burden. That's why I thought adding the global configuration point EnableContentHashing would help most of them. The documentation will be updated too.

@raman-m It's ready for review. I'm still using Bddfy, it was straightforward. And there is a silly problem the application is expecting TtlSeconds to be explicitly set for each route, since TtlSeconds > 0 -> IsCached.

@raman-m added some unit tests for the CacheOptionsCreator and further suggestions

@raman-m it's sound for me, should I squash and merge?

@ggnaegi Please don't merge yet! Hold on for a 10-20 minutes... I want to present a commit related to the property name.

@ggnaegi The commit found at https://github.com/ThreeMammals/Ocelot/pull/2058/commits/3290aa64f0b2f4b16d929d5acee4ddb10ae1dae5 reflects my earlier suggestion regarding the proposed renaming. I'm confident that this commit does not introduce any breaking changes. Please share your thoughts.

@ggnaegi commented:

it's sound for me, should I squash and merge?

Yes, please do, provided you have no objections to my last commit https://github.com/ThreeMammals/Ocelot/pull/2058/commits/3290aa64f0b2f4b16d929d5acee4ddb10ae1dae5.

Ready for delivery ✅

Code review ✔✔✔
Team approvals ✔✔✔
Unit testing ✔ → CacheOptionsCreatorTests → link
Acceptance testing ✔ → CachingTests  → link to 2 tests
Updated feature docs ✔ → docs/features/caching.rst → link

@raman-m I'm not sure about FileCacheOptions being renamed to CacheOptions. I would prefer keeping FileCacheOptions for now. For me it was all sound and we are going now a step too far. I would prefer having those changes in another PR.

@ggnaegi Understood! Reverting...

@ggnaegi
The recent changes have been reverted. I think we're ready to merge now. 😃

@raman-m Ok, I got it, I just realized I named the Global FileCacheOptions CacheOptions... Well... Are you ok with that, and we announce the community that in a next version we will rename it?

TODO

[ ] Renaming from FileCacheOptions to CacheOptions, after release

thanks for adding all the comments and commits guys!


Add hook for handling/preventing back behavior (Android)

Something similar to https://github.com/vonovak/react-navigation-backhandler with hooks

@satya164 I'm using this in my app and it seems to work, does it feel right to you?
export const useBackHandler = (handler: () => boolean) => {
  const focusState = useFocusState();
  const canHandleBack = (focusState.isFocusing || focusState.isFocused) && !focusState.isBlurring;

  useEffect(() => {
    if (canHandleBack) {
      // Normally, it's fast to unsub/resub
      BackHandler.addEventListener('hardwareBackPress', handler);
      return () =>
        BackHandler.removeEventListener('hardwareBackPress', handler);
    }
  }, [canHandleBack, handler]);
};

Basically, we want to handle android back hardware button, when focusing/focused, but not blurring (because the screen is already leaving anyway).
That seems pretty similar to the lib of @vonovak that handles back between willFocus and willBlur.
As far as I remember in v5 you don't have focusing/focused/blurring/blurred transition? so how do you know how to handle back during those transitions?

@satya164 btw that seems like a nice usecase for implementing your useFocusEffect you discussed here: https://github.com/react-navigation/hooks/issues/9#issuecomment-533919336
Does this feel similar to what you have in v5?
// Useful to access the latest user-provided value
const useGetter = <S extends {}>(value: S): (() => S) => {
  const ref = useRef(value);
  useLayoutEffect(() => {
    ref.current = value;
  });
  return useCallback(() => ref.current, [ref]);
};

export const useFocusEffect = (
  effect: () => void | (() => void | undefined),
) => {
  const getEffect = useGetter(effect);
  const focusState = useFocusState();
  const runEffect =
    (focusState.isFocusing || focusState.isFocused) && !focusState.isBlurring;

  useEffect(() => {
    if (runEffect) {
      return getEffect()();
    }
  }, [runEffect, getEffect]);
};

export const useBackHandler = (backHandler: () => boolean) => {
  const getBackHandler = useGetter(backHandler);
  useFocusEffect(() => {
    const latestBackHandler = getBackHandler();
    // Normally, it's fast to unsub/resub
    BackHandler.addEventListener('hardwareBackPress', latestBackHandler);
    return () =>
      BackHandler.removeEventListener('hardwareBackPress', latestBackHandler);
  });
};

does it feel right to you?

I think it's okay. The disadvantage here is that it adds 2 extra re-renders on mount and 1 on unmount which can affect the animation smoothness (even if it's native driver).

As far as I remember in v5 you don't have focusing/focused/blurring/blurred transition?

We don't have focusing and blurring events, but we still fire focus and blur events in addition to transitionStart and transitionEnd events which are specific to stack.
useFocusEffect subscribes to these events internally and manages running the effect: https://github.com/react-navigation/navigation-ex/blob/1153d5575b5d37b5c889158600162be8132b61eb/packages/core/src/useFocusEffect.tsx#L16

Does this feel similar to what you have in v5? In my app it seems to work fine.

The problem is extra re-renders. Even with re-animated we've found that these re-renders during animation really hurt the smoothness.

Should we cleanup that focus effect at the beginning or end of the blur transition?

In v5 the focus and blur events fire at the beginning of the transition. I think this is a good place to cleanup the listener. Since the previous screen is already losing focus, no reason it needs to keep listening to the event.

Yeah, I noticed too that rendering reanimated nodes can sometimes be costy (but reduced it well memoized). Also noticed that reanimated heavy things takes some time to mount, not sure if this is discussed anywhere?
So you mean I should rather, like your code, not rely at all on useState to avoid any unnecessary render right?
Also, not fan of "useFocusState()", as people will use v5 sooner or later I'd rather expose a useFocused() hook that would just return a boolean. Not really fan of "useFocusState()" and as you said it's likely to trigger unnecessary render + make v5 migration more complicated for users.

Yeah we know about issues with reanimated perf regarding mount/re-render and trying to improve it.

So you mean I should rather, like your code, not rely at all on useState to avoid any unnecessary render right?

It would be ideal, imo

we agreed this backhandler should be implemented by users. This pattern has been added to doc:
const useBackHandler = (backHandler: () => boolean) => {
  useFocusEffect(() => {
    const subscription = BackHandler.addEventListener('hardwareBackPress', backHandler);
    return () => subscription.remove();
  });
};

in https://github.com/react-navigation/hooks/pull/43


Fix the layout for category if there is < 3 products

https://www.dropbox.com/s/r4ep060n6rstxx9/Zrzut ekranu 2017-11-04 18.18.40.png?dl=0

@patrykpiston products should be aligned to the left.


Updating a collection at once

Hi, I couldn't find an example online on how to this so I'm opening this issue.
Basically what I'm looking for is the behaviour ActiveRecord's update_all has, so I don't need to manually load many models in memory and then updating each one separately - which is way more computationally expensive.
The way I thought would be something like this:
const models = yield Model.where('updatable', true);

models.set('field', false);
models.save();

Hi @brenolf
Bookshelf uses Knex underneath to communicate with the DB. So you could use knex for this.
Model.where('updatable', true)
.query() // Get Knex instance related to model
.update() // http://knexjs.org/#Builder-update

Thanks @vellotis. As a matter of fact this was the way I've done, but it seemed less "natural" for me. I'm closing this issue nevertheless.


Added CDX serializer fields

Added CDX serializer support for

Metadata.Tools,
Metadata.Name,
Component.Type (better validation from Node.PrimaryPurpose),
Component.Copyright,

Also added a temporary strings.ToUpper() for HashAlgorithm_value lookup.

lol... always the whitespace...  let me try this again.

Awesome, thanks !

Well done @veramine !


[risk=no] Refactor Role Values

No Jira ticket. Note that this is already a fairly large PR so I am intentionally not trying to refactor everything role-based. Will address more cleanup in separate PRs.
Changes

Moved all references to the current user roles to one place so we're not copy/pasting strings everywhere
Move DACUserRoles -> UserRoles since that's what they really are, roles applicable to any user, not just DAC members

Coverage increased (+0.02%) to 61.984% when pulling 1f606826e43d6f8db387f30e12f8701731938514 on refactor-role-values into 64b5d70e0f373a73dccdb9f214e52c791eb93ec3 on develop.

Thank you!


Peek cannot peek on mapped network drive if PowerToys is running as admin

Microsoft PowerToys version
0.70.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Peek
Steps to reproduce

Map a network drive to a share in an unelevated process.
Launch an elevated app (such as cmd or pwsh) and check that the drive is indeed not available when elevated.
Open File Explorer unelevated.
Navigate to the network drive letter.
Peek any file or folder on it.

✔ Expected Behavior
Peek works as expected.
❌ Actual Behavior
Because the drive is only mapped for unelevated processes, Peek fails to read the file and you get an error message.0
Not sure what a good fix for this is. Given there is also an issue with Mouse Without Borders regarding elevation, a long term solution may be to split PowerToys into two processes, one of which runs code that benefits from elevation, the other which runs the bulk of the code.
Another user reported they could not Peek files inside ZIP files (this is because Peek uses normal file system APIs to read files, so it can't Peek shell objects that don't have real files backing them). So it may make sense to use the shell API to read the files, I am not sure if that would help for this case or not.
Other Software
No response

@SamChaps
Looks like we have to start the module always non-elevated like we did for PT Run.

I wonder if we should restrict Peek from running as admin. Is running File Explorer Windows as admin a thing?
Also, according to ChatGPT, right now we're hitting all of these with Peek:

There might be an interest in keeping Peek available for admin or system windows.

Can replicate this by creating a local network share like this:
net use t: \\localhost\c$\tempshare

When running as admin, Peek won't be able to view these files.

Similar problem occurs if PowerToys is running under a different user after an update (my admin is separate) so accessing files in the personal folder fails as well (though I would have assumed that admins can read everything as they are, well, admins with elevated rights)

@binarynoise , I think the fix I'm proposing should fix you case as well.

Fixed in 0.71 release of PowerToys. aka.ms/installpowertoys


feat: Add metadata parameter to Request for additional context

Description
This PR allows you to hold metadata in the request object. the idea is that you can pull in from the context.request object later in the router.
I could think of a few ways this would be used. In my case I'm scraping an api and I can't garnet the url / ids don't change so I want my own internal id that I want to pass along to when I persist the results.
Someone else had a similar desire to this https://github.com/apify/crawlee-python/discussions/531 thread.
Please let me know what y'all think. if you want me to add any more documentation for it anywhere. I have a solution that I can continue to use without this PR but i see this as a small thing I can do. Thanks for keeping it open source.
Testing

TODO

Checklist

[ ] CI passed

Nope thank you closing.


Productize rapidpro2pg

App Services have written a program similar to couch2pg that pulls data from RapidPro and makes it available in PostgreSQL for ease of analysis. We should consider productizing this solution.

Something new came up up based on partner data use and the use of SMS aggregators.
Most partners want to track how messages are delivered to patients/contacts and this is all housed on the sms aggregators (e.g. Africa's talking) while on RapidPro we only get either wired or failed but not the exact status.
@kitsao are we able to also have the SMS delivery status to RapidPro - CHT?


Feeature request: Add time and version info to exported files

In the output of the tool it is currently unclear at what time (and date) the tool ran and with which version.
It would be nice to add this information to the json output.

Fixed with v.3.0.0


Resolve #277 Part 2 Create mapping after constructor in Sqlitex…

…DatabaseSource
I'm so sorry for missing this in your original commit.

 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.

Looks good thanks!


Azimuth Error

As I attempt get azimuth, the result is more different than SunCalc.org. Suncalc.org says 298.54° but suncalcPy says 2.0824 radians or 119.31°  for intersection of Greenwich and Equator(0,0)

It was based on wrong UTC

It was based on the reference frame. SunCalcPy uses south as reference. Bu SunCalc.org uses north as reference


[AIRFLOW-512] Fix 'bellow' typo in docs & comments

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-512

Testing Done:

N/A, but ran core tests: ./run_unit_tests.sh tests.core:CoreTest -s

👍


Freeze to Transformers 4.0

Remove warnings + errors caused by deprecations.
Do you agree with the frozen versions?

The requirements in the setup.py file are not the same as in requirements.txt.

Good catch. Better?

Otherwise it is looking good, the example works. The tests are passing.


Comments & Flags contains a perpetual spinning icon

Steps to reproduce:

Open any case and filter for small variants in vueapp interface
Open modal interface for any variant
Select the tab "Comments & Flags"

Expected behavior:
Show recent comments and allow adding comments
Actual behavior:
A spinning icon is shown perpetually:

Also adding a text variant comments is being prevented by the spinner.


Request failed, please try again

Type: Bug
Using the https://github.com/chrisdias/thecatsaidnode repo, objective is to upgrade to TypeScript.
Prompt: "upgrade this workspace to typescript. make sure to include all the necessary dependencies and devdependencies, update the scripts, import statements, etc. put the source in the /src folder and the output in the /dist folder. make sure any assets can be found."
Results: failse to finish conversion of www.ts with error "Sorry, your request failed. Please try again. Requestid: 7273aabc-47a0-4d3a-950a-264050931a8f"
Extension version: 0.23.2024120902
VS Code version: Code - Insiders 1.96.0-insider (475acb4b9bdd77d4204224990bf288ed5bfe6b0e, 2024-12-09T22:28:54.579Z)
OS version: Windows_NT x64 10.0.22635
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) Ultra 7 165H (22 x 3072)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
31.64GB (14.75GB free)

Process Argv
--crash-reporter-id 26ae5cbd-0f12-42b8-90be-67559bef8fef

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
vsaa593:30376534
py29gd2263:31024238
c4g48928:30535728
962ge761:30841072
pythonnoceb:30776497
dsvsc014:30777825
dsvsc015:30821418
pythonmypyd1:30859725
h48ei257:31000450
pythontbext0:30879054
cppperfnew:30980852
pythonait:30973460
dvdeprecation:31040973
dwnewjupytercf:31046870
nativerepl1:31134653
pythonrstrctxt:31093868
nativeloc1:31118317
cf971741:31144450
parsetypescript:31116713
e80f6927:31120813
iacca1:31150324
notype1:31143044
dwcopilot:31158714
h409b430:31177054
390bf810:31183120
5b1c1929:31184661

Is there anything in your output channel?

Is this with Chat or Edits?

this was with edits and claude

and there were no logs, i dont think i had the right settings turned on :(


WIP: Adding BulkMutationGCJClient to translate calls to GCJ's Batcher

Initial Implementation of BulkMutationGCJClient

Have added MAX_RPC_WAIT_TIME to 12 mins for IBulkMutation#flush.
Maintaining unresolved response futures instead of their count (to be able to flush).
Have not implemented AutoCloseable for this, As existing BulkMutation doesn't provide the close.

Please use an OperationAccountant instead of a list of Futures.  It ought to be more efficient for your needs.

@sduskis / @igorbernstein2
This PR is ready for a fresh round of review. Please have a look.

@rahulKQL, if this is ready for review, please remove "WIP: " from the title.

@sduskis
Thanks for the feedback, I have addressed them. Could you please have a look.

@rahulKQL, please merge from master.  It looks like there was a conflict with another one of your PRs.

@sduskis
This PR is now ready for final review. Please have a look.


Using grep to limit default NM_SRC_BUILD_TARGET

Using grep to limit default NM_SRC_BUILD_TARGET to tags that match
regex: "^[0-9]+[.][0-9]+[.][0-9]+$" .  This avoids building from tags like "1.20.0-rc" or "1.17.0-unstable"
This is a fix for issue  #1433

My assumption was that someone who source-builds will seek out a specific tag, and the default is just there as a fallback - and yes, this seems fine to change.


Uncaught Error: ENOENT: no such file or directory, lstat '~/hello-world/src/components/header.mod...

[Enter steps to reproduce:]

...
...

Atom: 1.38.2 x64
Electron: 2.0.18
OS: Ubuntu 18.04.2
Thrown From: Atom Core
Stack Trace
Uncaught Error: ENOENT: no such file or directory, lstat '/home/manish/hello-world/src/components/header.module.scss'
At fs.js:1661

Error: ENOENT: no such file or directory, lstat '/home/manish/hello-world/src/components/header.module.scss'
    at Proxy.realpathSync (fs.js:1661:15)
    at Proxy.fs.realpathSync (ELECTRON_ASAR.js:336:29)
    at TreeView.t.exports.TreeView.moveEntry (/snap/atom/232/usr/share/atom/resources/app/static/<embedded>:14:1000267)
    at TreeView.t.exports.TreeView.onDrop (/snap/atom/232/usr/share/atom/resources/app/static/<embedded>:14:1006926)
    at HTMLDivElement.<anonymous> (/snap/atom/232/usr/share/atom/resources/app/static/<embedded>:14:980611)

Commands
  3x -2:31.7.0 tree-view:remove (span.name.icon.icon-file-text)

Non-Core Packages
open-in-browser 0.5.2
open-in-browsers 0.0.30
script 3.18.1
terminal-plus 0.14.5

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/tree-view/issues/1282 so you can subscribe there for updates if you'd like.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


WP 5.2: better colors

"Nearest Neighbor Vertex Colors (NNVC) ist sehr einfach und wird bei kleineren Grid-Resolutions schlechte Ergebnisse liefern. Mit Philipp, Stefan und/oder Michi absprechen, wie man das verbessern kann. Mögliche Lösungen:

Sub-division des Meshes und dann NNVC
automatisches UV-Mapping und Erzeugung von Texturen aus den Punktwolken.
Texturen aus den Fotos?"
Paper: "Texture Defragmentation for Photo-Reconstructed 3D Models" will come to Meshlab
Interessantes Paper: "Point2color: 3D Point Cloud Colorization Using a Conditional Generative
Network and Differentiable Rendering for Airborne LiDAR"

@joschi1212
very interesting paper, combines surface reconstruction and texturing. "Deep Hybrid Self-Prior for Full 3D Mesh Generation": https://arxiv.org/pdf/2108.08017.pdf
Repo: https://github.com/weixk2015/DHSP3D

COLMAP writes the camera poses to binary or text files: https://colmap.github.io/format.html#sparse-reconstruction
I see that the binary version is active in our project, since I have a "cameras.bin" in my local test dir .\upload\4c5c6d38-255e-4708-b5f8-41e029e36340\step1\v000\sparse\0\
so, we have the camera poses already available for further processing.

one more related paper: https://arxiv.org/pdf/2101.10734.pdf

Thanks, I will look into it!

Maybe, we can take some parts from Alicevision/Meshroom: https://github.com/alicevision/meshroom
Here is some more info from their docu: https://meshroom-manual.readthedocs.io/en/bibtex1/node-reference/nodes/Texturing.html
and  https://meshroom-manual.readthedocs.io/en/latest/feature-documentation/nodes/Texturing.html
Their most interesting reference is Seamless Image-Based Texture Atlases using Multi-band Blending: http://imagine.enpc.fr/publications/papers/ICPR08a.pdf

Nerfstudio also has a texturing based on xatlas, see export_textured_mesh(): https://github.com/nerfstudio-project/nerfstudio/blob/4545dcb6b30538013e27ddcdcbcecfc0402c5007/nerfstudio/exporter/texture_utils.py
Not sure if this is better than the current attempt but it combines color extraction and unwrapping. If the NeRF-Variant works, we might switch to Nerfstudio anyway.

we want reconstruction and coloring independent of each other to get the best method for each. we checked several methods and packages but didn't find an easy solution yet.
openmvs: how to replace reconstruction?
xatlas: how to project images on mesh (with texture)?
nerfstudio: how to replace nerf with photo?
TEXTure: how to use?
alicevision: how to use texturing? use pipeline except reconstruction?

found a repo based on a 2014 paper that solves exactly our problem: mesh+ registered images -> texture
https://github.com/nmoehrle/mvs-texturing
https://www.youtube.com/watch?v=Ie-qLJdmlLI
also this improvement, but sadly without public source: https://cseweb.ucsd.edu/~viscomp/projects/SIG17TextureMapping/

still not solved. no available library exactly matches our problem. a seminar paper shows that AliceVision provides the best quality, then OpenMVG. easiest to implement is probably https://github.com/nmoehrle/mvs-texturing


The Visual Studio Code icon... Really become really ugly.....

Perhaps orange represents the future of quantum computing ?

But, indeed, this drab color, brown yellow, is out of tune with other colorful icons.
At the very least, you should provide custom configuration that allows me to replace icons.
Such as an anime beauty? 😂  Ha ha ha!

I think at least blue and brown, or purple！

Same as in #35783 please close your issue

The icon for VSCode 1.16 (the original one) is here.

Disagree


Add SPIRE bundle APIs

Additional functions are located here:
https://github.com/spiffe/spire-api-sdk/blob/main/proto/spire/api/server/bundle/v1/bundle.proto
We will likely wish to add the following APIs:

path
v1 path
spire function

/api/bundle/get
GET /api/v1/spire/bundles
GetBundles

/api/federations/bundles/get
GET /api/v1/spire/federations/bundles
GetFederatedBundles

/api/federations/bundles/create
POST /api/v1/spire/federations/bundles
BatchCreateFederatedBundle

/api/federations/bundles/update
PATCH /api/v1/spire/federations/bundles
BatchUpdateFederatedBundle

/api/federations/bundles/delete
DELETE /api/v1/spire/federations/bundles
BatchDeleteFederatedBundle

Upon review will only add to the new api


Fix contributing typo

resolve

@lukehinds You're right I reverted it.

Thanks @Glyphack , could you squash the commits into one commit please. Details here:
https://github.com/PyCQA/bandit/blob/master/CONTRIBUTING.md#squash-commits


Add codeql semantic analysis

This adds a new CodeQl analysis to our CI for every pull request (https://github.com/github/codeql-action)
Additionally the analysis runs sheduled every night at 2am
The queries can be found here: https://github.com/github/codeql/tree/main/javascript/config/suites/javascript

Note, that "Code scanning results / CodeQL" will not be able before being merged. It's basically comparing the PR with the base branch and informs, how many vulnerabilities have been fixed or new introduced, compared to the base branch.

This was a really good idea @jankapunkt. I didn't even realize it existed so thanks for the head up!


does not work when inline.

This still doesn't work using v0.26.
Example:
Sample text should be slip here  but it is not.

Results in:

Sample text should be slip here but it is not.

Should result in:

Sample text should be slip here

However this works using v0.16

This as designed. You will have to put some newline in there. Not sure why that is a problem.

It used to work.  So this was an unpublished breaking change.  The reason it is a problem is because of the following:
Sample text should be slip here

but it is not.

Sample text should be slip here

but it is not.


[tgui] Change the cmake feature flags

The cmake options that TGUI uses has changed:
https://github.com/texus/TGUI/blob/2aaacf812d71d0856c54df32cc3c5e375f742e59/src/Backend/CMakeLists.txt https://tgui.eu/tutorials/latest-stable/backends/

[X] Changes comply with the maintainer guide.
[ ] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

Changes sourced from:
https://github.com/texus/TGUI/blob/v1.1.0/src/Backend/CMakeLists.txt#L25-L26
https://github.com/texus/TGUI/blob/v1.1.0/src/CMakeLists.txt#L414-L420

Features sdl2,sfml are tested successfully in the following triplet:
x86-windows
x64-windows
x64-windows-static

The usage test passed on x64-windows (header files found):
tgui provides CMake targets:

    find_package(TGUI CONFIG REQUIRED)
    target_link_libraries(main PRIVATE TGUI::TGUI)


What happens when browser is set to larger font size?

I have my browser set to show larger text and the FreeBMD2 search form doesn't look very good:

Compared to FreeREG:

I know FreeREG doesn't try to cram a lot of fields in like FreeBMD2, but still, can we accommodate for this?

EDIT: wrong FreeREG snip uploaded- now replaced.

@PatReynolds to test.
Potential solution is to have an interim between mobile and website.

Hi @DeniseColbert , Made few changes.
Could do a test in test2 and let me know if it is better

Looking good at 175% on a desktop:

Narrower screen, a couple of problems:
main content bar is not responsive:

Scans and Postems and Transcribers not behaving as expected (I would have expected the title "Scans" then the scans, then Postems and Transcribers.

Original issue persists:

Possibly you fixed responsiveness to zoom function @Vino-S ? This appears to be what Pat is referring to in her comments above. However, my screen isn't zoomed at all; I have my browser settings showing larger font.

@Vino-S to implement fix as per @DeniseColbert comment above.

Hi @DeniseColbert,
I had made some changes for Year field in 'Date Range From' and 'Date range to' of the search form. Please have a look and let me know if it is better than the previous implementation.
Ready for testing in test2

@DeniseColbert please take a look when you can.

Still looks the same:

I wonder if we can tell the browser to ignore the text inside the fields when increasing font size? @Vino-S will look into this

@AlOneill in scrum, we discussed removing the placeholder text in the First name/Surname fields. Is there a reason (perhaps for accessibility) we need this text inside the fields?

@DeniseColbert
I forgot to say that it is strictly one label for one field, which is why the labels for dates are invalid: one label for two fields (quarter and year, two separate fields) is broken, is an error and will confuse a screen reader user.
The label "Record type" with for="record_type" is not linked to any field and so is orphaned — also an error.
We must get these details correct or the search form will remain inaccessible to screen reader users.

@DeniseColbert and @Vino-S please pick up on @AlOneill comments and amend markup accordingly.

[ ] Remove placeholder text
[ ] Address "The label "Record type" with for="record_type" is not linked to any field and so is orphaned — also an error"

@Vino-S to do/fix
rror."

@Vino-S @DeniseColbert
The "orphaned label" issue has been fixed and that has been noted as part of another issue (reducing space around record types) — nothing more to do there.

Thanks all


segmented control disabled section

I have copied the mantine docs for segmentedControl.
The whole component disabled feature is already implemented, but the ability to disable individual control segments is not yet merged, see https://github.com/snehilvj/dash-mantine-components/issues/450 .
Therefore this documentation shouldn't be merged until the above request is merged in dash-mantine-components.

Thanks for adding the disabled segments example!
We should also update the Data section to mention the disabled option as well https://www.dash-mantine-components.com/components/segmentedcontrol#data-prop

I have now added this to the data section as requested. But maybe it is a bit of unnecessary complexity for most users, especially since it's right above the new disabled section. It also would differ from the base Mantine docs. Anyway, am happy to leave or revert, whichever you prefer.

I like the update - thanks!
As you noted, we'll just wait for the next release before merging
:woman_dancing:


Is the implementation of Dropout correct?

The Keras implementation of Dropout references this paper: http://www.cs.toronto.edu/~rsalakhu/papers/srivastava14a.pdf
The following excerpt is from that paper.
"The idea is to use a single neural net at test time without dropout. The weights
of this network are scaled-down versions of the trained weights. If a unit is retained with
probability p during training, the outgoing weights of that unit are multiplied by p at test
time as shown in Figure 2."
The Keras documentation mentions that dropout is only used at train time, and the following line from the Dropout implementation
x = K.in_train_phase(K.dropout(x, level=self.p), x)
seems to indicate that indeed outputs from layers are simply passed along during test time. Further, I cannot find code which scales down the weights after training is complete as the paper suggests. I'm hoping somebody can show me said code in the repo and put my mind at ease :-).

@mklawonn
It seems that Keras takes an different approach. Instead of sub-sampling in training and scaled down in testing,  it uses scale-up (by 1/p) at training phrase and not transform in testing.
Here in the implementation of Dropout layer, you see it calls K.dropout. https://github.com/fchollet/keras/blob/master/keras/layers/core.py#L87
And in K.dropout, the output from dropout is scaled up 1/p.  https://github.com/fchollet/keras/blob/master/keras/backend/theano_backend.py#L929

@linxihui Interesting. This is not equivalent to the traditional approach, correct? The activations which fire (i.e are not dropped out) at each layer will output values twice as high as they would otherwise. This should influence the loss and thus the whole training phase differently than simply dropping activations. So in the end you'll probably converge to a different model than if the traditional dropout method had been used. The effects of this procedure are unclear to me.
If I am correct and this alternate dropout method is not equivalent to the original method, I feel it should be made explicit in the documentation. It's possible users (like me) were/are incorrectly assuming that dropout here is the same as dropout in the literature.

You seen very confused. Of course they are equivalent.
On Jul 26, 2016 11:42 AM, "Matt Klawonn"<EMAIL_ADDRESS>wrote:

@linxihui https://github.com/linxihui Interesting. This is not
equivalent to the traditional approach, correct? The activations which fire
(i.e are not dropped out) at each layer will output values twice as high as
they would otherwise. This should influence the loss and thus the whole
training phase differently than simply dropping activations. So in the end
you'll probably converge to a different model than if the traditional
dropout method had been used. The effects of this procedure are unclear to
me.
If I am correct and this alternate dropout method is not equivalent to the
original method, I feel it should be made explicit in the documentation.
It's possible users (like me) were/are incorrectly assuming that dropout
here is the same as dropout in the literature.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/3305#issuecomment-235365130,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArWb41qekaPhq-j-OYGByE_y5YwdFqtks5qZlUBgaJpZM4JULYS
.

@fchollet Indeed I am. Could you humor me and explain why they are equivalent?
Consider network 1 which has dropout applied as Keras applies it, and network 2 which has dropout applied as in the original paper, with all other parameters/hyperparameters/design choices being the same. If the activations in network 1 are being scaled by a factor of 1/retain_probability at train time, then
Point 1: I think the loss and gradients computed by network 1 will be different than network 2.
As a result the weight updates will be different, and
Point 2: I don't see why the weights in network 1 and network 2 would converge to the same state.
Is there something I am missing with point 1 or point 2?
If not, is there something provable about the expected output for the two networks at test time? If there is, it is very much not obvious to me.

@mklawonn This could help: http://cs231n.github.io/neural-networks-2/, see Dropout in the Regularization part. I think it's also done this way in Torch, for example.

@tmannen Thanks for the link! Now I know what to call this alternative (and potentially equivalent) method: Inverted Dropout.
That link, unfortunately, is still lacking in an explanation of why inverted dropout and dropout are equivalent. Now that I know what to call it, however, I have been searching for more information on inverted dropout. I found the following links.
https://github.com/deeplearning4j/deeplearning4j/issues/373
http://stats.stackexchange.com/questions/205932/dropout-scaling-the-activation-versus-inverting-the-dropout
Also, a few other links which simply mention they use inverted dropout as opposed to dropout. I have yet to see anyone even claim that the two are equivalent, much less prove it. It does seem they are doing similar things in principle, but there appears to be no analysis on how they perform relative to one another. Can anyone confirm or deny that
1)They are exactly equivalent.
2)They simply do similar things
Thanks for everyone's help so far!


Make uadetector compatible with udger.com database

Added options in UADetectorServiceFactory to specify the data and version url, including custom fallback database.
Fixed incompatibility issues with current udget.com UAS database including some minor changes to the embedded uas.xml database.

André Rouél » uadetector #216 FAILURE
Looks like there's a problem with this pull request
(what's this?)

Sorry, didn't see the tests in net.sf.uadetector.json. Don't know what the purpose of that module is.
Don't understand yet the failing core tests on buildhive. When running those tests within the IDE both are passing.

hey , as you can see there are two failing tests in net.sf.uadetector:uadetector-core as well

André Rouél » uadetector #217 FAILURE
Looks like there's a problem with this pull request
(what's this?)

I removed some unnecessary tabs and whitespaces and merged it. Thank you for preparing these changes.

Unless Jaroslav allows you to bundle a version of the commercial database (e.g. reduced or X months old), the resource project that in the past included the latest version of the uas.xml database lost its purpose. The factory class could be moved into core and it would simply the project code base and structure.

@before When you release these changes as new version, a lot of the current documentation will need updates including new code examples. I'd be happy to help with that.


:ghost: Add slides from kai-community call

Can you add a readme, that will show someone how to run the slides as a slideshow and the tool you used?


bug: Vercel deployment fails

Is there an existing issue for this?

[X] I have looked through the existing issues

Current Behavior
Running : yarn vercel

Steps To Reproduce
Alternatively to reproduce locally :
Update viem and wagmi to latest version :
1.yarn workspace @se-2/nextjs up wagmi viem
Run check-types :
2.yarn next:check-types
However through above steps you get different type errors then vercel :

Anything else?
I think the reason its failing on vercel deployement and not locally is because since we have ^ for viem and wagmi in package.json because of which on Vercel while yarn install it gets latest patch version of both viem and wagmi and it seems that in latest viem and wagmi its broken.
I think this problem is again related to -> https://github.com/wagmi-dev/wagmi/pull/2421#issue-1724776783

Thanks for reporting this Shiv.
What I don't understand is why Vercel installs a different version of the packages. Even if we have ^ in package.json, it should take the exact version from yarn.lock (that's the goal of this file!)

What I don't understand is why Vercel installs a different version of the packages. Even if we have ^ in package.json, it should take the exact version from yarn.lock (that's the goal of this file!)

Yeah this makes sense !!
I tried going through the deployment logs and found this :

Also if you look at the souce uploaded it dosen't have yarn.lock (I think because we are just uploading nextjs workspace and yarn.lock is present at root ?)

Oh damn, you are right @technophile-04!! Thanks for looking into it.
Maybe we could explore if there is a way to make a dedicated yarn.lock for each package?? (this will also be great for the CLI)
I think we can merge PRs #420 #421 for now

Another option is to make the root yarn.lock avaliable on the Vercel build (I think we look for something similar when playing when moving the scaffold.config.ts to the root of the repo)

Maybe we could explore if there is a way to make a dedicated yarn.lock for each package?? (this will also be great for the CLI)

Yes yes will research on this nicely

Another option is to make the root yarn.lock available on the Vercel build (I think we look for something similar when playing with moving the scaffold.config.ts to the root of the repo)

Yup but with this option we had to mention root directory manually while deployemnt :( checkout point 1. of -> https://github.com/scaffold-eth/scaffold-eth-2/pull/231#issuecomment-1472265152
I have also created some discussion on vercel  regarding this :
https://github.com/vercel/vercel/discussions/10211
https://github.com/orgs/vercel/discussions/2145
Nevertheless, I like the first option more having a dedicated yarn.lock if that's feasible but let's see 🙌

Maybe we could explore if there is a way to make a dedicated yarn.lock for each package?? (this will also be great for the CLI)

I found this -> https://github.com/yarnpkg/berry/issues/1223...I think there is no direct solution to doing this in yarn :( .... There are some hacky solutions mentioned in the issue using some people's plugin but I went through and didn't find any stable, well maintained and regularly used plugin
cc @rin-st @sverps also if they have some better approach or if I am missing and there might an easy solution already (because I feel this should have been present in yarn already).
Also just mentioning pnpm here 😅 (It's workspaces has this option to create separate lock files)  ....I tried it out and its really good also we might also benefit from it using it in our CLI

Also just mentioning pnpm here sweat_smile (It's workspaces has this option to create separate lock files) ....I tried it out and its really good also we might also benefit from it using it in our CLI due to its some what faster installations

We kept using yarn for historical reasons and it was easy to set up. But I won't be opposed to switching to pnpm if we have a good reason for it (like the per-package lock files) It supports monorepos too and it seems to be super fast.
 I just texted Austin to get his take.
But also let's make sure that there isn't an easier solution that doesn't imply switching.

I believe pnpm is a good solution.

@technophile-04 could you draft a PR (against main) for the pnpm "tests" that you made when you have the chance? So we can also see how pnpm looks on SE-2.
I really like the individual lock files, and it seems that's a modern and performant solution.

@technophile-04 could you draft a PR (against main) for the pnpm "tests" that you made when you have the chance? So we can also see how pnpm looks on SE-2.

For sure !!!! lol I deleted all the changes but the migration is pretty much straightforward forward and easy so will draft a PR soon 🙌


Skeletons can't reform

If skeletons die, they're supposed to turn into their skulls, which have the power to reform back into their original bodies.
Instead, currently skeletons turn into ghosts which spawn at the station's origin point upon dying and cannot be cloned. They're instantly gibbed. Essentially you're playing a race with a massive downside.

This was removed with cloning. Skeletons probably need a rebalance of some kind.

might just close this because skeletons are being rebalanced and are not getting reform back as an ability. i'll leave it to someone else's discretion.

i'll close this :suspect:


Update .readthedocs.yml

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

Project coverage is 99.51%. Comparing base (1fe8981) to head (368b387).
Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #49   +/-   ##
=======================================
  Coverage   99.51%   99.51%
=======================================
  Files           8        8
  Lines        1025     1025
=======================================
  Hits         1020     1020
  Misses          5        5


Improve workflow for releases

Howdy folks,
just a small suggestion for improvement:
I recently updated our internal dependencies (linopy is one of them) and stumbled over the version number of linopy. On 03.05.2024 linopy 0.3.9 was released on PyPI. However, there is no mention of this here on GitHub, the release is still 0.3.8... I suggest a more consistent approach in the CD. Especially the release notes for 0.3.9 I really had to search for.
Otherwise thanks, linopy rocks!

Thanks for notifying. I always forget about the GH release. I don't know if there is an automatic solution to make a release whenever there is a new tag. But anyway, thanks


Documentation needs to be updated to reflect [default] profile usage

We ran into an issue where one of our developers had [default] profile credentials already setup in the aws credentials file. Nowhere in your documentation did we see that these would be overwritten.

I ran into this as well.
The following text really threw me off my guard:

To use these credentials, please call the aws cli with the --profile option (e.g. aws --profile 421m/0 ec2 describe-instances)

This lead me to believe my default role was kept intact, even though it was overwritten with the same data as the profile that was just created.
I'd really hate for someone to accidentally throw away important resources due to this. I'm willing to spend some time to fix this if my PR is welcome.

I'm happy to accept PRs here. We don't have an active maintainer atm, and I'm not a Java guy, but I'd be more than happy to merge fixes in to keep things chugging along.


fix: 🐛 menu text in  is not aligned with baseline of other text

Checklist (for the issue creator)

[x] I have verified the bug exists in the latest version of @synergy-design-system/components (e.g. via pnpm install @synergy-design-system/components).
[x] I can provide a simple, reproducible example of the issue (e.g. via code-sandbox or sample code, ...).
[ ] I would be willing to provide a PR for this issue.

Version
The bug occurred with the following version of the component library: 2.9.0
Component
The issue arises when using the following component(s): syn-prio-nav
Environment
The issue occurred with the following operating system(s): all
The issue occurred in the following browser(s): all
Description

https://codepen.io/xkabylgSICKAG/pen/qBeBreZ?editors=1110

This can be fixed by setting display: flex; align-items: center; on the ::part(content-container) of syn-nav-item, but the syn-nav-item inside the prio-nav isn't accessible to our applications.
Acceptance criteria

[ ] All DoD items successfully completed
[ ] All text in the priority menu share a common baseline

@xkabylgSICKAG do you have a real usecase for this?

@klaarseSICKAG
It just looks ugly IMO, when the text hops up a few px when I resize the window. Doesn't seem intentional?

Hi @xkabylgSICKAG, I do not see any jumps in the provided codepen. Am I missing something?

@schilchSICKAG
In the provided codepen I have created two prio navs and set the size of the elements to be so that you can see the issue without having to resize the window. If you manually adjust the size of the prio-nav using dev tools you can see that when it starts to show the "menu" text, that text is not rendered at the same baseline as the text was before resizing.

Okay, I think I see what you mean. Thanks for the input, will have a look into this.
@klaarseSICKAG:
Just use the devtools to adjust the width to something really small, you will see the "more" link will have a visual jump when there is no more navitem visible (approx. 110px)


Change the text

Change the text From
[Please Choice Your Heroes to Type Hero Name]

@Prajwal0225  Please assign me to work on this issue.

Hey @saurav2208 , you can start working on this issue. If it's possible, please aim to create a pull request (PR) within the next 2 days.

@Prajwal0225 Sure!


Implement Account Creation Screen with Username, Email, and Password Fields

Objective: Develop a screen that allows users to create a new account.
Requirements:

[ ] Design and implement an account creation screen with fields for name, email, and password.
[ ] Include a "Create Account" button to submit the registration form.
[ ] Add validation for input fields to ensure they are correctly filled out and meet requirements (e.g., password strength).
[ ] Display error messages for any invalid input or if the account already exists.
[ ] Integrate navigation logic to redirect the user to the main dashboard upon successful account creation.

Acceptance Criteria:

The account creation screen is accessible and displays correctly on various devices.
Users can enter their name, email, and password.
Users receive appropriate error messages for invalid input or existing accounts.
Successful account creation redirects users to the main dashboard.

#26
Closed cause incorrect milestone
Moved to > Epic 1: User Registration


[Spark] Update the Spark library to align with the latest file format design

Proposed changes
In the latest design of GraphAr file format, the vertex_count and edge_count files are introduced for storing the edges.
This change adds related methods in the GraphAr Spark library for reading/writing vertex_count and edge_count files.
Types of changes
What types of changes does your code introduce to GraphAr?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
solve #143

LGTM


Please add a possiblity to run powershell task V2 remotely

Troubleshooting
It would be nice if we could run powershells script remote like
scriptname C:\scripts\myscript.ps1
servers : domain1\server1; domain2\server2
username : (somename from a variable)
password : (password from variable)
Note C:\scripts points to the local drive of the server, not of the agent.
Environment

Server - VSTS or TFS on-premises?
VSTS only
Agent - Hosted or Private:
Private agent
Current agent version: '2.119.1'
Microsoft Windows Server 2012R2 6.3.9600

Issue Description
See above
Error logs

Have a look at the "PowerShell on Target Machines" task.
https://docs.microsoft.com/en-us/vsts/build-release/tasks/deploy/powershell-on-target-machines


On branch edburns-54-test-aztemplate-mac Account for use of greadlink on macOS

Fixes #54
modified:   arm-ttk/README.md
modified:   arm-ttk/Test-AzTemplate.sh

Tested on macOS and Ubuntu (under WSL2).


add break and continue statements

Submisison Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
Add break and continue statements to Stan language.
Intended Effect
More flexible control structures.
How to Verify
Unit tests for uses and illegal uses not in loops.
Side Effects
No.
Documentation
Language reference in the manual updated with behavior, examples, and BNF with side conditions.
Reviewer Suggestions
Anyone.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Awesome. Looks great.


Added a game

Added a small game (sample project) to showcase folks that in this manner they can create new projects (small/big dosen't matter) and upload it Here.

Newbie

...


How to get an email by id

I wanted to how I can I get an email by its id
$oFolder = $oClient->getFolder('INBOX');
$message = $folder->query()->limit(1)->get()->first();
$aMessage = $oFolder->search()->messageId($id)->get();

Tried this but does work

See https://github.com/Webklex/laravel-imap#fetch-a-specific-message

Using this way, I have fetched the specific email by UId
`$folders = $client->getFolders();
        echo "<pre>";

        foreach($folders as $folder) {
            $email = $folder->getMessage($uid = 4663);
            print_r($email);

        }`

But the main issue is -
it returns the email in the object format which is difficult to convert it to an array or to print into the view.

You need to handle that object and display what you need from it. Want to display the subject? Then. just do ->getSubject() etc


Bug: reset-password enhancements and bug fix

Resolved issues:

models are not loaded.  Regression from #736 (https://github.com/meanjs/mean/pull/736/files#diff-d8679454c2d07935926663de0e9cc9a1L34)
callback was not passed corresponding user data.
process exits before mail queue is processed.

Enhancements:

suppress error messages unless debug mailer option is true.
from e-mail address comes from config.
added marker messages to each callback line. (Example: [processed/total] Could not send email for x)
added status message on-complete to tell how many successful e-mails were sent.

LGTM.. tested, and confirmed it's sending the emails.
Thanks @rhutchison

Cool, thanks guys.

@lirantal @rhutchison Slight issue with this script. If there aren't any users in the database, this will hang with no exit. It may seem trivial, but if this script is automated or added to a batch it would reek havoc.
You can add this simple fix after the err check at the beginning of the User.find() callback
if (!users.length) {
      console.log();
      console.log(chalk.yellow('No users were found.'));
      process.exit(0);
    }

I have a more elegant solution to the no users found issue in the PR i submitted to your fork for the user-refactor email migration.

@mleanos please re-test

@rhutchison I'm going to be away from my computer for most of tonight. I'll test it tomorrow morning.

@rhutchison I've tested this and it works great. However, what's the purpose of the alerts()? No alert dialog's popped up for me; is that the intention? If so, I think merely a console log is sufficient.

@mleanos alert is just a mapped function to chalk.green, chalk.red, or chalk.yellow depending on the result status.

@rhutchison thank you. I obviously, didn't pay enough attention. Jumped the gun, when I saw the alert :)
LGTM.

Great. Thanks @rhutchison for fixing this.


Issue with "TAXII Feed" in @site/docs/reference/integrations/taxii-feed.md

Describe the problem
Taac is no longer there.

Page: TAXII Feed
Source: https://github.com/demisto/content/blob/master//Packs/FeedTAXII/Integrations/FeedTAXII/README.md

Screenshots

Environment

OS: [e.g. Windows]
Browser: [e.g. chrome, safari, firefox..]
Browser Version:

Suggested fix

Fixed by this PR


#5312 Added "AddHiddenType" to LogManager class that specifies which types to ignore.

[ ] Waiting for NLog v5.3 (And implementation as LogManager.Setup()-extension-method.

Refactored the SkipAssembly method into a ShouldSkip method to take both Hidden assemblies and types into account. Refactored LookupAssemblyFromStackFrame method into a LookupAssembly method that takes a MethodBase for better performance, as both places that call it use the MethodBase that is retrieved at the top of the method.
Added a HiddenTypeTest that tests the functionality.

Thank you for the pull-request. I propose to move the AddHiddenAssembly away from NLog.LogManager. But instead redirect to these Setup-extension-methods:
NLog.LogManager.Setup().SetupLogFactory(setup => setup.AddCallSiteHiddenAssembly(hideAssembly));
NLog.LogManager.Setup().SetupLogFactory(setup => setup.AddCallSiteHiddenClassType(hideClassType));

If you could resolve the current merge-conflicts, and apply the suggested changes from #5484 then it can be merged for NLog v5.3.
Alternative then I can merge #5484, and manually merge this pull-request after manuall fixing merge-conflicts.

Thank you! Sorry I didn't get to your request above. Thank you so much for supporting the community!

NLog v5.3.1 has been released, that allows you to hide specific class-type, instead of the entire assembly.
Thanks again for the contribution.

Thank you for your hard work!


How to set highlighted font text color in vscode?

Thanks for creating this! It was fun to play around with.
Is there a way to set the highlighted font text color to always be a single color? I'm having an issue with vscode+neovim+leap where the highlighted text takes on the color of the character that it's replacing. In some cases, it results in a color that is very hard to distinguish from the highlight background color.

FWIW, this doesn't seem to be an issue outside of vscode.

I know the problem probably lies somewhere outside the scope of leap.nvim but I was wondering if you had any ideas for what could be worth looking into on the vscode/neovim side.
Solutions I tried
From https://github.com/ggandor/leap.nvim/issues/15#issuecomment-1207485369
I tried adding this to vscode settings.json but that didn't seem to do anything.
"vscode-neovim.highlightGroups.highlights": {
    "Cursor": {
      "backgroundColor": "theme.editor.findMatchBackground",
      "borderColor": "theme.editor.findMatchBorder",
    },
  },

From https://github.com/ggandor/leap.nvim/issues/63#issuecomment-1276254068
Adding lua vim.api.nvim_set_hl(0, 'LeapLabelPrimary', {ctermfg="White", ctermbg="DarkBlue", bold = true }) to init.vim.
Seems to have gotten rid of the highlight background? Didn't fix the font color either.

I was wondering if you had any ideas for what could be worth looking into on the vscode/neovim side

No idea, unfortunately. But I guess the vscode-neovim people can help, you should ask them instead of me :)

For me the hilighted text takes the same appearance as the original font which makes it hard to see, did you end up finding a solution to this?


Faster Huffman encoding

Slightly better optimization of the Huffman encoding method, that emits bits in 8 byte chunks, instead of the current byte by byte implementation. The result is about twice as fast. In addition it also implements encoding in a single pass.

I wrote a short benchmark that encodes a set of actual headers I got from the cloudflare blog:
https://gist.github.com/rust-play/6dd13945e7eb2895ca0b62f85f5809d9
The results are:
Apple M1:
Master: 2.87 s
#1470 : 1.17 s
#1470  + this : 0.83 s
AMD EPYC 7713:
Master: 5.84
#1470: 1.84
#1470  + this: 1.28
So overall we are looking at about a 4x faster header encoding.


[Bug] [api-server] application.yaml about ldap is wrong

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
https://github.com/apache/dolphinscheduler/blob/a28e206166a10010d36a5dc0d9e2fd54f799678b/dolphinscheduler-api/src/main/resources/application.yaml#L150
https://github.com/apache/dolphinscheduler/blob/a28e206166a10010d36a5dc0d9e2fd54f799678b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/security/impl/ldap/LdapService.java#L53
application.yaml is base.dn but in LdapService.java use base-dn
What you expected to happen
use same property.
How to reproduce
3.0.x's bug
Anything else
No response
Version
3.0.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

 @Value("${security.authentication.ldap.base.dn:null}")     private String ldapBaseDn;
It had been resolved.


Article: How to generate code using Roslyn source generators in real world scenarios

https://www.cazzulino.com/source-generators.html

Closed by c9e991dc
Thanks! Next time you can add those via PR :)


Added some additional exceptions

This doesn't fix "\n" being parsed or even empty values.  It throws exceptions when preg_replace fails or does nothing.  I'm not sure if you'd like additional exception types added.  Just provide some feedback if you'd like some changes.

Ping @danielstokes. Are you able to finish this off please?

yeah, I can.  I'll finish it this week.

I've been unable to come up with a test case that actually enters any of these cases. When can this possibly fail?


Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[ ] iOS
[ -] Android
[ ] UWP
[ ] .NET Standard

Version of Library
6.2.6
Version of OS(s) listed above with issue
Android 8.0
Steps to Reproduce

Start GATT Server using sample app and connect to GATT Server using NRF Connect mobile app as GATT Client
Enable Notifications on characteristic from NRF Connect app
Stop GATT Server (GattServer.Dispose())
GATT Client is not getting disconnected

Expected Behavior
Stopping GATT Server would inform the client to disconnect.  I tried to call GattServer.ClearServices or GattServer.RemoveService before calling GattServer.Dispose method but no luck.
GATT Server logs
[BluetoothGattServer] close()
[BluetoothGattServer] unregisterCallback() - mServerIf=6
GATT Client logs - nRF Connect mobile app
Connection parameters updated (interval:7.5ms, latency:0, timeout: 5000ms)
The same implementation is working fine in iOS. The client app is getting disconnected once the GATT Server is stopped by calling Dispose method.
I am not sure its an issue with Android or library. I will test this with native Android GATT Server implementation and raise a PR if i am able to find the root cause.
Actual Behavior
Stopping the GATT Server will notify the client to disconnect

Just a quick update. I have tested Google Sample GATT server app and found that Gatt Client(nRF Connect mobile app) is not getting disconnected after calling GattServer.Close() method. After killing the GATT Server , the advertisement got stopped but somehow android client is still maintaining the previous connection.
I thought of trying a workaround by informing the client app to disconnect (via notification) before stopping the GATT Server.
Any workaround or proper solution to fix this issue?

this is simply the way android behaves and therefore not an issue with the library itself.


Evaluate Random against Dieharder

The source code claims that the sequence of ints generated passes the famous Diehard tests of randomness.
But does it Dieharder?

This is really great work! I'll try to adapt your (code)[https://github.com/smondet/rng-contest] to see if it also passes TestU01


Multiple draggable at the same time?

Hi,
I could not find any info or example.
Would it be possible to have multiple draggable items and drag them all at the same time into a container?
Eg. Select multiple element list with click + command and then drag them all into an empty container.

Yes this is possible but is something you need to implement in consumer-land, refer to https://github.com/clauderic/dnd-kit/pull/588


Bump dependency listr to 0.14.0

Description
I found that dependency listr 0.13.0 cannot work with RxJS 6.
This commit had fixed that problem. In addition, listr is currently in version 0.14.0.
I was wondering if we could bump listr to 0.14.0.
Steps to reproduce
npm i lint-staged -D
npm i rxjs@beta

lint-staged

Debug Logs

expand to view
Error: Cannot find any-observable implementation nor global.Observable.

Environment

OS: macOS High Sierra
Node.js: v9.5.0
lint-staged: v7.0.4

Hey @yenshih, please see #426. We can continue the discussion there, closing this issue.

Great, Thank you so much!


[3781] DQT QTS submission

Context
https://trello.com/c/DnKaKQRZ/3781-m-dqt-qts-submission
Changes proposed in this pull request

Update AwardRecommendationsController to send to DTTP/DQT depending on which
feature flags are enabled.
Create RecommendForAward service which makes the put request to DQT with the
relevant parameters and returns the award date received in the response.
Create RecommendForAwardJob which calls the services and sets the awarded_at
date on the trainee.
Create Dqt::Params::Award params for formatting the request parameters

Guidance to review
DQT API specs:
https://github.com/DFE-Digital/qualified-teachers-api/blob/main/docs/api-specs/v2.json#L8
See /v2/teachers/{trn}/itt-outcome
Important business
~* Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?`
~* Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?~

 [M] DQT QTS submission


Passing parameters via package.json

Hi, I'm trying to pass the "module" as parameter on the package.json but I can't find the proper syntax.
How would you do that ?
Thanks

Ok, I've finally managed to do it like this:
{
  "name": "Blah",
  "version": "1.0.0",
  "devDependencies": {
    "browserify-ng-html2js": "^1.1.0"
  },
  "browserify": {
    "transform": [
      [ "browserify-ng-html2js", { "module": "ngPartials" } ]
    ]
  }
}

So hard to find the correct syntax in Browserify documentation... awful
Anyway, hope it helps someone

Aha, I guess we should add this to the readme

+1


Compilation error, is it a problem with the eigen version?

Thank you very much for your great open source work. Following is my compilation error：

G3Reg/Thirdparty/clique_solver/include/clique_solver/graph.h:242:25: error: ‘using SpConstraint = using SpMat = class Eigen::SparseMatrix’ {aka ‘class Eigen::SparseMatrix’} has no member named ‘coeffs’; did you mean ‘coeff’
G3Reg/Thirdparty/clique_solver/src/clipper.cpp:164:22: error: no match for ‘operator+’ (operand types are ‘clique_solver::SpAffinity’ {aka ‘Eigen::SparseMatrix’} and ‘const IdentityReturnType’ {aka ‘const Eigen::CwiseNullaryOp<Eigen::internal::scalar_identity_op, Eigen::Matrix<double, -1, -1> >’})

I guess it may be about the version of Eigen. Mine is 3.3.7. Could you find your version? It can be seen at Eigen/src/Core/util/Macros.h.

I guess it may be about the version of Eigen. Mine is 3.3.7. Could you find your version? It can be seen at Eigen/src/Core/util/Macros.h.

Mine also is 3.3.7.


Added a new section for Deployment and testing guidelines for Developers

The issue resolved by this Pull Request:
Resolves https://github.com/opendatahub-io/data-science-pipelines-operator/issues/159
Description of your changes:
Added new section with Deployment and testing process for developers.

/lgtm cancel
Looks like there are some merge conflicts and the PR needs rebase, could you please resolve them?

Opening a new PR.


[Event Change Request] Codeunit 99 "Posting Selection Management" OnConfirmPostSalesDocumentOnBeforeSalesOrderGetSalesInvoicePostingPolicy

Describe the request
Hello,
the various events in CU 99:

OnConfirmPostSalesDocumentOnBeforeSalesOrderGetSalesInvoicePostingPolicy
OnConfirmPostSalesDocumentOnBeforeSalesOrderReturnGetSalesInvoicePostingPolicy
...

are unuseful because the following method overrides the custom behaviour. Example:

I suggest to change the order of the events or to add a new event.
The events are missing in the purchase area, please complete.
Thank you,
Simone
Additional context
We need to manage Invoice Posting Policy per document type.Internal work item: AB#495474

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


Steam Flatpak Path with BoilR Flatpak

Fedora Silverblue 36 with BoilR Flatpak
The path for the steam flatpak directory is:
/var/home//.var/app/com.valvesoftware.Steam/.steam/steam/
It's quite a pain to type in so a paste function would be useful. It also doesn't get found automatically if that's possible to add, even better..

I think the solution here is to look for the  /var/home//.var/app/com.valvesoftware.Steam/.steam/steam/ folder automatically, it should be easy enough to add. I will get it done asap.

Thank you!!


Update native code creating a flat map proof [ECR-2313]

Overview
Update the native code converting Rust map proof into Java proof.
Assumes that it's integrated after #469 . Use ca0ec3b31375af5c0467774bee725202753818d9 if need to use before

See: https://jira.bf.local/browse/ECR-2313
Definition of Done

[x] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[x] Public API has Javadoc
[x] Method preconditions are checked and documented in the Javadoc of the method
[ ] The continuous integration build passes

create_java_map_proof_entries vs create_java_map_entries naming seems confusing to me
Yes :upside_down_face:

create_java_proof_nodes (as tree nodes in a contour)? Would you suggest anything?

@vitvakatu I have some troubles with updating to the rustfmt-preview — cargo won't pick it up — would you reformat the thing, please?

@dmitry-timofeev done

Thanks everyone, merged.


Idea/Discussion: generic prop for event callback functions

Currently
We implement event callback props for specific events as needed. We specify those in the PropTypes object as:
Component.PropTypes = {
    ..,
    onMyEvent : PropTypes.func,
    ...
};

Problem
Sometimes one might need to add an arbitrary event listener to a component that we may not have explicitly specified/implemented (e.g. mouseOver/mouseOut).
With our current approach, the solution would be to ensure that mouseOver/mouseOut – and any other events deemed necessary – are  explicitly added  to all components.
This is fine, but requires a lot of duplicated effort. What’s more it’s a little too rigid, for example: what if we want to use mouseEnter/mouseLeave instead?
Idea
We add a generic on prop to all components that accepts an object of event handler functions:
Component.PropTypes = {
    ..,
    on : PropTypes.objectOf( PropTypes.func ),
    ...
};

You could then specify any needed events on the component:
<MyComponent on={ {
    click     : clickHandler,
    mouseOver : mouseOverHandler,
    ...
} } />

The appropriate event listeners would be attached to the appropriate place in the DOM tree rendered by the component (usually the outermost element):
function MyComponent ( { on, ... } )
{
    return (
        <div { ...addEvents( on ) }>
             ...component markup...
        </div>
    );
}

(The addEvents function above would simply return a new object in which each property name is changed from event to onEvent. It could also filter out any invalid event types.)
Alternative implementation
Instead of explicitly adding an on prop we could filter the props that the component does not explicitly use/implement and attach the appropriate event listeners:
function MyComponent ( { children, ...props } )
{
    return (
        <div { ...addEvents( props  ) }>
             { children }
        </div>
    );
}

(Here the addEvents function would filter props and return all properties whose name starts with on and whose value is a function.)

Basically: nope. This is the wrong way of thinking about event handling in our components. Our event callback props should provide abstractions strictly in terms of components and never in terms of their constituent DOM elements.
If someone wanted to add an arbitrary listener for DOM-level events, they could simply wrap the Nessie component in a <div> with the appropriate event listener (not that you should generally need to do such a thing).


Docker build doesn't work when repository has a "." in it

Are you experiencing an issue with...

[ x] shields.io
:beetle: Description

:link: https://img.shields.io/docker/build/russ0519/docker-airflow-2.7.svg

:bulb: Possible Solution
I'm guessing the regex is not escaping the "." properly.

Could this be the issue from #1400?

@paulmelnikow Thank you, it was.


[Go Live][June 28]

Push to live

Docs Build status updates of commit 3726140:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/adaptive-cards/manage-bot-label.png
:white_check_mark:Succeeded

msteams-platform/assets/images/azure-bot.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/createazurebot.png
:white_check_mark:Succeeded

msteams-platform/assets/images/bots/MicrosoftAppID.png
:white_check_mark:Succeeded

msteams-platform/assets/images/bots/ngrokURL.png
:white_check_mark:Succeeded

msteams-platform/assets/images/conversation-bot/azure-bot.png
:white_check_mark:Succeeded

msteams-platform/assets/images/conversation-bot/manage-bot-label.png
:white_check_mark:Succeeded

msteams-platform/assets/images/conversation-bot/ngrok.png
:white_check_mark:Succeeded

msteams-platform/assets/images/manage-bot-label.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/meeting-content-bubble/createazurebot.png
:white_check_mark:Succeeded

msteams-platform/assets/images/meeting-content-bubble/meeting-content-bubble-messaging-end-point.png
:white_check_mark:Succeeded

msteams-platform/assets/images/meeting-content-bubble/meeting-content-bubble-view-configuration.png
:white_check_mark:Succeeded

msteams-platform/assets/images/meetings-side-panel/config.PNG
:white_check_mark:Succeeded

msteams-platform/assets/images/meetings-side-panel/createazurebot.png
:white_check_mark:Succeeded

msteams-platform/assets/images/meetings-side-panel/ngrok1.png
:white_check_mark:Succeeded

msteams-platform/assets/images/meeting-token-generator/sbs-azure-home-bot.png
:white_check_mark:Succeeded

msteams-platform/assets/images/meeting-token-generator/sbs-microsoft-app-id.png
:white_check_mark:Succeeded

msteams-platform/assets/images/meeting-token-generator/sbs-ngrok-page.png
:white_check_mark:Succeeded

msteams-platform/assets/images/ngrok-url.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/sbs-messagingextension-action/manage.PNG
:white_check_mark:Succeeded

msteams-platform/assets/images/sbs-messagingextension-action/ngrok.PNG
:white_check_mark:Succeeded

msteams-platform/assets/images/teams-file-upload-bot/azure-new.png
:white_check_mark:Succeeded

msteams-platform/assets/images/teams-file-upload-bot/manage.png
:white_check_mark:Succeeded

msteams-platform/assets/images/teams-file-upload-bot/ngrok.PNG
:white_check_mark:Succeeded

msteams-platform/assets/images/Teams-link-unfurling/config1.png
:white_check_mark:Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Point trait:  correct doc comment

[x] I agree to follow the project's code of conduct.
[ ] I added an entry to rstar/CHANGELOG.md if knowledge of this change could be valuable to users.

Thanks!


Add models for FreeRTOS GSG's

Thank you for contributing to the Azure IoT Plug and Play Models repository
This project has adopted the Microsoft Open Source Code of Conduct. For more information see the Code of Conduct FAQ or contact<EMAIL_ADDRESS>with any additional questions or comments.
The repository pipeline will ensure minimum validation of incoming DTDL models.

PR validation steps are described in the tools Wiki.
The same validation can be run locally via the dmr-client CLI.

Requested info template for model(s) submission
When submitting models to the repository we ask that you provide as much of the following meta information around your models and related devices as possible. This info will be used to improve Plug and Play.
:star2: Please replace the markdown comment examples with your own values.
Company Info
Microsoft
IoT Plug and Play Device Info
Azure IoT FreeRTOS middleware samples
https://github.com/Azure-Samples/iot-middleware-freertos-samples
Model Submission Goals

Guidance to users for connecting FreeRTOS devices to Azure IoT

Code Owners & Reserved Names
NA, CODEOWNERS has been removed.

LGTM


Documentation Out of Date for 1.0.*

I'm trying to figure out what's going on here so I can bump the installed version of this to the new 1.0.* major (which, congrats on releasing, by the way).
In the change log I see:
Raven.config(...) instead of new raven.Client(...)
Raven.install() instead of client.patchGlobal()
However it doesn't look like the documentation reflects these changes yet. Maybe we can get a better idea what the new usage is supposed to be for this?
Thanks!

I'm still finishing the complete docs (glad you noticed, sorry for the delay!), but most of the new usage is covered in the forum post where we announced it and the prior 1.0.0-beta post.
I'll keep this open until those docs are updated (likely later today).

Ah, I didn't even know you guys had a forum but that looks like a better description of what I'm looking for. Thanks Lewis!

Docs have been updated here - have a look and please feel free to open issues related to typos, thinly-covered sections, possible clarifications, etc!

@LewisJEllis I'm still seeing the old documentation at the link you provided. Is it possible it has reverted back for some reason?

Hmm, sure enough, I'm seeing the old docs now too - that's really strange. I'll investigate.

@imbstack it looks like the repo where we combine all our different raven- docs somehow got a couple of its submodules kicked back to November in a recent change, so raven-node and raven-python docs were a few months out of date. Fixed - thanks for reporting.

Thanks very much for the quick fix!


Upgrading to ws package to a newer safe version

Github have tagged all versions of ws which < 3.3.1 as vulnerable (High severity). It would be nice to upgrade mhub to work with a newer version.

Thanks for the heads up, good idea! I'll try to release it this week.

Done! 1.0.1 is just released.


New ProductProjection model

Description
As part of the effort to remove the local test data models we have in the Merchant Center monorepo, we noticed the ProductProjection model did not exist yet in this repository, so this PR introduces it.
Here is the public technical documentation about this model.

I think you forgot to update the CODEOWNERS file.

I think you forgot to update the CODEOWNERS file.

Updated here: 6651adb3e276eeb267c33c290b18a519b0177a6d


Starlink API

This isn't certainly an issue but I was curious if one of the devs could provide me with a more straightforward API to interact with my car where I can send remote commands through an easy python script from my terminal.

I'm not sure I understand the ask. Are you looking for more documentation beyond that described in DEVELOPERS.md? Or are you asking for the python package to be rearchitected to make its API "more straightforward"?.
If all you want to do is send remote commands via a shell, then why not just use the provided CLI utility?
# example to lock the doors
$ subarulink lock

I just figured it out. I was a bit confused about using it via a shell! But thank you again! Appreciate the help!


updating header component to use wcm text and image component

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[ ] I ran all tests locally and they pass.

This change is included in the PR: https://github.com/adobe/aem-core-forms-components/pull/779


AttributeError: 'module' object has no attribute 'prototype_search_with_coverage_state'

Hello,
I am trying to run your code but an error (written in title) occurs in train.py file.
Any idea?
B.R.
p.s. running on python2.7, theano is in the latest version.

Hi,
Have you installed NMT (''python setup.py develop'') before running the code? If you installed GroundHog before on the same server, please run "python setup.py develop" to install the NMT code. If it doesn't work, maybe you can build a new Python environment (e.g., using Anaconda) and install the NMT code in the new environment.
Best,
Zhaopeng

Solved! It was libblas lib which is in need by scipy.
Thank you.


Cleanup calendar model

Changes:

Rename calendarData to data
Use CalendarComponent only
Use diamond operator in two places

Also fixed TripsTest, I do not know why it was working for you, but I had the same issue as in #2750

Merged, thanks.


Strange AttributeError

I'm running a fairly complicated pipeline but it works surprisingly well.
( this dag calls this dag which calls this dag )
However, sometimes, I encounter this error below on the DAG. All the tasks, however, complete just fine.
Any idea what this may come from? Have you seen this before? Any tips on debugging? I will also keep searching myself but I thought I'd flag this.
Traceback (most recent call last):
  File "/home/xf11bm/miniconda3/envs/lightflow-pipeline/lib/python3.6/site-packages/kombu/serialization.py", line 50, in _reraise_errors
    yield
  File "/home/xf11bm/miniconda3/envs/lightflow-pipeline/lib/python3.6/site-packages/kombu/serialization.py", line 263, in loads
    return decode(data)
  File "/home/xf11bm/projects/lightflow/lightflow/queue/pickle.py", line 15, in cloudpickle_loads
    return load(BytesIO(s))
AttributeError: type object 'int64' has no attribute '__index__'

Just to add, when I re-run this exact pipeline, it seems to be the same dataset. I will try to use the remote debugger a little bit when i have time. Will have to binary search...


how to do enrichment analysis like Gene ontology(GO)

Hi, Dear developer
I found plant-ontology(PO) is very useful for me, But I am wondering whether I can do enrichment analysis like GO. Can you recommend some pipline or case-study about PO enrichment analysis?
Best wishes
Guandong Shang

You can do an ontology enrichment query of the Planteome Database for a list of genes here: http://planteome.org/oat/
How well this works will depend upon the species selected, as the data has to be in the database.
Please click on the "manual" button for more details on using the tool
You can also do a filtered search for annotations at Planteome.org
The enrichment tool and visualizations are currently under development, so please feel free to provide feedback at: http://planteome.org/contact

Thanks for your reply :)


Quantum spin inverter and mail tube are VERY bugged

Description
If you send the quantum spin inverter through a mailing unit and teleport to it before the quantum spin inverter has reached the end of the pipe, it COMPLETELY DELETES you.
Reproduction
1: get two quantum spin inverters
2: send one down a mailing unit and flush
3: swap places with the spin inverter BEFORE it reaches the other end
4: watch as you arrive to the other end of the mail unit and your entity gets COMPLETELY deleted
Additional context
did this on round #43894, got deleted, ahelped that my entity got deleted
this seems like an issue with mail units not being able to fit living entities but instead of just force ejecting them it just deletes them
related to #24095 possibly, but this one stands out as it's own bug since it's probably something people will try and inevitably be round removed over

from disc

Content and transform system name a more iconic duo.

This is just like old disposals all over again, dispose yourself off-grid and you're sent to the shadow realm.

Fixed by #24772

bug of the year 2024 squashed


Implement a "From" in Mailservice

Even when MTS is "only" a sample application, it may be used as a running devonfw example application that generates real mails.
A minor enhancement were, if the "From" of those sent mails could be configured. This may need introducing of some new property and using it via setFrom inside
java/mtsj/core/src/main/java/com/devonfw/application/mtsj/mailservice/logic/impl/MailImpl.java

This is related to an older commit where a formerly implemented setFrom was removed, because it has derived the value from property spring.mail.username, which itself was removed.
But the spring.mail.username is for authentication at SMTP server, this may be the same value as an email address of the sender - but often enough is different to this. So setFrom should be reintroduced based on a new property.
Unfortunately there is not "from" property handling inside springboot autoconfigure Mailproperties, so I guess such a property would need to be defined inside namescope of mts not spring.


Comment replies don't include MoreChildren

The issue is in the Things.Comment object:
        [JsonProperty("replies")]
        public CommentContainer Replies { get; set; }

As you can see, the Replies property is defined as a CommentContainer.  Unfortunately, I've just realized that this property can also be a "more"-type listing, which I hadn't observed before because apparently it only exists in deeply-threaded comment replies, so the library doesn't even have a Thing for that right now.  I'll need to add one.
Here's an example of what we're looking at:
[
   {
      kind: "Listing",
      data: {
         modhash: "fdroq5szue77292449297c2f1006fa25f44bcc75be8c22592a",
         dist: 1,
         children: [...],
         after: null,
         before: null
      }
   },
   {
      kind: "Listing",
      data: {
         modhash: "fdroq5szue77292449297c2f1006fa25f44bcc75be8c22592a",
         dist: null,
         children: [
            {
               kind: "t1",
               data: {
                  total_awards_received: 0,
                  approved_at_utc: null,
                  ups: 978,
                  awarders: [ ],
                  mod_reason_by: null,
                  banned_by: null,
                  removal_reason: null,
                  link_id: "t3_2lt3d0",
                  author_flair_template_id: null,
                  likes: null,
                  no_follow: false,
                  replies: {
                     kind: "Listing",
                     data: {
                     modhash: "fdroq5szue77292449297c2f1006fa25f44bcc75be8c22592a",
                     dist: null,
                     children: [
                        {
                           kind: "t1",
                           data: {
                              total_awards_received: 0,
                              approved_at_utc: null,
                              ups: 441,
                              awarders: [ ],
                              mod_reason_by: null,
                              banned_by: null,
                              removal_reason: null,
                              link_id: "t3_2lt3d0",
                              author_flair_template_id: null,
                              likes: null,
                              no_follow: false,
                              replies: {
                                 kind: "Listing",
                                 data: {
                                    modhash: "fdroq5szue77292449297c2f1006fa25f44bcc75be8c22592a",
                                    dist: null,
                                    children: [
                                       {
                                          kind: "t1",
                                          data: {
                                          total_awards_received: 0,
                                          approved_at_utc: null,
                                          ups: 48,
                                          awarders: [ ],
                                          mod_reason_by: null,
                                          banned_by: null,
                                          author_flair_type: "text",
                                          removal_reason: null,
                                          link_id: "t3_2lt3d0",
                                          author_flair_template_id: null,
                                          likes: null,
                                          replies: {
                                             kind: "Listing",
                                             data: {
                                             modhash: "fdroq5szue77292449297c2f1006fa25f44bcc75be8c22592a",
                                             dist: null,
                                             children: [
                                             {
                                                kind: "more",
                                                data: {
                                                   count: 3,
                                                   name: "t1_clyelyg",
                                                   id: "clyelyg",
                                                   parent_id: "t1_clyaevo",
                                                   depth: 3,
                                                   children: [
                                                      "clyelyg",
                                                      "clyiysi"
                                                      ]
                                                   }
                                                }
                                                ],
                                                after: null,
                                                before: null
                                             }
                                          },

URL:  https://www.reddit.com/r/news/comments/2lt3d0/top_01s_wealth_about_to_overtake_bottom_90_for/.json?limit=500&sort=confidence
As you can see, the replies are all comment containers until a certain depth is reached, then they display the MoreChildren data, instead.  I'll need to do some refactoring to accommodate this.

Related: #73

I'm currently blocked on this by a possible API bug:
https://www.reddit.com/r/redditdev/comments/dqz2jy/api_bug_the_apiinfo_endpoint_does_not_populate/?

Fixed and merged into 1.3.


Latest image (1.40.5) missing unzip

Using this dockerfile:
FROM denoland/deno
WORKDIR /app
ADD . /app
RUN deno compile --allow-net server.ts

FROM gcr.io/distroless/cc-debian12:nonroot
COPY --from=0 /app/server /home/nonroot/server
ENTRYPOINT ["./server"]

Give me this error:
1.970 Check file:///app/server.ts
3.351 Compile file:///app/server.ts to server
3.351 Download https://dl.deno.land/release/v1.40.5/denort-x86_64-unknown-linux-gnu.zip
15.84 error: Writing server
15.84
15.84 Caused by:
15.84     `unzip` was not found in your PATH, please install `unzip`
------
Dockerfile:4
--------------------
   2 |     WORKDIR /app
   3 |     ADD . /app
   4 | >>> RUN deno compile --allow-net server.ts
   5 |
   6 |     FROM gcr.io/distroless/cc-debian12:nonroot
--------------------
ERROR: failed to solve: process "/bin/sh -c deno compile --allow-net server.ts" did not complete successfully: exit code: 1

It works with tag 1.40.4

Having the same issue with deno:1.41.0.

Still failing with denoland/deno:1.41.3

Without the unzip the compile does not work.
Is it a valid MR to be done?

Since deno use unzip, I would think it should be in the image.  On the other hand, it is not welcome at runtime, so I don't know, but in my case I dit this:
FROM denoland/deno
WORKDIR /app
ADD . /app
RUN : \
 && apt-get update \
 && apt-get install -y unzip \
 && deno compile --allow-net server.ts

FROM gcr.io/distroless/cc-debian12:nonroot
COPY --from=0 /app/server /home/nonroot/server
ENTRYPOINT ["./server"]

Maybe I could just use the denoland/deno:distroless variant tho


[AIRFLOW-2431] Add the navigation bar color parameter for RBAC UI

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2431
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a JIRA issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
Add navbar_color feature to RBAC UI

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
manual test

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

@bolkedebruin  @r39132

Codecov Report

Merging #3326 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3326   +/-   ##
=======================================
  Coverage   75.89%   75.89%
=======================================
  Files         197      197
  Lines       14729    14729
=======================================
  Hits        11178    11178
  Misses       3551     3551

Impacted Files
Coverage Δ

airflow/www_rbac/app.py
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Oops, this is a good point @Eronarn. Could maybe @Eronarn or @jgao54 create a PR to fix this? Thanks!


Remove deprecated code to add node name into log pattern of log4j property file

Description
[Describe what this change achieves]
Issues Resolved
https://github.com/opensearch-project/OpenSearch/issues/4274
Check List

[ ] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #4568 (239a2a5) into main (7ebb2af) will increase coverage by 0.04%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main    #4568      +/-   ##
============================================
+ Coverage     70.68%   70.73%   +0.04%
- Complexity    57379    57402      +23
============================================
  Files          4628     4628
  Lines        276073   276049      -24
  Branches      40421    40414       -7
============================================
+ Hits         195146   195258     +112
+ Misses        64562    64374     -188
- Partials      16365    16417      +52

Impacted Files
Coverage Δ

...org/opensearch/common/logging/LogConfigurator.java
24.63% <0.00%> (+6.35%)
:arrow_up:

...a/org/opensearch/client/cluster/ProxyModeInfo.java
0.00% <0.00%> (-60.00%)
:arrow_down:

.../java/org/opensearch/node/NodeClosedException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

.../java/org/opensearch/search/dfs/AggregatedDfs.java
51.61% <0.00%> (-45.17%)
:arrow_down:

...ava/org/opensearch/search/dfs/DfsSearchResult.java
43.26% <0.00%> (-35.58%)
:arrow_down:

.../opensearch/client/indices/CloseIndexResponse.java
56.25% <0.00%> (-35.00%)
:arrow_down:

...va/org/opensearch/node/AdaptiveSelectionStats.java
11.11% <0.00%> (-33.34%)
:arrow_down:

...min/cluster/snapshots/get/GetSnapshotsRequest.java
52.63% <0.00%> (-31.58%)
:arrow_down:

...rc/main/java/org/opensearch/ingest/IngestInfo.java
51.72% <0.00%> (-27.59%)
:arrow_down:

.../main/java/org/opensearch/search/dfs/DfsPhase.java
38.23% <0.00%> (-26.48%)
:arrow_down:

... and 459 more

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