New command: teams channel member add

Usage
teams channel member add
Description
Adds user to specified Microsoft Teams private channel.
Options

Option
Description

--teamId [teamId]
The Id of the Microsoft Team. Specify either teamId or teamName but not both.

--teamName [teamName]
The name of the Microsoft Team. Specify either teamId or teamName but not both.

--channelId [channelId]
The Id of the Microsoft Team channel. Specify either channelId or channelName but not both.

--channelName [channelName]
The name of the Microsoft Team channel. Specify either channelId or channelName but not both.

--userName [userName]
User's UPN (user principal name, eg. johndoe@example.com). Specify either userName or userId but not both.

--userId [userId]
User's Azure AD Id. Specify either userName or userId but not both.

-r, --role [role]
The role to be assigned to the new user: owner. When empty, user is added as member.

Extra

Duplicate command teams conversationmember add should be removed
Set teams conversationmember add as alias for teams channel member add

Additional Info
Docs: Add member to channel
Discussed in #3168

Maintainers could you have a look at the proposed command?

Hi @milanholemans,
Thank you for your submission. Looks good to me.
I will assign you the issue.
Thank you

To align the command with our conventions:

let's shorten userName and userId to name and id, because they refer to the last noun in the command's name and we don't need to repeat the context
let's remove the -i and -n shorts because they're ambiguous as to which name and ID they refer to
looking at the referenced docs, it seems like owners is the only allowed option. Let's double check our spec, and also specify allowed values uses correct casing (lowercase in this case)

@waldekmastykarz I have adjusted the specs with your feedback.
For the role property, when you don't specify a role in your Graph request, the user will be added as member (or guest if it's an external account). If you specify owner as role in your Graph request, the user will be added as owner. I think it's best to provide something, maybe a boolean --owner instead of a choice field? Anyway, I found it a bit confusing that members have no role while Guests and Owners do. What do you think?
Example team with 1 owner, 1 member and 1 guest:

Hi,
Maybe an isOwner flag might do the trick.
Cheers

Thinking of it some more, I wonder if it's not a part of intentional design to have a role option with right now just one value, but in the future potentially more values. I wonder if it weren't more future-proof to align with the API and have a --role option that allows just one value named owner at the moment, but which could allow more roles in the future if need be.

This is definitely part of the design because you can already pass 3 different values. We do not know which additional options will be available in future

This is definitely part of the design because you can already pass 3 different values.

I think the confusing part is that the property can have 3 values, but the request accepts only one. I think it does make sense to align with the API to accommodate for future changes that we don't know about today.

So you think we should provide a role option that accepts only 1 value owner ?

So you think we should provide a role option that accepts only 1 value owner ?

yes, let's do that

@waldekmastykarz can I start working on this command? I added an extra section to the issue to remove the existing command teams conversationmember add.

@milanholemans all yours. Thx

Question, currently this issue has 2 user options (--userName and --userId). When I check the ms docs I notice that I can call Graph with both user ID and user UPN (it's the same request). Is it an idea to merge these 2 optional options into one required option -u, --user <user>?
Also should I include the options of the command teams conversationmember add (command which will be removed) and give a deprecation message for these options?

Hi @milanholemans,
by verifying again I see that the command conversationmember add already implements all desired features with specs that we should keep. If we do not do this and follow the specs presented here, we are going to break the running commands. The command is in place for 13 months.
You find details here: https://github.com/pnp/cli-microsoft365/blob/main/docs/docs/cmd/teams/conversationmember/conversationmember-add.md
I would suggest the following approach:

keep the conversationmember add as it is
repurpose this issue to a rename of conversationmember add to member add
rename the current conversationmember add to member add and add an alias for conversationmember add

@waldekmastykarz, @milanholemans: any input?
Thank you,
Patrick

They only thing I'd add is to include a deprecation message for the alias as we'll remove it in the next major version.

Hi @milanholemans,
do you have any questions? Do you want to proceed with the activities?
Thank you,
Patrick

All right thanks, I'll just rename the command.


Extending block pollute each others output

Sorry for the weird title, but this is a tricky one and couldn't really find a good name for this issue.
See this requirebin for more context.
I'm not sure I'm not doing something totally wrong here, so let me know if I miss something and I'll try to fill a PR if it's totally valid scenario.

When you want to include a partial that itself extends a layout, you can't use the {{> partial}} syntax. You use {{#embed 'partial'}}{{/embed}} or {{{embed 'partial'}}} instead. If you change the container template in the <body> of your demo to the following, it works as expected:
<script type="text/template" id="container">
{{#extend 'otherBase'}}
  {{#content 'otherBaseContent'}}
    {{{embed 'firstDescendant'}}}
    {{{embed 'secondDescendant'}}}
  {{/content}}
{{/extend}}
</script>

https://github.com/shannonmoeller/handlebars-layouts#embed-partial-context-keyvalue-

Ah my bad, I knew about embed, but always use it instead of extend, not when calling directly the partial.
/me being dumb, closing this

No worries. Not dumb. Maybe this thread will help someone else too.


Graphics overhaul tracking

This ticket began as a simple "re-export the sprites in higher resolution" idea, and has evolved over nearly 9 months to become quite complicated. Here's a reasonably concise list of what we actually want to do, based on changes from Factorio 0.17 so far (as of 0.17.29) and plans revealed in recent FFF posts:

[x] Redesign buildings to better fit new HD style used by Wube (especially chests, per FFF 283) [#88]

[x] Add robot pick-up/drop-off animations to the logistic warehouses/storehouses (also FFF 283) [#88]

Factorio 0.17.5 added a LogisticContainerPrototype::landing_location_offset property to control where robots dock, upon our request, which means our new buildings won't be constrained to putting the robot docking hatch in the center

[x] Overhaul icons to match [#88]

[x] High resolution item icons

[x] At least bump to 64x64 per FFF 277 [#88]
[x] Multiple icon resolutions with simplified designs at smaller sizes [#88]

Factorio itself will likely use a mix of two approaches, based on FFF 290 and FFF 291
We should simply use whichever gives us the best results for our icons

[x] Technology icons to match base game techs as of Factorio 1.1: [#88]

"Base game technology icons now use 4 mipmaps and icon size 256 instead of icon size 128 and no mipmaps"
Moved here from #82, as despite being posted in the "breaking changes" forum thread it doesn't break us

[x] Make it easier to tell which entity icon is which at-a-glance (per mod portal feedback indicating that the current icon sizes are easily confused) [#88]

Especially important if we decide to integrate "Small" and "Tiny" warehouses as proposed by @JDOGG88 in #68—there would then be at least six different icons to distinguish

[ ] Apply look-up tables (LUTs) to sprites/icons? (FFF 320)
[x] Make thumbnail.png for the mod portal/GUI [#88]

Technically already done in dbf6e7c, but the current demo map/screenshot will need to be tweaked/regenerated after the rest of this list is finished

Original issue description from mid-2018 follows.

It would be nice to launch high-resolution sprites along with, or shortly after, Factorio 0.17's release. Warehousing will need to be updated anyway, and that update might as well come with a nice bonus. (Wube will no doubt upgrade a number of base-game entities to HD, as they have in past major updates, so Warehousing would really just be joining the party. :smile_cat:)
@Nova-Kast I'm assigning this issue to you because you're the only one who can actually generate new images. (If you want to do #32 properly at the same time, that would be fine too. :grin:)

FFF 269 today announced Factorio 0.17 is planned to release in January 2019. That's a good target for getting this done.

GitHub, what do you mean I unassigned Nova…? Must have had something to do with repo permissions and/or moving the repo into an organization. GitHub messes with stuff when you do that. :slightly_frowning_face:
Well anyway… FFF 283 announced a couple graphics-related things that we probably want to act on:

The in-game mod GUI will show a thumbnail image for each mod, and the way it's implemented requires including a thumbnail.png file (144x144 pixels) in the mod package.
Chests were upgraded to high-resolution. The most relevant change is that logistic chests now have a roboport-like animation—something it would be nice for Warehousing's buildings to have, too.

I'll see about picking a thumbnail.png to include in future releases—probably just the one we have now, at first. Whatever we use, an updated thumbnail should come with the graphics refresh, whenever it actually happens (so the thumbnail will match the new entity sprites).
Adding animations to the warehouse/storehouse buildings will probably be its own project, separate from upgrading the sprites to high-res. That is, unless @Nova-Kast is feeling really ambitious and wants to do everything at once. :grin:

Today's FFF 290 revealed that Wube are working toward better icon scaling. It's possible Factorio will gain support for multiple icon sizes to be used at different zoom levels, and we should keep this on the radar until they come to a final decision on how they want to handle it.
I'll update the OP with a simple list of things we're keeping tabs on, because this issue has gotten quite complicated with all the FFF-based follow-ups.

Any idea on a potential timeline for the new graphics or is the plan to wait until Wube makes their final decision?  I'm just anxious to see what the new graphics are going to look like. 😂

Any idea on a potential timeline for the new graphics

When @Nova-Kast is ready and able to work on them, they'll get done. No timeline at present.

Tweaked icon-size list entry (and children) to include FFF 291.

Added LUTs to the list (see FFF 320).

So I was thinking about making a "plugin" for small and tiny warehousing just until the new graphics are done and/or depending on the popularity of it.  It would just be a separate mod that makes Warehousing dependent and adds in the small and tiny versions.  It may make it easier to decide whether or not to add them into the main mod based on popularity.  I'd like your feedback before I create it and upload it to the mod portal.

@JDOGG88 An add-on mod is fine with me. If you'd be creating and distributing derivative works based on this mod's graphics, though, you should ask @Nova-Kast first. (Quickest way to do that is through @Anoyomouse; PM on the Factorio forums if neither of them replies here in a day or two.)

Nah it wouldn't be anything derivative using the graphics.  It would just be pulling my Small & Tiny code out and use it to make a "Warehousing Compression" mod with the ultimate goal of seeing how people like it and whether or not to actually add it into the main mod.

Might as well have at it, @JDOGG88. Have fun!

If you have the time and want to take a look https://github.com/JDOGG88/Warehouse-Compression

Hey @Nova-Kast, are you still working on these new graphics?

Linking to PR #88, snouz has done some amazing work redoing all the sprites as well as tweaking them for in game

Once #88 is done I'll try to figure out if LUTs are still relevant, or if this whole thing can be closed out.

Once #88 is done I'll try to figure out if LUTs are still relevant, or if this whole thing can be closed out.

Here's Wube LUT script
https://github.com/wube/factorio-LUT-tool
I have no idea if it's relevant. It says the script "applies colors" to an image, but there aren't several images for each entity, so maybe that's just something that is done in-engine?

I don't think the LUT script is relevant. In fact, I think that whole checklist item can just… not happen. We don't need it, since you specifically offered to create new graphics because you could make them match Factorio's current look.
So, in anticipation of the next release with those new graphics that check all the boxes above, I'll just close this out and focus on testing/releasing when I get the time!


Should we remove the timeout from ActorProxy by default

Describe the proposal
ActorProxy uses a default HttpClient which comes with a 100 second timeout. This might be surprising, because users can add their own cancellation policies, but can't be more permissive than what the built-in timeout supports.
We're adding other mechanisms to give users control over this, but I wanted to raise the topic specifically - should we remove the default timeout? Have you been impacted by this?

This may be silly question but... is there any way to determine if an Actor with a specific Id is Idle or Busy?
It might be nice to have a way to know the current state of an Actor before doing the call and hitting this timeout. Right?


Utterance/transcribe problem - audio becomes utterance

Hello,
Premise: I am not very experienced with neither dialogflow nor drachtio. Dratchio-server and fs are setup with docker (drachtio-freeswitch-mrf and drachtio-server). I have created a very simple dialoglow (es) .
I start a call from a sip client ( twinkle) calling the exposed docker udp port 5063 ("5063:5060/udp"), the welcome intent is detected and initial response is delivered. but , as the response in playing back, it appears as if the playback become the input for the detection of the utterances.
I believe this is happening because the behavior of the dialog is erratic and the dialogflow log seems to confirm my idea (
 )
I am surely missing or misconfiguring something. Can you point me on the right direction ?
BTW. Excellent work

are you saying the transcriptions you are receiving from dialogflow are of the audio file that is being played?  Then I would say you have an echo problem in your setup

Thanks for pointing me out in the right direction. It turns out that I had altered some fs configuration.
Excellent.


Update merge-transact-sql.md

invalid example using merge into an edge-table. The example code is not working in SQL Server 2017.
A valid example is required, but I have no idea which code could work. At least a warning should be until a valid code example will be provided.
the "ON MATCH (Person-(livesIn)->City" causes the error when used in the merge statement.

@aisbergde : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@aisbergde Thank you for this suggestion. Let me look into it more and I'll get back to you as soon as possible. #reassign:rothja

@aisbergde Apologies for the delay. I verified that this does not work in 2017 but works in 2019. I altered your commit to put it in the form of a short note before the sample. Thank you for pointing this out and sorry for the confusion!


Add documentation to modin/pandas/base.py

What do these changes do?

[x] commit message follows format outlined here
[x] passes flake8 modin
[x] passes black --check modin
[x] signed commit with git commit -s
[x] Resolves #2942
[x] tests added and passing

We should also enable numpydoc check for modin/pandas/base.py

Ok I enabled it but it fails. We need to merge fixed doc_checker to make it work.

LGTM! Just revert CI.yml changes.

We should be able to enable numpydoc against base.py after #3001 lands (or in it itself).

@devin-petersohn ready for review.

I've taken my fixes for _inherit_docstrings in a separate issue #3033 and fixing PR #3034, we should rebase this PR removing my commits when #3034 lands.


Asma_review.md

Rename docs.md to Asma_review.md

Thank you, Asmaa. Can you add it inside docs folder?

Thank you, Asmaa. Can you add it inside docs folder?

Thank you @BatoolMM, I have moved the file inside docs folder.

Awesome! Thank you, Asma!


Remove mocap packages (REP 144 Fix)

Packages removed to meet with REP 144 (https://github.com/ros/rosdistro/pull/39475#issuecomment-1894443133), replaced by https://github.com/ros/rosdistro/pull/39615 and https://github.com/ros/rosdistro/pull/39616

Hii @fmrico, there is no value in updating Foxy as it is EOL, can you please update the PR to have changes to Humble only?

Hi @quarkytale, I have just restored Foxy file


Create robots.txt

Added robots.txt file advised by Daniel and Ann regarding some suspicious downlaods.


EVALSYS-1449 change hibernate DTD path to www.hibernate.org

from hibernate.sourceforge.net

@jonespm this looks correct, right?

Yep, and it resolves to a good DTD at http://hibernate.org/dtd/hibernate-mapping-3.0.dtd but the published still includes the www.
+1


feat: Add checksums for files

This PR add the CRC32C and MD5 hash to the File skel.
Resolves: #1176

@phorward i used md5 and  crc32c because they are the only two that are provided by google storage. otherwise we would have to download the whole blob and hash it ourselves.


Automation Testing

Problem description
Describe the current state of affairs and why it needs to change.
Requirements
Exact requirement for the product - what needs to be done?
Definition of done
How do we know we've fully dealt with it?
Acceptance criteria
What are the criteria by which we determine if the issue has been resolved?

Closing as a duplicate of #785 and #786


[Logo To Use] -> [Logos]

logotouse.com
The internet’s source of free logos. Get over million free logo to use.
Link: wwwl.ogotouse.com

@ezaz-ahmed Hey, Thanks for the PR, can you please create a new PR with the resource without formatting all the file?
Thanks!


Unable to build a working npk with modify_npk

I tried some scenarios all of which failed when I tried to flash into the router... Firmwares chosen were 6.39.3 and 6.40.8

I extracted a working routeros npk and modified the rootfs filesystem , packed it with mksquashfs and then built npk using the modify_npk.
In this I didn't make any changes in the rootfs filesystem., Everything else was same.
In this I didn't even extract the root-squashfs ... Just built the npk file from the rootfs.squashfs

And for building the npk , I have followed exactly what is stated in the repo.

Hey! Thanks for submitting an issue. I need more details though.

Specifically, which RouterOS are you installing over?
How are you upgrading?
What is the error message and can you produce a screenshot?
What are the exact instructions you using to create the modified npk? From start to finish.

I can't help if I can't recreate the issue.

modify_npk -n wrasse -s /home/new/Downloads/_routeros-mipsbe-6.39.3.npk.extracted/1000.squashfs -f /home/new/Downloads/routeros-mipsbe-6.39.3.npk
wherein this case the 1000.squashfs is the unchanged squashfs file from binwalk extract
To upgrade I transfer the wrasse.npk to the "Files" of mikrotik router and do a restart
￼
I am performing the upgrade on a router whose current version is 6.39
Following is the error that I get there in the log after the upgrade fails


[Raycast Wallpaper] Suggestions (Loading Time and Wallpaper exclusions)

Extension
https://www.raycast.com/koinzhang/raycast-wallpaper
Description
I have two suggestions:

When I run Set Raycast Wallpaper, it takes a few seconds to load all the pictures. It's really annoying to have to wait for a preview.

→ Maybe you could work with lighter pictures instead of the large 6K pictures (it seems you are taking the official wallpaper pictures).

I really love the Auto Switch Raycast Wallpaper, but I would really appreciate having the ability to exclude a wallpaper from the auto switch list.

→ You could let users set pictures as an exclusion and create something like a sub-list.
Who will benefit from this feature?
No response
Anything else?
No response

Hi @NicolasVaillant.
For problem one, the current extension has been optimized. When you enter the extension, the cached pictures will be loaded first (the first time you enter the extension, you need to download the cache, and it will take a long time), so that you can quickly load the pictures
For question two, I will consider submitting a PR to update

Hi @NicolasVaillant.
It is recommended to use the Grid layout. The problem of image loading delay does exist in the List layout.
Also I submitted a PR #8007 to add a new action for Set Raycast Wallpaper: Exclude From Auto Switch
@raycastbot close this issue


typo.

Types of changes

[ ] New feature
[ ] Bug fix
[ ] Documentation change
[ ] Coding style change
[ ] Refactoring
[ ] Performance improvement
[ ] Test cases
[ ] Continuous integration
[ ] Typescript definition change
[ ] Breaking change

Background and context

Solution

How is the change tested?

Changelog

Changelog(CN)
Changelog(EN)
Related issues

Checklist:

[ ] Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
[ ] Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others should be submitted to master branch)

Other information

@zhangenming hi 感谢pr。麻烦遵守一下commit规范哈。3q


Can not generate triangles

OS: Debian 10
Python 3.7.3
python3 example.py
min -0.84543, -0.84543, -0.84543
max 0.845431, 0.845431, 0.845431
step 0.0104847, 0.0104847, 0.0104847
4492125 samples in 64 batches with 8 workers
0 triangles in 0.741009 seconds

Try printing any exceptions that occur here: https://github.com/fogleman/sdf/blob/main/sdf/mesh.py#L33

Try printing any exceptions that occur here: https://github.com/fogleman/sdf/blob/main/sdf/mesh.py#L33

I found the issue. The version of my scikit-image package is too low. It's working now after I upgraded it.
This project is very inspiring  to me. Thanks!

I found the issue. The version of my scikit-image package is too low. It's working now after I upgraded it.
This project is very inspiring  to me. Thanks!


[WIP] Inference on layer

WIP PR for #7
Features :

Working option to run inference on a single layer, with all functionalities
Improvements to inference code (worker and widget)

MISSING :

Proper button behaviour
Orientation of the output can be wrong sometimes, will have to determine best default case
Testing all features
Proper error handling/reset when running a job without a folder/layer selected

Codecov Report

Merging #26 (083ebbf) into main (eb33f31) will decrease coverage by 1.11%.
The diff coverage is 13.82%.

@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
- Coverage   37.80%   36.68%   -1.12%
==========================================
  Files          37       37
  Lines        3383     3538     +155
==========================================
+ Hits         1279     1298      +19
- Misses       2104     2240     +136

Impacted Files
Coverage Δ

napari_cellseg3d/model_instance_seg.py
25.00% <0.00%> (-1.16%)
:arrow_down:

napari_cellseg3d/models/model_SegResNet.py
45.45% <0.00%> (-4.55%)
:arrow_down:

napari_cellseg3d/models/model_VNet.py
42.85% <0.00%> (-4.21%)
:arrow_down:

napari_cellseg3d/plugin_helper.py
0.00% <ø> (ø)

napari_cellseg3d/plugin_model_inference.py
0.00% <0.00%> (ø)

napari_cellseg3d/plugin_model_training.py
63.36% <0.00%> (-0.86%)
:arrow_down:

napari_cellseg3d/utils.py
32.77% <0.00%> (+1.19%)
:arrow_up:

napari_cellseg3d/model_workers.py
15.76% <11.26%> (+0.49%)
:arrow_up:

napari_cellseg3d/log_utility.py
29.78% <20.58%> (-23.55%)
:arrow_down:

napari_cellseg3d/models/model_TRAILMAP.py
27.08% <21.95%> (-26.77%)
:arrow_down:

... and 4 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


libwebkit2gtk-4.0 not available in Ubuntu 24.04

libwebkit2gtk-4.1 is default on Ubuntu 24.04
🔋 100% ❯ sudo apt search libwebkit2gtk

libwebkit2gtk-4.0-doc/noble-updates,noble-security 2.44.2-0ubuntu<IP_ADDRESS> all
  WebKitGTK documentation (transitional dummy package)

libwebkit2gtk-4.1-0/noble-updates,noble-security,now 2.44.2-0ubuntu<IP_ADDRESS> amd64 [installed]
  Web content engine library for GTK

libwebkit2gtk-4.1-dev/noble-updates,noble-security 2.44.2-0ubuntu<IP_ADDRESS> amd64
  Web content engine library for GTK - development files

got an error when i build
🔋 100% ❯ dotnet run
Unable to load shared library 'Photino.Native' or one of its dependencies. In order to help diagnose loading problems, consider using a tool like strace. If you're using glibc, consider setting the LD_DEBUG environment variable:
libwebkit2gtk-4.0.so.37: cannot open shared object file: No such file or directory

I need to manually install libwebkit2gtk-4.0 to run

Yeah it seems like recent Linux OS don't ship with 4.0 anymore. Fedora also has switched to webkit2gtk-4.1

Manjaro - same here.

That's correct for development purposes. You have to install the 4.0 version and package the correct one in your package.
https://github.com/tryphotino/photino.Samples/tree/master/Photino.PublishPhotino/PublishPhotino

@philippjbauer This should definitely be documented somewhere then, because otherwise people will just be confused why not even the demo projects work out of the box.


(BKR-313) Beaker is using pe-httpd as the puppet master service...

... in method 'with_puppet_running_on'

use pe-httpd pre 3.4, pe-puppetserver post 3.4

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/924/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/925/
Test FAILed.


Roll Flutter Engine from c50eb8a65097 to 0462d1937908 (1 revision)

https://github.com/flutter/engine/compare/c50eb8a65097...0462d1937908
2024-09-05<EMAIL_ADDRESS>Fix unexpected ViewFocus events when Text Editing utilities change focus in the middle of a blur call. (flutter/engine#54965)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC codefu@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


feat: add secret level transitions and player data in progress data

Description

Related Issue(s)

Changes Made

[x] Player States and position are now in progress data manager
[x] Secret level transitions

Screenshots (if applicable)

Checklist

[x] I have reviewed my own code changes.
[x] I have tested these changes locally.
[x] My code follows the project's coding style guidelines.
[x] I have updated the documentation accordingly.
[x] I have added appropriate labels to this pull request.

0e2c3307-3d07-4ec6-87b9-24215777fb10.webm
cuando mario es grande no puede acceder al nivel secreto


Introduce Redux for better state management b/w components

As of now the state management is component oriented. Create a store using react-redux or make use of react-hooks (like useReducer)
Links : https://reactjs.org/docs/hooks-reference.html#usereducer, https://react-redux.js.org/introduction/quick-start

you guys, might need to setup functionality b/w components to run it on localhost and the chrome extension can be obtained from build

Hey @Devilla I'm getting webpack plugin error while running build script on master:
throw new RemovedPluginError(errorMessage);
^
Error: webpack.optimize.UglifyJsPlugin has been removed, please use config.optimization.minimize instead.
at Object.get [as UglifyJsPlugin]
Which is the currently stable branch where I can run the app?

@Devilla There are many problems with this app. I'm getting "Hook was not found" error while running npm start:

Error: Plugin could not be registered at 'html-webpack-plugin-before-html-processing'. Hook was not found.

and while building, I get another error:

Error: webpack.optimize.UglifyJsPlugin has been removed, please use config.optimization.minimize instead.
Unfortunately many libraries are deprecated. fixing all these issues is a separate project in itself.


chore(main): release 1.4.0

:robot: I have created a release beep boop
1.4.0 (2024-11-15)
Features

add modifierContent to <FormField /> (#16) (5ca0fd0)
change bg color (#17) (195cb00)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.4.0
:sunflower:


Add a download button to PRE blocks that point to files

We have PRE blocks that can pull in external files directly. There is a Copy button for these PRE blocks, and it'd be nice to also have a Download button available.

This is no longer a supported feature of the site.


Sort protein families button does not work

When you click on an individual genome name and then the button 'Sort Protein Families' does not work.

fixed. thanks for reporting.

Checked. working.


Build Angular version 17 for @trimble-oss/modus-angular-components

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the Modus style guidelines

Proposal
Build Angular version 17 for @trimble-oss/modus-angular-components
Motivation and context
For Angular project v17+
What is the issue regarding ?
@trimble-oss/modus-angular-components
Priority
High
What product/project are you using Modus Components for ?
Prism
What is your team/division name ?
Prism
Are you willing to contribute ?
Yes
Are you using Modus Web Components in production ?
No

Hey there @quynh-le-trimble - it'd be great if you wanted to take this on! Let me know if you have any questions about it.

@coliff Sure, is there any documentation available where i can get these changes updated and published?

Hello! I added the 'angular-16' and these were the equivalent steps I took to do that.

Made of a copy of the modus-web-components\angular-workspace\ng15 folder and name it ng16 (in your case it'd be copying the ng16 folder and then name of the new folder ng17).
In the package.json in that folder, update engines (node) to >=18.13.0
Read through the upgrade guide: https://update.angular.io/?v=16.0-17.0

Thanks, @coliff! I pushed a PR with the changes you requested. Please let me know if any issues come up.


Failed to connect to Qdrant: [WinError 10061]

I get the following error when creating the embeddings:
Failed to connect to Qdrant: [WinError 10061] Cannot connect because the target computer actively refused the connection.
I added a row to my AVG firewall to allow Qdrant port 6333, however I still get this error.
How to solve this?
Thanks.

Deploy docker and docker compose on your machine local.
Then create compose-file.yaml (ref: https://qdrant.tech/documentation/guides/installation/)
services:
qdrant:
image: qdrant/qdrant:latest
restart: always
container_name: qdrant
ports:
- 6333:6333
- 6334:6334
expose:
- 6333
- 6334
- 6335
configs:
- source: qdrant_config
target: /qdrant/config/production.yaml
volumes:
- ./qdrant_data:/qdrant/storage
configs:
qdrant_config:
content: |
log_level: INFO
Then run : docker-compose up (ensure your file is in the same dir where you executed the command)
This will run qadrant and you can access it from the browser http://localhost:6333/dashboard
Then once you select embedding create from the app this will work fine.

Or directly start a container like this
docker run -p 6333:6333 -p 6334:6334 \
    -v $(pwd)/qdrant_storage:/qdrant/storage:z \
    qdrant/qdrant


Clarification about the UserName, Password, and HostName binding properties

It would be helpful to make some clarifications about the UserName, Password, and HostName binding properties in the following doc:
https://github.com/Azure/azure-functions-rabbitmq-extension/
For example, the UserName and Password properties need to reference app settings instead of directly referencing values or using %%. Eg: UserName = "UserNameAppSetting" is the correct format, whereas UserName = "%UserNameAppSetting%" or UserName = "MyValidUserName" will cause the Functions host to hit the following error at startup:
Microsoft.Azure.WebJobs.Host: Unable to resolve the value for property 'RabbitMQAttribute.UserName'. Make sure the setting exists and has a valid value.
Since HostName needs to be a direct value or use %% to reference an app setting, it would be particularly helpful to make the distinction about how UserName and Password work to avoid confusion.
Assuming these three properties are only intended for Output bindings, it would be good to call that out in the doc as well. Although I see a HostName property for Triggers, I don't see the UserName or Password properties for triggers. I do see the UserNameSetting and PasswordSetting properties for Triggers, but neither these properties appear to be read when using a Trigger.
TLDR:
I have the following, whichworks as expected in my Output binding:
binding properties:
` HostName = "%HostNameAppSetting%", UserName = "UserNameAppSetting", Password = "PasswordAppSetting",
local.settings.json:
    "HostNameAppSetting": "MyValidHostName",
    "UserNameAppSetting": "MyValidUserName",
    "PasswordAppSetting": "MyValidPassword"

However, the PasswordSetting or UserNameSetting bindings properties (which are present in place of Password and UserName) don't appear to bet honored in my Trigger. Eg:
HostName = "%HostNameAppSetting%", UserNameSetting = "UserNameAppSetting", PasswordSetting ="PasswordAppSetting",
Am using extension version 1.0.0-beta

Update about using HostName, UserNameSetting, and PasswordSetting in a Trigger binding:
These will work if you do either of the following:
a) Hard-code the values. Eg:
HostName = "MyValidHostName", UserNameSetting = "MyValidUserName", PasswordSetting ="MyValidPassword"
b) Use %% around the app setting name references. Eg:
HostName = "%HostNameAppSetting%", UserNameSetting = "%UserNameAppSetting%", PasswordSetting ="%PasswordAppSetting%"

Additional note: I notice that ConnectionStringSetting takes precedence over HostName, etc, so might be one more thing to call out in the doc.

docs are updated with further info about the binding properties


feat(docker): install dependencies from requirements.txt

there are two sources for pip now, one is the requirements.txt and another is the dockerfile, their contents are same and we should use one coherent dependence file in a project.

supported
https://github.com/mtobeiyf/keras-flask-deploy-webapp/blob/11fe80aa166145ea2784f781a07871681ed53287/Dockerfile#L3-L4


Fix warning hip hcc

fixes some of the tensile warnings in rocBLAS on hipclang and hcc

Thanks I'll merge once the checks complete.  Note there is still another hip-clang warning but there are multiple ways to fix it so leave it to your team, and not sure a typedef is even needed for an inline local.
--
In file included from .../rocBLAS/build/release/Tensile/Kernels.h:32:
.../rocBLAS/build/release/Tensile/KernelHeader.h:33:16: warning: anonymous non-C-compatible type given name for linkage purposes by typedef declaration; add a tag name here [-Wnon-c-typedef-for-linkage]
typedef union { int32_t i; char4 z; } PkInt8x4;
^
PkInt8x4
.../rocBLAS/build/release/Tensile/KernelHeader.h:33:11: note: type is not C-compatible due to this member declaration
typedef union { int32_t i; char4 z; } PkInt8x4;
^~~~~
.../rocBLAS/build/release/Tensile/KernelHeader.h:33:41: note: type is given name 'PkInt8x4' for linkage purposes by this typedef declaration
typedef union { int32_t i; char4 z; } PkInt8x4;
^


Move docker recipe to documentation along with instructions.

Fixes #99
I removed docker yaml file for kafka and moved the information into the proper documentation under getting-started section.
It also has explanation why it is not included in the hard-dependencies.

Here is the locally built documentation!
html.zip

Looks good to me.
Makes sense to exclude Kafka from the dependencies and put the setup instructions in the documentation instead if Kafka is not the only supported message delivery framework.
Are the Kafka tests run in the CI pipeline? I'm thinking that it is probably good to run them somewhere on a regular basis.
How was the 6GB limit selected?
Do we exceed that limit if we run the benchmark for a longer time period?

Are the Kafka tests run in the CI pipeline? I'm thinking that it is probably good to run them somewhere on a regular basis.

Not yet, but yes we do need an automated tests. There's an issue regarding this. #97

How was the 6GB limit selected?
Do we exceed that limit if we run the benchmark for a longer time period?

This number was given by ECDC so I think they have tested it within their testing environment.


Move task_info to _viash

          I wouldn't do this now, but I think we should try to move as many of these fields (if not all) to `_viash.yaml`.

Originally posted by @rcannood in https://github.com/openproblems-bio/task_template/pull/2#discussion_r1639547257

Closed in #5


added extra fields in upsert_record breaks _convert_record

upsert_record on DynamoTable breaks when using DynamoDictClass record.  In the upsert_record code it converts the passed in record to a dict and then adds extra fields which it passes to _convert_record.  The default implementation attempts to pass the dict into the record_class constructor.  It doesn't look like dataclasses are designed to handle this method of initialization or maybe there is a bug as I get an error about missing required positional arguments for all of the required fields in the record.
Below is a test (pytest) that exhibits the behavior.
 I prepared a patch and going to release it ASAP.

Please do not use dataclasses in dynamo records. Just remove this decorator and the code should work now.


Create function for simulating whole trajectories

Simulate solution of master equation by applying uniformization incrementally, e.g., starting at time point 0 w/ initial conditions, solve for t=0.01. That solution is now the initial state for getting the solution at t=0.02, e.g. This avoids problems related to stiffness: When the (time * uniformization rate) gets large, uniformization becomes inefficient due to the many matrix multiplications that have to be performed.
It would be interesting to consider adaptive time stepping or adaptation of the uniformization rate, but that might be too much initially.

Should take advantage of the fact that p(t + dt) = exp((t + dt) * Q) p(t)

Sketch:
function solve(Q, p0, tspan, dt, params)
soln = zeros(maximum(tspan) / dt + 1, size(Q, 1))

if minimum(tspan) != 0
soln[1,:] = uniformize(Q, p0, params.k, minimum(tspan))
else
soln[1,:] = p0
end

for i in 2:size(soln, 1)
soln[i, :] = uniformize(Q, soln[i-1,:], params.k, dt)
end
soln
end

Will probably need to redo interface of uniformization() to take a params NamedTuple (#14) first to make passing parameters smoother.


after dragging on the pan core graph, the cursor stays on hand-closed

Original issue by @bmesuere on Fri Jul 17 2015 at 17:01.
Closed by @bmesuere on Wed Jul 29 2015 at 13:57.

This is a bug in Chrome and already fixed in Canary
Original comment by @bmesuere on Wed Jul 29 2015 at 13:57.


Implement Array.from and Array.of

Closes #1183, alternative to #1235.
This allows constructs like Array.from({ length: 3 }, (_, index) => index * 2) (which works in js).
Array.of simply transforms into a table expression.

much better than my PR ><
thank you!


Storable and Data.Vector.Unboxed.Unbox instances for Quantities

This PR adds Storable and Unbox instances for Quantity so they can be used efficiently in arrays. The vector instances are enabled with a flag to avoid paying for the extra dependency if not used (although I would rather have them enabled by default since vector is a pretty standard package)

Great, thanks! I will review this in more detail after work and merge it.
@bjornbm How do you feel about the issue of taking the vector dependency vs having the package flag?

A few comments:

Should there be version bounds on the vector dependency? I notice there is a requirement of version 0.11 for a certain feature. I think our whole library doesn't work at all on versions older than GHC 7.8, so perhaps we should pick a contemporaneous version of vector as a lower bound? I'm not sure what version that would be, though.
Is it possible to put all the INLINE pragmas in a consistent location? Some are given below the declaration to which they apply, some above, and some grouped at the start of the instance. I suspect we need a lot more similar pragmas in other places for performance reasons (see #87 and the comments there), so it will be good to be consistent. I'm not sure what is the common style choice on this? But if I had to guess I would copy Edward (e.g. at https://github.com/ekmett/intervals/blob/master/src/Numeric/Interval/Internal.hs) and put them below the declarations?
Welcome to the project! Have you been using this much? We'd appreciate any other feedback you have on spots for improvement, or on the two major things we are trying to decide before we release this to hackage, namely #72 and whether Units should carry information about their names.

Hi

I've added a lower bound on 0.10 which is a guess based on what stack's lts-2 has (this LTS version requires GHC 7.8.4)
Done. Regarding the need for more pragmas, I believe the Quantity newtypes are being erased since I've profiled an app which used them and they don't appear at all on the heap profiles, just the unboxed doubles which they wrap. Anyway, a bench suite wouldn't hurt to make sure.
Thanks! I'm using it primarily on behave-hs for automatic unit conversion at the interface of the API but not internally since it is a port of a C lib based on the Rothermel fire spread model which has many constants based on imperial units. Apart from this, I haven't used it much (yet) so I don't think I can contribute meaningfully to those discussions (at the moment)

Cool, thanks. Very interesting work on the fire modelling!
I am going to merge this, and then change it to directly take the vector dependency on the grounds that text and vector are practically as widespread as base anyway, since we already require GHC. Also on the grounds that I couldn't figure out from google how to get travis to do the builds with and without the flag set. If Bjorn doesn't think it's a good idea then we will change it back to the flag setup.
Since taking the vector dependency also brings in a dependency on deepseq I will also take the opportunity to provide an NFData instance, which should also be useful for performance testing.

Hmmm. I started making the changes I just talked about in my branch, and ran up against a build error (https://travis-ci.org/dmcclean/dimensional-dk/jobs/85417777) in the 7.8.4 build (with the vector features enabled). I think it is an AMP thing. Working on it.

Just had to refer to fmap by its other name liftM to keep 7.8.4 happy. See 7f93d0a5feda48ee0054abd1c04c7462817639b0.

Thanks for the merge!
Sorry for the fmap problem, I'm no longer using 7.8.4 and didn't spot it. If going back to the flag setup it could be tested with travis by adding another entry to the matrix in travis.yml, something like:
matrix:
 include:
   - env: CABALVER=1.18 GHCVER=7.8.4 ARGS="-fvector"
   ...
   - env: CABALVER=1.18 GHCVER=7.8.4 ARGS=""
   ...

and modifying the cabal install and cabal configure commands to use it, eg: cabal install $ARGS. Let me know if it happens and I'll set up.

Cool, thanks. We try to keep the 7.8.4 build working since it is the most recent version of the compiler that I have managed to get working on Raspberry Pi, where I have a couple of projects. But I use 7.10 on my development computer so I wouldn't have noticed either until travis complained; it's a pretty great tool for catching things like that.


Can we use a variable inside a macro/equivalence class ?

Research the equivalence class transformer.
Questions:

Is it good to keep them in one class or break them into several classes ?

@shraddha-patel will be taking a look at this.
Design thoughts:

Want to allow use of existing variable values as inputs to macros in the EquivalenceClassTransformer
Need to be able to distinguish between literal values and variable names somehow (maybe $ ? )

This should not be baked in to the EquivalenceClassTransformer - should be its own entity useable with any transformer.
Also related to use of EL in general (longer-term).


The current "About" section closely resembles the homepage and lacks a distinct identity

Proposal
I will completely redesign the "About" section to differentiate it from the homepage. This will include:
-Unique layout and content structure
-Enhanced visuals and design elements that better represent the section's purpose
-A more informative and engaging narrative that tells the story or mission of the brand
Expected Behavior
The new "About" section will have a unique, visually appealing design that is distinct from the homepage, providing a clear and compelling introduction to the brand, its mission, and values.
Screenshots & Screen-recording

https://github.com/user-attachments/assets/c8dacae0-9615-4044-be55-6d9a63ee4ed0

issue #278


Codespace/Remote-Container Support

This PR adds codespace and remote-container support.

@johanste is created a .sh file for linux, we'll update to that.


confidentialledger - cspell found spelling errors

Spell check scanning of confidentialledger detected spelling errors. This service directory is opted out of PR spell checking in PR #17142 to keep merges unblocked.
What to do

Ensure Node.js is installed (https://nodejs.org/en/download/). This will include npx
Delete the service's entry in .vscode/cspell.json's ignorePaths field. It will look like: sdk/confidentialledger/**/*. You need to do this to enable checking the files.
From the root of the repo run spell check using npx cspell lint --config '.vscode/cspell.json' --no-must-find-files "sdk/confidentialledger/**"
Fix detections according to http://aka.ms/azsdk/engsys/spellcheck use the "False positives" section to fix false positives
Check in changes (including the change to .vscode/cspell.json where the ignorePaths is updated to remove the entry for this service.). You may need to run git add -f .vscode/cspell.json to force adding the changes to the file in git.

Spell checking output
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/karma.conf.js:77:18 - Unknown word (lcovonly)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/karma.conf.js:77:49 - Unknown word (lcov)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/package.json:64:67 - Unknown word (prettierrc)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/package.json:68:52 - Unknown word (prettierrc)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/README.md:28:19 - Unknown word (Condifential)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/README.md:88:81 - Unknown word (Merkle)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/README.md:88:158 - Unknown word (Merkle)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/review/confidential-ledger.api.md:118:5 - Unknown word (mrenclave)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/review/confidential-ledger.api.md:404:18 - Unknown word (Merkle)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/review/confidential-ledger.api.md:471:12 - Unknown word (Merkle)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/samples/v1/javascript/README.md:17:28 - Unknown word (getenclavequotesaad)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/samples/v1/javascript/README.md:60:2 - Unknown word (getenclavequotesaad)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/samples/v1/typescript/README.md:17:28 - Unknown word (getenclavequotesaad)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/samples/v1/typescript/README.md:72:2 - Unknown word (getenclavequotesaad)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/src/generated/src/models.ts:46:7 - Unknown word (MRENCLAVE)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/src/generated/src/models.ts:47:3 - Unknown word (mrenclave)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/src/generated/src/models.ts:50:60 - Unknown word (oeverify)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/src/generated/src/models.ts:94:7 - Unknown word (Merkle)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/src/generated/src/models.ts:98:7 - Unknown word (Merkle)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/src/generated/src/models.ts:99:10 - Unknown word (Merkle)
/home/djurek/azure-sdk-for-js/sdk/confidentialledger/confidential-ledger-rest/src/generated/src/models.ts:100:19 - Unknown word (Merkle)

Closing this as we will re-open issues per-package and according to feedback in https://github.com/Azure/azure-sdk-for-js/pull/17142


Connect to the Fediverse

Can you support ActivityPub, so the platform can support the Fediverse, including Mastrodon and Threads, however as a read-only instance where people can comment on their old posts and view them on modern apps, including Threads. I hope Meta connects Facebook and Instagram to the Fediverse along with Threads, TikTok connects TikTok posts and comments, and Google follows suit with YouTube community posts, Google Play and Maps reviews, and soon after a CEO rearrangement Reddit and Twitter get connected to the Fediverse, but we need baby steps first. I hope we get the ActivityPub protocol as ubiquitous as EMail with basically every social platform and social feature using it.

Guide: https://socialhub.activitypub.rocks/t/guide-for-new-activitypub-implementers/479
This is only intended as a way for Fediverse users to view old Miiverse posts and accounts, not the ability to view other instances and post any content.

Dup of #42 ?


Fix misaligned buildserver stat reports

Change-Id: I5a158bef21dc31133c1510ed4c10fcc6eae941cc

This PR fixes an issue with the SimpleStatReporter where the logic of the nextStep command was actually recording the performance of the previous step under the name of the next step. Doing so required the reporter to maintain slightly more state since it has to remember the name of the in-progress step between calls.


ceiling_dirs parameter in Repository.discover is optional

libgit2's git_repository_discover function has an optional ceiling_dirs parameter. We should flag it as such in our JSON file.

Do I need to do anything else to get this merged in?

just gotta bring it to my attention, again :smile:.

@implausible Thanks for reviewing and merging!


Add java-side unit tests to fastVM

Description
Please include a brief summary of the change that this pull request proposes. Include any relevant motivation and context. List any dependencies required for this change.

Adds java unit tests to the major classes related to the fastVM. Some refactorings for testing were done and some for cleanups. The cleanups were only done in small classes like ExecutionContext/Helper/Result and not the larger ones, where with lack of existing unit tests I didn't feel confident enough to alter much.
Currently this PR uses branch test_fastvm in the fastvm until that branch is merged into precompiled in this PR.

Fixes Issue # N/A.
Type of change
Insert x into the following checkboxes to confirm (eg. [x]):

[ ] Bug fix.
[ ] New feature.
[ ] Enhancement.
[x] Unit test.
[ ] Breaking change (a fix or feature that causes existing functionality to not work as expected).
[ ] Requires documentation update.

Testing
Please describe the tests you used to validate this pull request. Provide any relevant details for test configurations as well as any instructions to reproduce these results.

The tests I added!

Verification
Insert x into the following checkboxes to confirm (eg. [x]):

[x] I have self-reviewed my own code and conformed to the style guidelines of this project.
[x] New and existing tests pass locally with my changes.
[x] I have added tests for my fix or feature.
[ ] I have made appropriate changes to the corresponding documentation.
[x] My code generates no new warnings.
[x] Any dependent changes have been made.

The reason I added the line this.output = (output == null) ? ByteUtil.EMPTY_BYTE_ARRAY : output; to AbstractExecutionResult is to fix a small bug where, if output is null, once you encode the object using toBytes and then decode using parse, output always gets set to an empty byte array and is never set null, so you don't get the same object back truly. Other code depends on this empty byte usage so instead I just made sure the constructor gets rid of the null right away

Also: TransactionResult was not a unit test priority for me atm, the vm is mostly interested in ExecutionResult. The classes are similar and TransactionResult is mostly simple 1-line methods, I will add testing for this class as part of my next PR


RoutePolicy: Add routing rules for linklocal address

This commit adds route policy rules for link local IP address on both interfaces
to fix same subnet link local address configuration on both interfaces.
Tested By:
verify both interfaces link local addresses reachable.

jenkins run tests please

jenkins run tests please


Update hncconfig webhook to handle object overwriting and refactor forest sourceObjects field

Update forest field sourceObjects to store all the names of user-created
objects by GVK as long as there's an object reconciler for that GVK.
Change object reonciler behaviors dependent on the previous forest field
originalObjects, which only stores source objects for 'Propagate' types.
Add test cases for corner cases.
Add new hncconfig webhook rules and new test cases. Check if changing
type mode(s) to Propagate would cause overwriting source by looking
up the objectNames data. Deny the request and output all the conflicts
if it would cause overwriting.
Tested by "make test" and manually on a GKE cluster with multiple
objects with the same name in the same tree or different trees. Passed
all e2e tests too.
The webhook message would look like this:
Cannot update configuration because setting type "/v1, Kind=Secret" to 'Propagate' mode would overwrite user-created object(s):
  Object "my-creds2" in namespace "acme-org" would overwrite the one in "team-a"
  Object "my-creds2" in namespace "acme-org" would overwrite the one in "team-b"
  Object "my-creds" in namespace "bcme-org" would overwrite the one in "team-c"
  Object "my-creds2" in namespace "bcme-org" would overwrite the one in "team-c"
To fix this, please rename or remove the conflicting objects first.

Part of #1102, #1076

/assign @adrianludwin
/assign @rjbez17
/cc @GinnyJI

/retest

Update: Passed all e2e tests.
Initially I got:
Summarizing 3 Failures:

1.[Fail] Demo [It] Should test basic functionalities in demo
/usr/local/google/home/yiqigao/go/src/sigs.k8s.io/multi-tenancy/incubator/hnc/pkg/testutils/testutils.go:110

2. [Fail] HNC should delete and create a new Rolebinding instead of updating it [It] Should delete and create a Rolebinding when HNC is undeployed - issue #798
/usr/local/google/home/yiqigao/go/src/sigs.k8s.io/multi-tenancy/incubator/hnc/pkg/testutils/testutils.go:110

3. [Fail] When deleting CRDs [It] should not delete subnamespaces
/usr/local/google/home/yiqigao/go/src/sigs.k8s.io/multi-tenancy/incubator/hnc/pkg/testutils/testutils.go:69

Ran 29 of 29 Specs in 1030.354 seconds
FAIL! -- 26 Passed | 3 Failed | 0 Pending | 0 Skipped

1 & 2 passed when I ran it manually. 3 passed by itself if it's a stand-alone test (with Fit).
The 1st demo failure also failed on master. I suspect there's a bug in the test that it should use RoleBinding instead of Role here: https://github.com/kubernetes-sigs/multi-tenancy/blob/f990be3264cac6edb7618880b859522d25d40989/incubator/hnc/test/e2e/demo_test.go#L55
/assign @adrianludwin
/cc @GinnyJI

Yes, @GinnyJI is looking into the e2e test failures on master. I'm comfortable that this change is good and doesn't break anything.

/lgtm
/hold cancel


FlatPak

Since this is specifically for the Deck, and the Deck only officially supports FlatPak installs, I think it would be a good idea to support them.

That's a very good idea. I am going to look into that eventually.
Does anybody know if flatpak supports running as a background service/daemon, with low-level access to USB devices and ability to host a UDP server?

Based on my limited knowlege this is all supported.

Steam Link is distributed by flatpak and it passes the control of Steam Deck controls entirely to the host, so I think USB access has to be also possible somehow.

You may have already figured this out but you can use --device=all in your yaml or json file when building the flatpak to do this  (see the flatpak docs for more info). This passes the entire /dev directory to the flatpak application. The permissions are those of the user, so whatever the user can access in the /dev directory on their system, the flatpak app should be able to access, including usb. You have to use all for now as there isn't a specific usb option as you can see from the flatpak issue here
Additional Note:
Steam Deck has a udev rule for usb/hidraw access by the deck user (given the user is physically logged in). You can see it when logged into a Steam Deck with:
cat /lib/udev/rules.d/70-steam-jupiter-input.rules

as the following:
# USB devices
SUBSYSTEM=="usb", TAG+="uaccess"

# HID devices over hidraw
KERNEL=="hidraw*", TAG+="uaccess"

# Steam Controller udev write access

However, it looks like you aren't reading via the usb or hidraw interfaces and are instead reading the /dev/usb/hiddevX file where X is the device corresponding to the Steam Deck. In this case, you need elevated permissions which it seems like you set up with a group that you add the Steam Deck user to in your install script. As long as this is done, you should be able to use --device=all for the flatpak and usb direct access should work.
You can see example manifests on flathub using --device=all such as the manifests for the citra and yuzu emulators.

@streetpea thank you for the write-up.
I've read up on udev a little and it looks like the rule:
SUBSYSTEM=="usb", TAG+="uaccess"
should allow access to all usb/hiddev* also but it doesn't. Any idea why that might be? I'm not even sure if those actually allow access to hidraw, I will crash that.
The rule applied to group usbaccess by sdgyrodsu install script is actually more restrictive (as it is only for a specific device, not entire usb subsystem) and it works that way.
Maybe there's some ACL configuration that allows only steam process to access those or something?

@kmicki it probably has to do with another rule interfering then. Anyway, you can tell it's not getting applied properly if you look at the extended permissions (acls) with getfacl  (note: the extended attributes denoted by a + in ls -l which are added by the +=uaccess udev rule)... for usb you get
getfacl /dev/usb/*

USB Output:
getfacl: Removing leading '/' from absolute path names
# file: dev/usb/hiddev0
# owner: root
# group: root
user::rw-
group::---
other::---

For hidraw devices you get
getfacl /dev/hidraw*

Hidraw Output:
getfacl: Removing leading '/' from absolute path names
# file: dev/hidraw0
# owner: root
# group: root
user::rw-
user:deck:rw-
group::---
mask::rw-
other::---

# file: dev/hidraw1
# owner: root
# group: root
user::rw-
user:deck:rw-
group::---
mask::rw-
other::---

# file: dev/hidraw2
# owner: root
# group: root
user::rw-
user:deck:rw-
group::---
mask::rw-
other::---

# file: dev/hidraw3
# owner: root
# group: root
user::rw-
user:deck:rw-
group::---
mask::rw-
other::---

# file: dev/hidraw4
# owner: root
# group: root
user::rw-
user:deck:rw-
group::---
mask::rw-
other::---

As you can see, the deck user has rw to hidraw devices but only the root user (file owner) has rw to the usb device so it looks like the +=access for the usb devices isn't going through (probably because there is another rule taking precedence)

After more analysis I think access (ACL) just doesn't work the same as rules for users/groups.
AFAIR I tried to access the device using hidapi but it failed because it was already used by Steam (hence getting data via hiddev*).
I noticed that Steam Link is able to get data from the device simultaneously with Steam so it has to be possible somehow using files that are accessible to deck user by default. It would be great if root agree was not necessary for installation.

I am able to get access using hidapi with hidraw interface without requiring root to install extra udev rules: https://github.com/streetpea/chiaki4deck/blob/cfc982a622b592c07eea4a27aacb97a00bc71642/steamdeck_native/src/sdeck.c#L153 If you have any questions lmk.

@streetpea Thank you. It was very helpful. I will switch to using hidapi-hidraw in the next release. This will bring it closer to creating a Flatpak.

@kmicki Awesome, if you haven't already made progress on a flatpak build I can test one out for you if you want.

I don't know if we can help you or advice you on this.
We have talked a bit to the Vita3K team about building flatpaks as they don't have one either.
But if you want you can join our Discord and we can add you to the 🚧-open-development channel if  you are open for discussion.
That goes for the both of you :) @streetpea @kmicki
//RetroDECK Team


Edit logic diagrams

Added implementation for FindTasksBeforeCommand and edit variable names

Codecov Report

Merging #186 (dffc66d) into master (afb758e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #186   +/-   ##
=========================================
  Coverage     73.97%   73.97%
  Complexity      657      657
=========================================
  Files           110      110
  Lines          2017     2017
  Branches        225      225
=========================================
  Hits           1492     1492
  Misses          447      447
  Partials         78       78

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 afb758e...dffc66d. Read the comment docs.


Made changes to reflect latest product behavior

@sideeksh : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Udemy - Unable to extract course id

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.02.11. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.02.11

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)

I'm not able to download udemy video, error is: Unable to extract course id. I login with Udemy for Business - Team account.
I tried these URL with same error:
https://www.udemy.com/learn-ansible/
https://www.udemy.com/learn-ansible/learn/v4/t/lecture/7133354
If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-u', 'PRIVATE', '-p', 'PRIVATE', 'https://www.udemy.com/learn-ansible/learn/v4/t/lecture/7133354', '--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.02.11
[debug] Python version 3.6.4 (CPython) - Linux-4.14.18-300.fc27.x86_64-x86_64-with-fedora-27-Twenty_Seven
[debug] exe versions: none
[debug] Proxy map: {}
[udemy] Downloading login popup
[udemy] Logging in
[udemy] 7133354: Downloading webpage
ERROR: Unable to extract course id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/home/shot/.local/lib/python3.6/site-packages/youtube_dl/YoutubeDL.py", line 785, in extract_info
    ie_result = ie.extract(url)
  File "/home/shot/.local/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 440, in extract
    ie_result = self._real_extract(url)
  File "/home/shot/.local/lib/python3.6/site-packages/youtube_dl/extractor/udemy.py", line 187, in _real_extract
    course_id, _ = self._extract_course_info(webpage, lecture_id)
  File "/home/shot/.local/lib/python3.6/site-packages/youtube_dl/extractor/udemy.py", line 69, in _extract_course_info
    r'data-course-id=["\'](\d+)', webpage, 'course id')
  File "/home/shot/.local/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 796, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract course id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

https://www.udemy.com/learn-ansible/
https://www.udemy.com/learn-ansible/learn/v4/t/lecture/7133354

Description of your issue, suggested solution and other information
Probably udemy changed website, so udemy extractor can't find course id.

Sorry, now everything is working fine. I have to login to www.udemy.com first with my Udemy for business account. After that, downloading started working. Thanks for a great work.


[Tip]Retrieve Screenshots from appDetails()

Hello,
The default method dosn't retrieve screenshots of a game.
To get the images follow the steps :

Open App.php file

"../vendor/syntax/steam-api/src/Syntax/SteamApi/Containers/App.php"

Replace the content with the code on pastebin :

https://pastebin.com/JM5myhx2
What did i change ?
I have added two lines.
The class variable :
public $screenshots;
and this line on constructor :
$this->screenshots  = $this->checkIssetCollection($app, 'screenshots', new Collection());
Almost the same was done with Mac- & Linuxrequirements.
Happy coding 👍

Thanks for the tip! :D

Thanks for the tip! :D


antigen wrong name in README.md

Taken from README.md
antigen theme https://github.com/denysdovhan/spaceship-zsh-theme spaceship-zsh-theme
The spaceship-zsh-theme should match the filename of the theme, which is spaceship(.zsh-theme).
I had to change the command to the below to get it works.
antigen theme https://github.com/denysdovhan/spaceship-zsh-theme spaceship

Yep, absolutely. I've made a mistake.
Do you wan't to fix that, or I'll do it?

You can do it, I just pointed it out.

@Sinderella done.


Gradio All-in-One: Preprocess, Train, TensorBoard, and Interface

Hi, @KdaiP
I’m not sure where to post this, so I’ll share it here.
After a lot of testing, I’m working on a quick Gradio all-in-one solution . It includes preprocessing, training, TensorBoard , and the interface.
I’m still working on it and haven’t finished yet. I need to fix some issues, but I hope you’ll like it once it’s ready. I’ll send it for testing as soon as it’s done. Since I’m new to GitHub, i dont know how share code for your repo  ?
Here’s a quick preview of what it looks like so far. Please enable the voice feature, as it’s currently muted, to experience the interface.
Let me know what you think!
https://github.com/user-attachments/assets/fd3cf287-4952-4e6f-906b-d84616dbc272

Hi, This sounds really cool! Once you've finished your modifications, you can create a pull request to share the code.
Also, it seems like the issue of audio cutouts has improved in this new model.
Maybe in the future, we can add components for data annotation and data cleaning (which I'm currently organizing), similar to the one-click packages for TTS and GPT-SoVITS, to create an out-of-the-box workflow.
Looking forward to seeing your progress!

Thank you all!
I've finished a new version, and I hope you like it.
I plan to upload it tomorrow and make a pull request as you suggested.
I have a quick question:
Can I create a fork, add the files, and then submit a pull request?
I'm still new to this process, so I just want to make sure.
In this version, I added:
The ability to start, stop, and resume training, with a save and config option so you don't lose progress.
The ability to start and stop TensorBoard.
A random sample selector for training data to make it easier to use, as well as a reference to compare results.
A seed option, allowing you to use a random seed or fix it if needed.
Automatic model downloads on the first run if the models aren't found locally.
I have a question about the sample in the progress tab:
Do we need to resample the files?
Let me know what you think of the new version. I ran some tests, and it's working great! You can easily fine-tune voices, and everything is well-organized. I didn't touch the core code, so there shouldn't be any conflicts. I just created a separate Python file and imported code from your repo, or copied some parts. I also added comments, so it's easy to understand and follow what I did.
https://github.com/user-attachments/assets/16800e16-3e95-42b2-a6c3-51aa31fd0c95

Hi, thank you for your hard work on this! Your implementation is both elegant and well-organized!
In addition, resample is not required because the training target of StableTTS (mel spectrogram) is already extracted from resampled audio.


Now query on workflow stub that was was used for start follows runIds

What was changed
Previously if workflow stub was created and used to start a workflow, query on this stub was targeting the specific runId that was created by this stub and didn't follow the chain of runs.
Now the workflow stub follows the chain of runIds until it is explicitly created for a specific runId.
This PR also cleans up the messy internals that provided potentially outdated runIds in logs of operations that were not targeting a specific runId.
Closes #1601

query on this stub was targeting the specific runId that was created by this stub and didn't follow the chain of runs.

In (some) other SDKs, by default the result from start will not set run ID on query so it'll auto-choose the latest for the workflow ID. But if they did create a handle/stub with an explicit run ID, then we use that. I assume this works similarly? Does it actually "follow runs" instead of just leaving run ID off?

@cretz

I assume this works similarly?

Exactly. After the change it will still focus on a specific runId if it was created with specific runId. But it will follow runIds if it wasn't.
Before if the stub is created without a runId and after that was used for a start, query suddenly targets only that first runId, which is highly counter-intuitive.


is there an interface to uninstall the BCUninstaller itself for update?

I'm using bcuninstaller 5.5 portable, and recently it remind me to update, I download the newer vision but can find uninstall option in the old one? so how can I uninstall it? just delete the root folder?

thanks for your work!

This is a portable version that can also run on a USB stick, for example, and does not need to be uninstalled. It is sufficient to delete the folder.

thank you


Drop a Forti

Details
Content to detect auth bypass on Fortinet appliances related to CVE-2022-40684
New Analytics

Fortinet Appliance Auth bypass

New Analytic Story
CVE-2022-40684 Fortinet Appliance Auth bypass
What does this PR have in it? Screenshots are worth 1000 words 😄
Checklist

[ ] Validate name matches <platform>_<mitre att&ck technique>_<short description> nomenclature
[ ] CI/CD jobs passed ✔
[ ] Validated SPL logic.
[ ] Validated tags, description, and how to implement.
[ ] Verified references match analytic.

Hi @MHaggis
On detections/web/fortinet_appliance_auth_bypass.yml
search: '| tstats count min(_time) as firstTime max(_time)
as lastTime from datamodel=Web where Web.url IN ("/api/v2/cmdb/system/admin/admin")
This search should only be ("/api/v2/cmdb/system/admin")
see log:

Hi @MHaggis
On detections/web/fortinet_appliance_auth_bypass.yml
search: '| tstats count min(time) as firstTime max(time) as lastTime from datamodel=Web where Web.url IN ("/api/v2/cmdb/system/admin/admin")
This search should only be ("/api/v2/cmdb/system/admin")
see log:

Thank you for the feedback!


next js support ?

m getting this error in next js  document is not defined

Hello, because of server-side rendering enabled by default with Next.js , you don't have access to the document object when the component is first rendered on the server. That is why you get this error : document is not defined
You can use Next's dynamic import : https://nextjs.org/docs/advanced-features/dynamic-import#with-no-ssr
code example :
import dynamic from 'next/dynamic'

const DynamicComponentWithNoSSR = dynamic(
  () => import('scrollable-component'),
  { ssr: false }
)

While disabling server-side rendering is a valid option, I don't necessarily recommend it.
For a better user experience, I would prefer to import the scrollable-component script directly on the client side, because that's how it's supposed to be used.
Here is the documentation on how to import a third-party script globally for your application.
This way it won't have to be lazy loaded and therefore will make the <scrollable-component> and its content directly accessible (better for SEO and accessibility).
To be able to import the scrollable-component script, you will have to use a custom Webpack Config and a Webpack Plugin to copy the file.

First install the plugin copy-webpack-plugin:
npm install --save-dev copy-webpack-plugin
Then, create a Webpack Config (next.config.js) similar to this one:
const CopyPlugin = require('copy-webpack-plugin');

const nextConfig = {
  webpack: (config) => {
    config.plugins.push(new CopyPlugin({
      patterns: [
        {
          from: './node_modules/scrollable-component/index.js',
          to: './static/scrollable-component.js',
        },
      ],
    }))

    return config
  },
}

module.exports = nextConfig

In your pages/_app.js, do something like this:
import Script from 'next/script'

export default function MyApp({ Component, pageProps }) {
  return (
    <>


Update README.md

Linux uses Control rather than the Command key, which is usually referred to as the Super-Key.

Thx Nathan!


Endpoint environment variables aren't in the docs

@jdef Can give some context, but I couldn't find docs for how pod endpoint environment variables are set:
ENDPOINT_{NAME} = ....
EP_CONTAINER_{NAME} = ...
EP_HOST_{NAME} = ...

should get docs on releases/1.4 and master.

Three vars:
ENDPOINT_{name}: when in host networking mode, this is the endpoint's
hostPort; in container mode this is the endpoint's containerPort
.. this should always be present, because hostPort is required for host
networking mode, and containerPort is required for container networking
mode.
EP_HOST_{name}: reflects the value of the endpoint's hostPort (if defined)
EP_CONTAINER_{name}: reflects the value of the endpoint's containerPort
(if defined)
.. the presence of the above two variables depends on the declaration of
the endpoint named {name}
On Wed, Feb 22, 2017 at 6:56 PM, Jason Gilanfarr<EMAIL_ADDRESS>wrote:

should get docs on releases/1.4 and master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/marathon/issues/5240#issuecomment-281845045,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACPVLEXwV2LW_ROV6Xji7nwqEmSEnCmPks5rfMtJgaJpZM4MJW6a
.

We also always capitalize the name.
On Thu, Feb 23, 2017 at 10:30 AM James DeFelice<EMAIL_ADDRESS>wrote:

Three vars:
ENDPOINT_{name}: when in host networking mode, this is the endpoint's
hostPort; in container mode this is the endpoint's containerPort
.. this should always be present, because hostPort is required for host
networking mode, and containerPort is required for container networking
mode.
EP_HOST_{name}: reflects the value of the endpoint's hostPort (if
defined)
EP_CONTAINER_{name}: reflects the value of the endpoint's containerPort
(if defined)
.. the presence of the above two variables depends on the declaration of
the endpoint named {name}
On Wed, Feb 22, 2017 at 6:56 PM, Jason Gilanfarr<EMAIL_ADDRESS>
wrote:

should get docs on releases/1.4 and master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/mesosphere/marathon/issues/5240#issuecomment-281845045
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ACPVLEXwV2LW_ROV6Xji7nwqEmSEnCmPks5rfMtJgaJpZM4MJW6a
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/marathon/issues/5240#issuecomment-282078382,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAweZrTZgd_SxFP1It_NNZ-Hy2kpTYchks5rfdAngaJpZM4MJW6a
.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2441. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Quant Server gives so much errors

Please fix i:
Object reference not set to an instance of an object

Bruh bro just initialize Quant Server 💀


[WIP] Toolbar button instance variable check

Continuation of  #10865
Describe the rationale and use case for this pull request.  Provide any background, examples, and images that provide further information to accurately describe what it is that you are adding to the repo.  Add subsections as necessary to organize and feel free to link and reference other PRs as necessary, but also include them in the links section below as a quick reference.
Guidelines:

Keep Pull Request titles short and to the point, ideally under 72 characters
Provide as much context/info in the description as necessary to get the reviewer up to the same domain knowledge level as yourself
Keep code changes as short as possible and implementing a single feature/fix/refactoring, when possible

Links [Optional]

http://documentation.for/library/that/I/am/adding
PR relevant issue or pull_request

Steps for Testing/QA [Optional]
If there are any manual steps that you would like the reviewer(s) to take to verify your changes, please describe in detail the steps to reproduce the features added by the pull request, or the bug before and after the change.

@miq-bot add_label wip, technical debt, ui

@miq-bot assign @PanSpagetka

@miq-bot remove_label wip

looks good

Euwe Backport details:
$ git log
commit 3cc64c40fd98beb1c056dd386f38e87cedcdbca4
Author: Martin Povolny<EMAIL_ADDRESS>Date:   Tue Sep 27 19:58:52 2016 +0200

    Merge pull request #11357 from romanblanco/toolbar_button_instance_variable_check

    Toolbar button instance variable check
    (cherry picked from commit df87e30fb5154820eae087e1bb0ecbe115040493)


networking library - Topolgy diagrams

Dear team,
We already bought 4 license for our Networking engineers, we need the following:

Library of network Icons (router, switch, firewall, server, VPN Tunnel)
The ability to atache links that can move with the form. For example if a create 2 router and connect to eachother if any of both is moved that the link still atached

Please your comments
regards
Sergio Prado
excalidraw account<EMAIL_ADDRESS>

Gave it a try myself: https://github.com/excalidraw/excalidraw-libraries/pull/29

It looks great, is that already a library available for networking?
regards
Sergio


CVE-2023-2976 reported for the component "com.google.guava:guava" in "build-info-extractor-maven3-2.40.0-uber.jar" file

Describe the bug
the CVE-2023-2976 reported for the component "com.google.guava:guava" used in the "build-info-extractor-maven3-2.40.0-uber.jar" file which needs to be fixed.
Current behavior
the CVE-2023-2976 reported for the component "com.google.guava:guava" used in the "build-info-extractor-maven3-2.40.0-uber.jar" file which needs to be fixed.
Reproduction steps

Download the package build-info-extractor-maven3-2.40.0-uber.jar provided by JFrog and scan it with the help of Xray.

Xray will report the CVE-2023-2976 for the component "com.google.guava:guava" used in the "build-info-extractor-maven3-2.40.0-uber.jar" file which needs to be fixed as shown below:

Expected behavior
Build-info-extractor-maven3-2.40.0-uber.jar should not be vulnerable. If the above CVE gets fixed, there will be no violations.
JFrog CLI version
2.46.2
Operating system type and version
Rhel 8
JFrog Artifactory version
latest
JFrog Xray version
latest

Thanks for reporting this issue @palaksahu1.
We created https://github.com/jfrog/build-info-go/pull/192 to include https://github.com/jfrog/build-info/pull/754 that should update Guava to 32.0.1-jre.

@palaksahu1
JFrog CLI 2.46.3 has been released.
This version includes the upgrade of the Guava dependency.
We'd appreciate your feedback on that.

Thank you @yahavi for the fix.


false positive 'selector-nest-combinators'

Given a value 'always' and a stylesheet:
.class-name {
	#{if(&, "&", "")} {

	}
}

The linter will report the error:
'Expected combinator "}" of type "tag" to be in a nested form'

Thanks for this project!

Thanks for reporting the issue :)
I already have fix for the issue, so I'll try to release a patch tonight or tomorrow.

fix released in v3.5.2
https://github.com/kristerkari/stylelint-scss/releases/tag/3.5.2


Support for serialize non ascii words

Now chinese kanji words serialized to :
å¤é¨æ¯ææå¡"
æä»¶åå,æ©æå¤é¨æå¡

not caused by FSharp.Json.
the FSharp.Data HTTP Utilities give wrong non-ascii outputs


node-gyp rebuild error on m1 Mac: ld: warning: ignoring file, building for macOS-x86_64 but attempting to link with file built for macOS-arm64

When I build, it outputs:
ld: warning: ignoring file /opt/homebrew/Cellar/libgphoto2/2.5.28/lib/libgphoto2.dylib, building for macOS-x86_64 but attempting to link with file built for macOS-arm64 ld: warning: ignoring file /opt/homebrew/Cellar/libgphoto2/2.5.28/lib/libgphoto2_port.dylib, building for macOS-x86_64 but attempting to link with file built for macOS-arm64
This is my binding.gyp file. I am not familiar with C++ or this library so I changed the target_arch to "arm" based on what I am seeing from the error but it is still not working. Can someone advise me please ? Thank you so much and appreciate it.

m1 Mac Monterey
node v13.14.0
npm v6.14.4
modifying https://github.com/lwille/node-gphoto2 to compile on my m1 macbook

{
  'targets': [
    {
      'target_name': 'gphoto2',
      'include_dirs': [
        "<!(node -e \"require('nan')\")"
      ],
      'sources': [
        'src/autodetect.cc',
        'src/binding.cc',
        'src/camera.cc',
        'src/camera_helpers.cc',
        'src/gphoto.cc'
      ],
      'link_settings': {
        'libraries': [
          '<!(/opt/homebrew/bin/pkg-config --libs libgphoto2)',
          '<!(/opt/homebrew/bin/pkg-config --libs libgphoto2_port)'
        ]
      },
      'cflags': [
        '--std=c++11'
      ],
      'cflags!': [
        '-fno-exceptions'
      ],
      'target_arch': 'arm',
      'conditions': [
        ['OS=="mac"', {
          'xcode_settings': {
            'GCC_ENABLE_CPP_EXCEPTIONS': 'YES',
            'OTHER_CPLUSPLUSFLAGS' : [
              '-std=c++11',
              '-stdlib=libc++',
              '<!(/opt/homebrew/bin/pkg-config --cflags libgphoto2)',
            ]
          }
        }]
      ]
    }
  ]
}

Command typed?
Full error log?

Command typed?
Full error log?

Hello. Thank you. The command I used is  node-gyp rebuild.
> node-gyp rebuild

  CXX(target) Release/obj.target/gphoto2/src/autodetect.o
  CXX(target) Release/obj.target/gphoto2/src/binding.o
  CXX(target) Release/obj.target/gphoto2/src/camera.o
  CXX(target) Release/obj.target/gphoto2/src/camera_helpers.o
  CXX(target) Release/obj.target/gphoto2/src/gphoto.o
  SOLINK_MODULE(target) Release/gphoto2.node
ld: warning: ignoring file /opt/homebrew/Cellar/libgphoto2/2.5.28/lib/libgphoto2.dylib, building for macOS-x86_64 but attempting to link with file built for macOS-arm64
ld: warning: ignoring file /opt/homebrew/Cellar/libgphoto2/2.5.28/lib/libgphoto2_port.dylib, building for macOS-x86_64 but attempting to link with file built for macOS-arm64

There is no error when I type this command. When I run the package, there is error so I suspect it is because of these 2 warnings.:
user@MacBook-Pro node test.js
dyld[26785]: missing symbol called
zsh: abort      node test.js

This looks like a homebrew problem to me.  Try:
brew doctor
brew —prefix  # ensure this returns: /opt/homebrew
brew uninstall libgphoto2
brew install libgphoto2

Oh...  My mistake.    So we want to:

have brew install the non-M1 library into /usr/local/Homebrew and then
have node-gyp rebuild look for the library in /usr/local/Homebrew

Hi Christian
Thank you for your update.
Best
On 6 Jan 2022, at 11:52 PM, Christian Clauss @.***> wrote:

Oh... My mistake.  So we want to:

have brew install the non-M1 library into /usr/local/Homebrew and then
have node-gyp rebuild look for the library in /usr/local/Homebrew

—
Reply to this email directly, view it on GitHubhttps://github.com/nodejs/node-gyp/issues/2586#issuecomment-1006698502, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFMHYW4OSXAHE7PHLWNHZ7LUUW3CZANCNFSM5LMKYVMQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>

Why not just brew install gphoto2

Brew install works, but I am looking to use this node wrapper > https://github.com/lwille/node-gphoto2

Check lwille/node-gphoto2#143 and maybe open a new issue there.

Thanks

Node version 8 install. node-gyp working

@cclauss

have node-gyp rebuild look for the library in /usr/local/Homebrew, not /opt/homebrew
Is there any argument for doing the same?

I ran into a similar problem with electron.js when I am trying to cross compile for macos for Intel from my M1 macbook.


Conform to PEP 420

backports is a namespace package, however, it has a init.py which is conforming the rejected PEP 382. PEP 420, which is final, states that

Namespace packages cannot contain an init.py

PEP 420 specification: https://www.python.org/dev/peps/pep-0420/#specification
PEP 382: https://www.python.org/dev/peps/pep-0382/
Discussion where we encountered this issue: https://github.com/NixOS/nixpkgs/issues/22319
Issue for backports.ssl_match_hostname

@FRidh pep420 doesn't apply to python2.7 - this is setuptools namespaces that need the file

Indeed, closing.


Change height of upload box

The upload input box has a hight of 38px and the input box has a hight of 40.53. Change the upload input box height.

Ticket cannot be done as the component is a restyled HTML element with very little additional customisability from what is already there. We cannot add extra padding to make it bigger, nor can we add more padding-left to the "No file chosen" text. The component will need a redesign @Majsol. Some considerations, if we separate out the component from the button with a label such as this example:

We will need to include some javascript and if the person doesn't have javascript, then there will be no text that indicates that shows the user has uploaded a file.

On the UCD Board: https://jira.ons.gov.uk/browse/CU-104.

This issue is currently parked on the UCD Board.


Visual mode

Is your feature request related to a problem? Please describe.
We can't select text as we do in vim by entering visual mode and moving the cursor.
Describe the solution you'd like
In normal mode:

Pressing v key should change to visual mode.
Pressing V key should change to line visual mode.

Additional context
Vimium has already faced this at https://github.com/philc/vimium/issues/61

+1


update schema existing check validation considering the distinction between uppercase and lowercase

Resolves #

Description
I have made a modification to consider case sensitivity when checking for the presence of a schema. Databricks manages schemas in lowercase, so if there isn't an exact match, it can cause problems in subsequent processes. For example, when setting materialized='incremental' in the SQL Config for an append operation, it checks for the existence of the table before appending the differential data. If the schema is passed in uppercase in this case, since the schema on Databricks is in lowercase, the check for the table's existence will fail, and instead of appending, it will run recreate the existing table even if using incremental option. We have now fixed it to ensure an exact match during the initial schema check.
{{ config(
    materialized='incremental',
    incremental_strategy='append',
) }}

Checklist

[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt-databricks next" section.

It's unclear how the change you're proposing accomplishes your stated goal.  It seems if anything, that your change will try to compare a potentially mixed-case schema against a list that I think is natively lower-cased.

@benc-db
Thank you for your reply. Let me explain. Both Normal Expected Case and Abnormal Unexpected Case
🔹 Normal Expected Case：Defining the schema in lowercase
We use the following profiles.yml and execute the incremental SQL. If the schema is defined in lowercase, there are no problems
databricks_demo:
  target: dev
  outputs:
    dev:
      catalog: catalog_demo
      host: <host>
      http_path: <http_path>
      schema: schema_demo
      threads: 1
      token:  "<token>"
      type: databricks

Execution Log
Upon checking the target table, it can be seen that it has been incremented

dbt run
01:48:33  2 of 2 START sql incremental model schema_demo.daily_model_2 ................... [RUN]
01:48:39  2 of 2 OK created sql incremental model schema_demo.daily_model_2 .............. [OK in 6.12s]

Table Hisotry：
Upon checking the history, it can be seen that it has been appended (Write).

🔹 　Abnormal Unexpected Case：Defining the schema in uppercase
We use the following profiles.yml and execute the incremental SQL. If the schema is defined in uppercase, it behaves unexpectedly
databricks_demo:
  target: dev
  outputs:
    dev:
      catalog: catalog_demo
      host: <host>
      http_path: <http_path>
      schema: Schema_Demo
      threads: 1
      token:  "<token>"
      type: databricks

Execution Log
Upon checking the target table, it can be seen that it has been incremented. But table History is wrong

dbt run
01:48:33  2 of 2 START sql incremental model Schema_Demo.daily_model_2 ................... [RUN]
01:48:39  2 of 2 OK created sql incremental model Schema_Demo.daily_model_2 .............. [OK in 6.12s]

Table History
However, upon checking the history, it turns out that it has been recreated.

This happens because when doing an incremental update, it checks for the existence of the table. If the schema is passed in uppercase, it is determined that the table does not exist and thus a recreate is triggered. While it is possible to fix this on the incremental side, I thought it would be more considerate to check beforehand if the schema defined in profiles.yml exactly matches.

Thank you for your reply. I am attempting to integration test, but I haven't yet been able to create the environment for that.I am trying to do following document way.
https://github.com/databricks/dbt-databricks/blob/main/CONTRIBUTING.MD#functional--integration-tests
What I want to achieve is that if the 'profiles' schema is passed in uppercase, it would be treated as a non-existent schema, leading to a connection failure and a fail state. Currently, even if passed in uppercase, it does not fail to connect, which causes issues in subsequent processes like I shared. By removing the 'lower', I believe it will compare the uppercase 'profiles' schema passed with the lowercase schema retrieved from Databricks, and fail the connection as a non-existent schema.

If the schema is passed in uppercase in this case, since the schema on Databricks is in lowercase, the check for the table's existence will fail

So, I'm now understanding that you are trying to make it fail at the schema step so that you don't recreate the table.  I didn't realize that was your goal initially.  Ok, so I think the fix that is more inline with our principles is to find out why the table check is failing and fix that instead.  In general we trying to make everything lowercase for comparison because, to my knowledge, Databricks is not supposed to be case sensitive.  I think getting rid of the lowercasing here will be too disruptive.  Can we instead identify why the table check is failing?

I think getting rid of the lowercasing here will be too disruptive. Can we instead identify why the table check is failing?

Yea of course we can identify and can discuss the good way to solve this issue. Thank you!

I have created the new PR  to investigate this schema issue. Could you also check this PR?
https://github.com/databricks/dbt-databricks/pull/538
This PR help to pass default schema option and will be useful to investigate this issue

Hi @benc-db
I fix this issue could you please check this change? I have test it locally and confirm the issue has been fixed. I pass the uppercase of schema  and confirm instead of recreate table, append the rows into existing tables
To pass the upper case in integration test , I need to fix existing integration environment.  I have created that change as different PR. So please check this PR as well
https://github.com/databricks/dbt-databricks/pull/541
Test Results
 .tox/integration-databricks-uc-sql-endpoint/bin/python -m pytest -v -m profile_databricks_uc_sql_endpoint -n4 tests/integration/incremental_strategies/test_incremental_strategies.py::TestDeltaStrategiesWarehouse::test_delta_strategies_databricks_uc_sql_endpoint
=========================================== test session starts ===========================================
platform darwin -- Python 3.11.5, pytest-7.4.3, pluggy-1.3.0 -- /Users/user/Library/CloudStorage/OneDrive/Desktop/workspace/dbt-databricks/.tox/integration-databricks-uc-sql-endpoint/bin/python
cachedir: .pytest_cache
rootdir: /Users/user/Library/CloudStorage/OneDrive/Desktop/workspace/dbt-databricks
configfile: pytest.ini
plugins: csv-3.0.0, flaky-3.7.0, dotenv-0.5.2, xdist-3.5.0
4 workers [1 item]
scheduling tests via LoadScheduling

tests/integration/incremental_strategies/test_incremental_strategies.py::TestDeltaStrategiesWarehouse::test_delta_strategies_databricks_uc_sql_endpoint

[gw0] [100%] PASSED tests/integration/incremental_strategies/test_incremental_strategies.py::TestDeltaStrategiesWarehouse::test_delta_strategies_databricks_uc_sql_endpoint

============================================ warnings summary =============================================
tests/integration/incremental_strategies/test_incremental_strategies.py::TestDeltaStrategiesWarehouse::test_delta_strategies_databricks_uc_sql_endpoint
tests/integration/incremental_strategies/test_incremental_strategies.py::TestDeltaStrategiesWarehouse::test_delta_strategies_databricks_uc_sql_endpoint
tests/integration/incremental_strategies/test_incremental_strategies.py::TestDeltaStrategiesWarehouse::test_delta_strategies_databricks_uc_sql_endpoint
tests/integration/incremental_strategies/test_incremental_strategies.py::TestDeltaStrategiesWarehouse::test_delta_strategies_databricks_uc_sql_endpoint
tests/integration/incremental_strategies/test_incremental_strategies.py::TestDeltaStrategiesWarehouse::test_delta_strategies_databricks_uc_sql_endpoint
tests/integration/incremental_strategies/test_incremental_strategies.py::TestDeltaStrategiesWarehouse::test_delta_strategies_databricks_uc_sql_endpoint
tests/integration/incremental_strategies/test_incremental_strategies.py::TestDeltaStrategiesWarehouse::test_delta_strategies_databricks_uc_sql_endpoint
  /Users/user/Library/CloudStorage/OneDrive/Desktop/workspace/dbt-databricks/dbt/adapters/databricks/connections.py:693: UserWarning: The cursor was closed by destructor.

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
================================ 1 passed, 7 warnings in 85.58s (0:01:25) =================================

Query History
Checked uppercase of schema  has been passed from integration test

Table History: Checked append the row as expected

I think this is the same issue
https://github.com/databricks/dbt-databricks/issues/523

Sorry for the delay, was taking holidays.  Will take a look today.

@benc-db Thank you I update Table History sample more better one.

Hello @benc-db Thank you for the other PR check that I requested ! I appreciate if you check this PR as well when you have time. Thank you.

@case-k-git please resolve the conflict, and then I will run tests and review :)

@benc-db  Thank you fix the conflict

Thanks for your help and patience.  Will merge today, and this will be in the next release (which I expect will come out next week).

@benc-db  Thank you !!