[staging-next] llvmPackages_{7-12}.compiler-rt: install resource files to DATADIR

This is in an effort to fix the following build failure shown by
chromium:
clang++: error: no such file or directory: '/nix/store/fhd89wrmkx6nflzjk0d6waz70bk3zc4i-clang-wrapper-12.0.0/resource-root/share/cfi_blacklist.txt'

As it turns out a change introduced via the gnu-install-dirs.patch
caused add_compiler_rt_resource_file to install resource files to
$dev/include (FULL_INCLUDEDIR) instead of $out/share (FULL_DATADIR)
which in turn meant that the clang wrappers we had didn't link those
files to its resource root at all.
Alternative fix to this would have been to link
compiler-rt.dev/include/*.txt to the wrappers resource-root/share as
well, but since this was handled inconsistently across the patch anyways
(the dfsan list is installed correctly), opt to handle this
consistently within the patch.
llvmPackages_{5,6} install the resource files to a completely different
location and need separate investigation.
lr $(nix-build -A llvmPackages_12.stdenv.cc)/resource-root/share/
/nix/store/5i59ga5gfw08mg1cxyya5dspw2m63gai-clang-wrapper-12.0.0/resource-root/share/
/nix/store/5i59ga5gfw08mg1cxyya5dspw2m63gai-clang-wrapper-12.0.0/resource-root/share/asan_blacklist.txt
/nix/store/5i59ga5gfw08mg1cxyya5dspw2m63gai-clang-wrapper-12.0.0/resource-root/share/cfi_blacklist.txt
/nix/store/5i59ga5gfw08mg1cxyya5dspw2m63gai-clang-wrapper-12.0.0/resource-root/share/dfsan_abilist.txt
/nix/store/5i59ga5gfw08mg1cxyya5dspw2m63gai-clang-wrapper-12.0.0/resource-root/share/hwasan_blacklist.txt
/nix/store/5i59ga5gfw08mg1cxyya5dspw2m63gai-clang-wrapper-12.0.0/resource-root/share/msan_blacklist.txt

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

> echo nix-build -j0 --builders 'ssh://edwin' -A\ llvmPackages_{7,8,9,10,11,12}.stdenv.cc | sh | while read line
                                                          ls $line/resource-root/share/
                                                      end
asan_blacklist.txt  cfi_blacklist.txt  dfsan_abilist.txt  hwasan_blacklist.txt  msan_blacklist.txt
asan_blacklist.txt  cfi_blacklist.txt  dfsan_abilist.txt  hwasan_blacklist.txt  msan_blacklist.txt
asan_blacklist.txt  cfi_blacklist.txt  dfsan_abilist.txt  hwasan_blacklist.txt  msan_blacklist.txt
asan_blacklist.txt  cfi_blacklist.txt  dfsan_abilist.txt  hwasan_blacklist.txt  msan_blacklist.txt
asan_blacklist.txt  cfi_blacklist.txt  dfsan_abilist.txt  hwasan_blacklist.txt  msan_blacklist.txt
asan_blacklist.txt  cfi_blacklist.txt  dfsan_abilist.txt  hwasan_blacklist.txt  msan_blacklist.txt

I'd strongly suggest that we should first have some solution that doesn't involve rebuilding darwin stdenv.  Otherwise it yet another few days of waiting.

I'd strongly suggest that we should first have some solution that doesn't involve rebuilding darwin stdenv. Otherwise it yet another few days of waiting (for chromium to work).

If we only touch mkExtraBuildCommands* in llvm/*/default.nix, that should achieve that, right?
(As a side note, seems like darwin stdenv doesn't link the resource files into the resource root at all).

In the worst case we could make the change linux-only for now or something.

Result of nixpkgs-review pr 123103 at 01971cf6 run on aarch64-linux 1

28 packages marked as broken and skipped:

clang-sierraHack
clang-sierraHack-stdenv
darwin.maloader
embree
embree2
gnome.gnome-documents
idris2
intel-compute-runtime
jucipp
libsForQt512.discover
...

4 packages failed to build:

couchdb3
openvpn-auth-ldap
saw-tools
sbt-with-scala-native

363 packages skipped due to time constraints:

adapta-gtk-theme
adoptopenjdk-icedtea-web
almanah
apostrophe
appleseed
arc-theme
arx-libertatis
asciidoc-full
asciidoc-full-with-plugins
astroid
...

30 packages built successfully:

clang (clang_7 ,llvmPackages.clang ,llvmPackages.libstdcxxClang ,llvmPackages_7.clang ,llvmPackages_7.libstdcxxClang)
clangStdenv (gnustep.stdenv ,llvmPackages.stdenv ,llvmPackages_7.stdenv)
clang_9 (haskellPackages.llvmPackages.clang ,haskellPackages.llvmPackages.libstdcxxClang ,llvmPackages_9.clang ,llvmPackages_9.libstdcxxClang)
fcitx5-mozc
glib-networking (gnome.glib-networking ,gnome.glib_networking)
gnome.gnome-shell-extensions
gnustep.back
gnustep.base
gnustep.gorm
gnustep.gui
gnustep.libobjc
gnustep.make
gnustep.projectcenter
gnustep.system_preferences
llvmPackages_9.stdenv (haskellPackages.llvmPackages.stdenv)
innernet
parinfer-rust (kakounePlugins.parinfer-rust)
kime
libproxy
mediatomb
openshift
ostree
python38Packages.johnnycanencrypt
python38Packages.tumpa
python39Packages.johnnycanencrypt
python39Packages.tumpa
spidermonkey_68
tests.cc-wrapper-clang (tests.cc-wrapper-clang-7)
txr
unar

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

In the worst case we could make the change linux-only for now or something.

The linux builders should be able to chew through it in a day. Darwin will suffer greatly as it takes many days for it to complete a stdenv rebuild; and darwin has delaying all of this enough as it has. Delaying staging-next any further is now taking away from doing stabilization from the next staging-next.

For linux, this only causes ~600 rebuilds [53/84/617 built ..... I'm fine with merging it assuming that it address the chromium issue.

I'm trying to find a workaround which saves us from the darwin stdenv rebuild. If that doesn't pan out we'll have to have two different patches for darwin and linux I guess.
In any case I would propose that we try to merge the proper fix (whatever it'll look like) with the next staging-next PR then?

@GrahamcOfBorg build chromium

Yikes :(. I'm sorry this one got in there.

Yea, not sure why I thought it was a good idea for ofborg to try and build chromium

Since the reduced changes were merged into staging-next. Should this be rebased on staging, so that the stdenv rebuild can be done on the next staging-next?

Sounds good to me.


Bump up Deployment version

I am creating a bunch of PRs for bumping up deployment version.  A pr contains all changes in a folder.

This change is

@nashasha1 thank you for the efforts, we do appreciate it. This topic of how to represent the changes to versions has come up previously and we don’t yet have solid guidance for our conventions in these docs other than to also specify the version that the example applies to for clarity. It’s going to be discussed (hopefully) in the SIG Docs meeting tomorrow, following on issue #7232, so I’m going to put a /hold on all of these until we have some improved instructions.
When you are doing these broad changes, it is also much easier for us to review the changes when they are all lumped together into a single PR with a single purpose (such as updating the API version tags), so doing these kinds of things as a single PR would be very much appreciated in the future
/hold

@heckj  Thank you for your reply. Sorry for confusing you guys. I thought the size is too big if I put all of these in one pr.  Next time, I will put them together and make a clear tag.
Waiting for your good news.

/assign

Could you please review that recommendation and update these PRs to include min-kubernetes-server-version in the front-matter of the various documents?

@heckj  Thanks for your reply. I will close this PR and move the changes to  kubernetes/kubernetes-docs-cn.


Bug on resume training on single GPU and multiple GPUs

System Info
accelerate: 0.12.0

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[X] My own task or dataset (give details below)

Reproduction
When loading checkpoint with accelerator.load_state() for resuming training, the DataLoader will sample data from scratch from epoch 0 step 0.
An extreme case in (https://github.com/huggingface/accelerate/blob/main/examples/complete_nlp_example.py) is that the codes can only train a small part of the whole data when resuming many times.
In this example, https://github.com/huggingface/accelerate/blob/main/examples/complete_nlp_example.py#L185, In fact,  it is resumed from epoch 0, not the start epoch.
import torch
import torch.utils.data as Data
from accelerate import Accelerator
from accelerate.utils import set_seed

accelerator = Accelerator()
BATCH_SIZE = 3
set_seed(42)
x = torch.linspace(1, 12, 12)
print(x)
y = torch.linspace(12, 1, 12)
print(y)
torch_dataset = Data.TensorDataset(x, y)

output_path = "save_path"

loader = Data.DataLoader(
    dataset=torch_dataset,
    batch_size=BATCH_SIZE,
    shuffle=True,
)

loader = accelerator.prepare(loader)
start_epoch = 0
# when start_epoch =2, how to reach epoch 2 and step 1?
for epoch in range(start_epoch, 5):
    for step, (batch_x, batch_y) in enumerate(loader):
        print(batch_x, batch_y)
        if epoch ==2 and step==1:
            accelerator.save_state(output_path)

When accelerator.load_state(output_path),  how to reach epoch 2 and step 1? I tested (one GPU and multiple GPUs) that the loader samples data from epoch 0 step 0. The function of set_epoch is expected.
Expected behavior
Expect reach the data at epoch 2 and step 1.

This is not a bug. You need to resume your training in the exact same distributed setup.


"Special channel" windows should not have date/unread markers

See:

These make little sense there I think.
I wanted to do the 30-second solution of simply hiding in CSS, but I don't know if that's the right thing to do or if it should be handled by the server instead, which seems cleaner to me. @xPaw, thoughts?

What if you /list on multiple days?

Right now it clears the whole page (as it should IMO) so really the markers are useless right now. I think it's a niche use case to open a channel list on different days to compare, so I'm not too worried.

I think this was fixed at some point?

Fixed in https://github.com/thelounge/thelounge/pull/1849 indeed, good catch!


Can not drag-n-drop to open files in Corel applications (CorelDRAW, Corel PhotoPaint)

Description
In the earlier (v2) versions of Files it was possible to drag-and-drop files (images, PDFs, etc.) from Files explorer in to the Corel applications to import them. For whatever reason it's not working anymore in Files v3 - trying to drag the image in to the Corel workspace leads to cursor pointer changing over to "Unavailable".
A quick sidenote - Corel is the second most popular commercial graphic and vector editing software vendor in the world after Adobe Photoshop/Illustrator, used by millions of designers, artists, architects and engineers across the world.
Steps To Reproduce

Open any folder containing valid images/PDFs in Files explorer.
Try to drag any applicable file over to any Corel application.
Instead of importing the file like it used before upgrading to Files v3, cursor changes to "Unavailable" icon and action is not possible.

Requirements
Drag-n-drop should work as it did in Files v2.
Files Version
<IP_ADDRESS>
Windows Version
23H2 (OS build 22631.3155)
Log File

Thanks for the report, We're tracking this in #11735. You can check there or click subscribe in the right sidebar for updates.


Navigated event is not fired correctly on new page creation

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
Site designs & site scripts
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[X] 💥 Google Chrome
[X] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser chrome and firefox

Describe the bug / error
Navigation event is not fired correctly on new page creation.  When the page is saved it is directed to the recycle bin. In the recycle bin it ends up duplicated in two different formats and folders.
In the recycle bin there are 2 original locations:

tenant/sites/nameofthesite/SitePages (the correct location) in aspx format
tenant/sites/nameofthesite/SitePages/News (the wrong location) in unknown format

I have deleted the folder location at: tenant/sites/nameofthesite/SitePages/News location but it reappears when a new page is created.
When running Get-SPOSite -Template REDIRECTSITE#0 the url above is not listed.
Steps to reproduce

Create new page
Save new page (/sites/nameofthepage/_api/sitepages/page(637)/SavePage

Expected behavior
When Creating a new page (or news post) the url is "SitePages/newpage.aspx?Mode=Edit"
Then to a generated the page the URL changes to e.g "SitePages/nmorjnzr.aspx?Mode=Edit"
Then to whatever title  set in the page e.g "SitePages/addedpage.aspx"

{"odata.error":{"code":"-1, Microsoft.SharePoint.Client.ClientServiceException","message":{"lang":"en-US","value":"Item does not exist. It may have been deleted by another user."}}}

+1 on this


Move use_anime_en_title to Burst.

for https://github.com/elgatito/plugin.video.elementum/issues/987
to be merged AFTER https://github.com/elgatito/script.elementum.burst/pull/389

Probably it makes sense.
I think we had this en -> original hack before anime:bool was passed to search addons.


s3TablePersister must pass rate limiter to compacted tables

Don't know how much this has been an issue in practice, but
it was definitely wrong.

PTAL


HTTPS SSL TLS Using websockets doesnt work

Line 381 in Websocket.jl:

sock = WebSocket(client.id, client.sock)

Receives a MbedTLS SSLContext in the client.sock value as per HttpServer.jl:
https://github.com/JuliaWeb/WebSockets.jl/issues/51
sess = MbedTLS.SSLContext()
     MbedTLS.setup!(sess, ssl_config)
     # set_priority_string!(sess)
     # set_credentials!(sess, cert_store)
     client = accept(server.http.sock)
     try
         MbedTLS.set_bio!(sess, client)
         MbedTLS.handshake(sess)
         # associate_stream(sess, client)
         # handshake!(sess)
     catch e
         println("Error establishing SSL connection: ", e)
         close(client)
         continue
     end
     client = Client(id_pool += 1, sess)
     client.parser = ClientParser(message_handler(server, client, websockets_enabled))
     @async process_client(server, client, websockets_enabled)`

As I'm not sure which project is better to help massage the socket into, I'm looking for help on this one. Please let me know what to do! What else might be needed to make websockets on TLS work? Thanks

Here's more interesting debug info:

HttpServer.Client{MbedTLS.SSLContext}(14,MbedTLS.SSLContext(Ptr{Void} @0x0000000003245910,MbedTLS.SSLConfig(Ptr{Void} @0x0000000002dff880,MbedTLS.CtrDrbg(Ptr{Void} @0x0000000002ca2f00,Nullable(MbedTLS.Entropy(Ptr{Void} @0x0000000002e54c90,Any[]))),#undef,(anonymous function),cert. version     : 3
serial number     : DB:7E:D8:55:3E:F1:AB:AC
issuer name       : C=US, ST=CA, L=San Francisco, O=BXI, CN=bxi.io<EMAIL_ADDRESS>subject name      : C=US, ST=CA, L=San Francisco, O=BXI, CN=bxi.io<EMAIL_ADDRESS>issued  on        : 2016-02-21 07:47:43
expires on        : 2017-02-20 07:47:43
signed using      : RSA with SHA-256
RSA key size      : 2048 bits
basic constraints : CA=true
,MbedTLS.PKContext(Ptr{Void} @0x0000000002cf1160)),true,TCPSocket(open, 0 bytes waiting)),HttpServer.ClientParser(HttpParser,HttpParser.ParserSettings(Ptr{Void} @0x00007fcb39e9ffc0,Ptr{Void} @0x00007fcb39ea35d0,Ptr{Void} @0x00007fcb39ea3810,Ptr{Void} @0x00007fcb39ea3850,Ptr{Void} @0x00007fcb39ea3a80,Ptr{Void} @0x00007fcb39ea4c80,Ptr{Void} @0x00007fcb39ea50a0,Ptr{Void} @0x00007fcb39ea5210)))WARNING: both Patchwork and Base export "Text"; uses of it in module Main must be qualified
ERROR (unhandled task failure): MethodError: convert has no method matching convert(::Type{TCPSocket}, ::MbedTLS.SSLContext)
This may have arisen from a call to the constructor TCPSocket(...),
since type constructors fall back to convert methods.
Closest candidates are:
TCPSocket(::Any)
call{T}(::Type{T}, ::Any)
convert{T}(::Type{T}, ::T)
...
in call at /home/a/.julia/v0.4/WebSockets/src/WebSockets.jl:57
in handle at /home/a/.julia/v0.4/WebSockets/src/WebSockets.jl:387
in on_message_complete at /home/a/.julia/v0.4/HttpServer/src/HttpServer.jl:393
in on_message_complete at /home/a/.julia/v0.4/HttpServer/src/RequestParser.jl:104

Bump


Feature/fbp 261 camunda plugin

TODOs:

[ ] task

[ ] sendTask

[ ] complexGateway

[ ] intermediateCatchEvent.link

[ ] outgoing/incoming sequence refs within elements

Codecov Report

Merging #265 (95b4d9d) into master (98acb0d) will decrease coverage by 0.06%.
The diff coverage is 60.82%.

@@             Coverage Diff              @@
##             master     #265      +/-   ##
============================================
- Coverage     61.27%   61.21%   -0.07%
- Complexity      997     1199     +202
============================================
  Files           136      176      +40
  Lines          4925     5724     +799
  Branches        692      762      +70
============================================
+ Hits           3018     3504     +486
- Misses         1675     1968     +293
- Partials        232      252      +20

Flag
Coverage Δ

unittests
61.21% <60.82%> (-0.07%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ugin/camunda/CamundaBpmnPluginToolWindowFactory.kt
0.00% <0.00%> (ø)

...munda/CamundaBpmnPluginToolWindowProjectService.kt
0.00% <0.00%> (ø)

.../intellij/plugin/camunda/CamundaPluginPreloaded.kt
0.00% <0.00%> (ø)

...nda/actions/AttachCamundaBpmnDebuggerToDbAction.kt
0.00% <0.00%> (ø)

...lij/plugin/camunda/actions/NewCamundaBpmnAction.kt
0.00% <0.00%> (ø)

...in/camunda/actions/ViewCamundaBpmnDiagramAction.kt
0.00% <0.00%> (ø)

...onents/popupmenu/CamundaCanvasPopupMenuProvider.kt
0.00% <0.00%> (ø)

...ugin/camunda/parser/nodes/process/BoundaryEvent.kt
0.00% <0.00%> (ø)

...amunda/parser/nodes/process/BpmnElementIdMapper.kt
60.00% <60.00%> (ø)

...in/camunda/parser/nodes/process/AdHocSubProcess.kt
63.63% <63.63%> (ø)

... and 30 more

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 98acb0d...95b4d9d. Read the comment docs.


A recent change broke a documentation link

A recent change to this repo is causing build errors on Azure docs.

Here is the article that now has 404's: https://docs.microsoft.com/en-us/azure/cosmos-db/sql-api-sdk-java-spring-v3?tabs=docs

My mistake, my PR was behind master. The 404's are still happening in prod but I'm guessing the docs site hasn't re-built yet
https://github.com/MicrosoftDocs/azure-docs-pr/commit/63647b757064d408023611acf991eef778e8de0d


Re-read a livebook from the filesystem

If you edit a livebook in a text editor while it is open in livebook -  for example to add the magic <!-- vim: syntax=markdown --> line - the livebook does not seem to pick up the changes made in the editor, the only way to see the change is to close the session and re-open it.
Is there a way to refresh the livebook to pick up these changes (even if it's a button)?
I understand this may get tricky with auto-save on.

Correct. It is complicated with auto-save. My hope is that soon GitHub will treat .livemd as markdown, and then we will no longer need this annotation and the problem will disappear. If that does not happen, then we should allow this annotation to happen within Livebook’s UI, but I want to wait a bit.

That makes sense, my other use case is I've been playing with generating livebook templates so testing that is tricky, but I appreciate that is not a usual requirement!


Groups of columns

Hello again 🙂
Some of our users have requested a feature that is not currently implemented in Grist which is the ability to define groups of columns within a table widget.
Their need is twofold :

within a big table of many dozens of columns, being able to collapse/hide a group of columns in a single click
easily navigate between business-relevant groups of columns, easily create views, etc.

We are aware that these two needs can be catered to via different views, coloring columns (it would be great if we could color headers only btw), and this comment seems to indicate that "hiding multiple columns in one click" is underway. But we thought we'd share this user feedback with you :)
WDYT ?

Ooh, I like this idea!
I am picturing something similar to how browser tabs can be grouped and ungrouped, effectively collapsing and uncollapsing a collections of tabs. Is that similar to what you had in mind? If a collapsed column group always hidden, or is there a state where a group is collapsed but visible in the widget for easy access. If a group can be visible-and-collapsed, what do you see in the column, if anything?
I wonder how it would look in the creator panel where columns can be hidden and unhidden (and rearranged) more easily. I'm referring to this part of the UI.

I am picturing parent-child nesting, with rules about how they can be rearranged, hidden, unhidden as a group.
Color headers would be great! Are you picturing the ability to set column header colors, or to automatically apply a column header color that is derived from the cell styles fill color (either the same color or a darker shade)?
Btw, you can hide multiple columns at once now. Select a range of adjacent columns and hide them either through the column context menu, or by using the shortcut alt + shift + -.

I am picturing something similar to how browser tabs can be grouped and ungrouped, effectively collapsing and uncollapsing a collections of tabs. Is that similar to what you had in mind?

Yes, this is what I had in mind :)

If a collapsed column group always hidden, or is there a state where a group is collapsed but visible in the widget for easy access. If a group can be visible-and-collapsed, what do you see in the column, if anything?

I would say that yes, this could be an interesting feature. I'd say if a group can be visible-and-collapsed you'd want a column with ... or an expand button like <-> for easy reopening ?

I wonder how it would look in the creator panel where columns can be hidden and unhidden (and rearranged) more easily. I'm referring to this part of the UI.

I am picturing parent-child nesting, with rules about how they can be rearranged, hidden, unhidden as a group.

I think I see what you mean, and I agree that we should be careful about this ! Will talk about it with @LouisDelbosc and see what we can do :)

Color headers would be great! Are you picturing the ability to set column header colors, or to automatically apply a column header color that is derived from the cell styles fill color (either the same color or a darker shade)?

Setting column header colors is what I had in mind. I think we should have the option of only having the header colored.

Btw, you can hide multiple columns at once now. Select a range of adjacent columns and hide them either through the column context menu, or by using the shortcut alt + shift + -.

Awesome ! Already started using this feature and loving it 🤩

Given we can hide multiple columns at once AND we can  now edit multiple column at once, maybe the first step could be to simply allow to apply a colour to a header column ? I mean as a column property.
The grouping seems to be quite complex at the UI/UX level and needs more thinking I'd say.
@anaisconce what do you think ? :)

@anaisconce I made some change suggested by @vviers comment. What do you think of it ?

Hello,
I would like to update this issue.
There were some ideas between the issue and Louis' PR:

create column groups
be able to give a color to the column header (other than the default column color)

The two functionalities seem to be different things, and the second seems easier and quicker to develop both technically and in terms of design. Would you mind if I put forward a proposal soon?

That sounds good, @CamilleLegeron Interested in seeing your proposal! 👍

Closed via #581

My bad, I suggested Vincent to close the issue. In fact, the enhancement issue is partially implemented as suggested @CamilleLegeron here. I moved too quickly!

I'm not sure that we want to tackle the bigger issue of columns groups just yet — tagging this as needs design but this is by no means a priority

We would love to have simple column grouping as in Excel / Sheets: Select columns, group them and collapse / expand them with a click.

to discuss https://www.figma.com/design/wcpetFt6aOKzTszcvPPWLQ/[05%2F24]-Grist-Design?node-id=3061-145641&t=KmV2MAenZUNU3Plk-1


[Bug] About us page is not responsive #539

Fixes Issue🛠539
Closes #539
Description👨💻
NOW THE TEXT IS NOT OVERLAPPING IT WAS RESPONSIVE  AND WHOLE PAGE BECOME RESPONSIVE NOW .
Type of change📄
Bug fix (non-breaking change which fixes an issue)
How this has been tested✅
IN MY OWN EDITOR
Checklist✅

[✅ ] My code follows the style guidelines of this project
[✅ ] I have performed a self-review of my own code
[✅ ] I have commented my code, particularly in hard-to-understand areas
[✅ ] I have added demonstration in the form of GIF/video file
[✅ ] I am an Open Source Contributor

Screenshots/GIF📷

Pls don't change the main UI for big screens.
Properly make it just responsive for smaller screens. Navbar UI should not be changed!! @student-aditipaliwal


Add pod workload metadata to CiliumEndpoint CRD

Rework of https://github.com/cilium/cilium/pull/27974 and https://github.com/cilium/cilium/pull/28373 (seem abandoned).
This PR adds a new Workload object to the Endpoint struct and CiliumEndpoint CRD (+ related) which can be used by Hubble to enrich flows.
The current implementation of Hubble parsers relies on the Endpoint Resolver (contains node-local endpoints) and the ipcache (clusterwide mapping collection) to resolve the SourceEndpoint and DestinationEndpoint of a flow. When a flow contains a non-local endpoint, meaning it originated or terminated on a remote node, we rely solely on the ipcache, which previously did not contain Workload information.
We now resolve the Workload at the same time as the pod whenever we create a new Endpoint, which allows Hubble to directly access it from the Endpoint Resolver instead of needing to compute it on-demand. Furthermore, the ipcache.K8sMetadata struct has been enriched with the Workload object so we can access it for remote node endpoints as well.
TODO

[ ] Confirm Hubble metrics correctly report cross-node workload information

Testing
Deploy a kind cluster
make kind
make kind-image
make kind-install-cilium

Create 2 deployments and services
kubectl create deployment hello-node --image=registry.k8s.io/e2e-test-images/agnhost:2.39 -- /agnhost netexec --http-port=8080
kubectl create deployment hello-node2 --image=registry.k8s.io/e2e-test-images/agnhost:2.39 -- /agnhost netexec --http-port=8080
# force deployments on different nodes
kubectl patch deployments hello-node -p '{"spec": {"template": {"spec": {"nodeSelector": {"kubernetes.io/hostname": "kind-worker"}}}}}'
kubectl patch deployments hello-node2 -p '{"spec": {"template": {"spec": {"nodeSelector": {"kubernetes.io/hostname": "kind-control-plane"}}}}}'
# create service for hello-node2
kubectl expose deployment hello-node2 --type=ClusterIP --port=8080

Generate http traffic
kubectl exec -it deploy/hello-node -- curl http://hello-node2:8080

Query flows using hubble
cd hubble && make
./hubble observe -P -o json --workload hello-node --last 1 | jq '.flow | {time: .time, node_name: .node_name, src:.IP.source, dst:.IP.destination, verdict:.verdict, srcPod: .source.pod_name, srcWorkload:.source.workloads, dstPod: .destination.pod_name,  dstWorkload:.destination.workloads}'

Before 😿
{
  "time": "2024-11-17T23:11:00.838025160Z",
  "node_name": "kind-kind/kind-worker",
  "src": "<IP_ADDRESS>",
  "dst": "<IP_ADDRESS>",
  "verdict": "FORWARDED",
  "srcPod": "hello-node-6f98c48486-k228p",
  "srcWorkload": [
    {
      "name": "hello-node",
      "kind": "Deployment"
    }
  ],
  "dstPod": "hello-node2-755dfcd76b-dm7bb",
>  "dstWorkload": null
}

After 😸
{
  "time": "2024-11-17T22:18:17.332194768Z",
  "node_name": "kind-kind/kind-control-plane",
  "src": "<IP_ADDRESS>",
  "dst": "<IP_ADDRESS>",
  "verdict": "FORWARDED",
  "srcPod": "hello-node-6f98c48486-xzwr7",
  "srcWorkload": [
    {
      "name": "hello-node",
      "kind": "Deployment"
    }
  ],
  "dstPod": "hello-node2-755dfcd76b-zcng2",
>  "dstWorkload": [
    {
      "name": "hello-node2",
      "kind": "Deployment"
    }
  ]
}

Fixes: https://github.com/cilium/cilium/issues/25676
Fix cross-node Hubble flows missing the Kubernetes workload name

This makes me wonder, shouldn't hubble-relay be responsible for enriching flow information with information across nodes instead?

Relay is only used for querying Hubble via gRPC. This information needs to be handled in Hubble in order for it to be usable within metrics and flow export.

Drive-by comment, if I'm not missing anything, this will not work when using CiliumEndpointSlices.

Any insight into what's required to make this work with CES, or would it be fundamentally incompatible? I'm not very familiar with endpoint slices, though I could see potential issues.

Drive-by comment, if I'm not missing anything, this will not work when using CiliumEndpointSlices.

Any insight into what's required to make this work with CES, or would it be fundamentally incompatible? I'm not very familiar with endpoint slices, though I could see potential issues.

In general it is just a different API containing stripped down CEP information (reconciled by the operator), which would need to be extended as well here (obviously at the cost of increasing its size). Besides that, all the processing on the agent side should be fairly similar, and the entries eventually flow into the ipcache as for CEPs and kvstore data.

As @joestringer mentioned, we want to try our best to avoid increasing the size of CES. And at the same time we don’t want to have different behavior when CES is enabled and disabled.
I would first suggest attempting to figure out a way not to include additional information into the CEP/CES API resources, but instead calculate it on each node, based on the known information.
If that’s not possible, then instead of adding a field, it would be better to consider using object.metadata.annotations field to add the workload info for each endpoint. This way, the workload name can be supported as opt-in or opt-out, via configuration, so that scalability isn’t affected across the board.
I don’t know how high the demand for this change is. Regardless, it has scalability implications.

As @joestringer mentioned, we want to try our best to avoid increasing the size of CES. And at the same time we don’t want to have different behavior when CES is enabled and disabled.
I would first suggest attempting to figure out a way not to include additional information into the CEP/CES API resources, but instead calculate it on each node, based on the known information.
If that’s not possible, then instead of adding a field, it would be better to consider using object.metadata.annotations field to add the workload info for each endpoint. This way, the workload name can be supported as opt-in or opt-out, via configuration, so that scalability isn’t affected across the board.
I don’t know how high the demand for this change is. Regardless, it has scalability implications.

It seems the original author first considered using metadata for this but during the review it was proposed to use a CEP field instead: https://github.com/cilium/cilium/pull/27974

I would first suggest attempting to figure out a way not to include additional information into the CEP/CES API resources, but instead calculate it on each node, based on the known information.

It seems like this is the current behaviour and it works well but only for node-local endpoints. We don't have enough information in the ipcache to resolve remote-node pod workloads.
I am not familiar enough with the code base as-is to know if there is another way to provide the information such that I can also compute it for remote. I can definitely dive more into it though. And suggestions appreciated!


Fix ID and description for twitter tokens

Description:
The ID and Description for the twitter tokens have been switched. This PR fixes that.
Checklist:

[x] Does your PR pass tests?
[ ] Have you written new tests for your changes?
[ ] Have you lint your code locally prior to submission?

🙇🏻 thanks for catching this @craigmarksmith!


I am locked out of my own gmail account as my 2 step verification does not work and my phone is in India and not in USA and is in roaming but I have visa mails that I am receiving on my account and I and stuck without access to my mails how can I resolve this issue?

Please help me get rid of this 2 step verification I am unable to get visa updates on my mail and I am messed up in India now

This is not Google support.


dropdowns (enums) no longer work on Chrome OSX in some circumstances

I don't know when this started but I've been seeing it around the net in various demos
Here's a simple repo
<!doctype html>
<html>
  <body>
  </body>
</html>
<script src="dat.gui.min.js"></script>
<script>
var guiWindow = function () {
    this.texture = "dotted";
};

function drawGui() {
    guiElement = new guiWindow();
    var a = new dat.GUI();

    var f3 = a.addFolder('General');
    f3.add(guiElement,"texture",["plane", "dotted","lined"]).listen().onChange(function (b) {});
    f3.open();
};
drawGui();
</script>

Run it, click the "dotted" value. Chrome goes crazy
Here's a live version
Here's a gif showing the issue. I only click one time and chrome goes into this fit bogging down the entire OS

PS: ignore the colors changing. That seems to have something to do with my gif compressor. Also trying it in Chrome Canary with no extensions it still happens. Other friends have confirmed this issue on their Macs. Happens in Chrome OSX but not Chrome Windows nor Safari OSX
Strangely the ones on the dat.gui demo site work but the code above seems like a pretty minimal repo.

Thanks for the bug and example. I've made a fix in ca19030


Add XAutoClaim command

Closes https://github.com/go-redis/redis/issues/1686
Redis docs: https://redis.io/commands/xautoclaim
I don't think ID is the right name for the cursor returned by Redis, but not sure what would be better. https://github.com/go-redis/redis/pull/1780/files#diff-4d7d1923693fc5ce892add2ea2907a744e77ea0b50c1939ccc5067cb48a466a3R1505

@monkey92t The assertion that failed passes locally. Are there any common race conditions you run into?
https://github.com/go-redis/redis/pull/1780/checks?check_run_id=2733660581#step:5:1586

@ericmillin Yes, there may be an unexpected data race, you can ignore it for now.

But the current workflow error is not accidental, it needs to be resolved
      Expected
          <string>: 2-0
      to equal
          <string>: 3-0

      /home/runner/work/redis/redis/commands_test.go:4399

@ericmillin  it should be 2-0
1) 1) "stream"
   2) 1) 1) "1-0"
         2) 1) "uno"
            2) "un"
      2) 1) "2-0"
         2) 1) "dos"
            2) "deux"
      3) 1) "3-0"
         2) 1) "tres"
            2) "troix"
<IP_ADDRESS>:6379> XAutoClaim stream group consumer 0 - count 2
1) "2-0"
2) 1) 1) "1-0"
      2) 1) "uno"
         2) "un"
   2) 1) "2-0"
      2) 1) "dos"
         2) "deux"
<IP_ADDRESS>:6379>

@monkey92t  Do you have any opinions on this: https://github.com/go-redis/redis/pull/1780/files#diff-4d7d1923693fc5ce892add2ea2907a744e77ea0b50c1939ccc5067cb48a466a3R1556-R1559
Seems to match the Redis semantics, but looks a bit confusing. I'm fairly new to Redis, though, hence my hesitation.

I made a little modification. What do you think of the new return value method? Any better suggestions?

@monkey92t That looks great and had crossed my mind. If you're good with it, let's merge it.
Thanks for all your help :)

OK, thanks


Custom Query String Parsing docs in V2 upgrade guide

I may have run into a small documentation error regarding custom query string parsing in V2:
https://github.com/rackt/react-router/blob/master/upgrade-guides/v2.0.0.md#custom-query-string-parsing
I think instead of what's listed there for V2 we actually need to use a custom history like in https://github.com/rackt/react-router/blob/master/upgrade-guides/v2.0.0.md#using-custom-histories.
So, changing this:
// v1.x
<Router
  parseQueryString={parse}
  stringifyQueryString={stringify}
/>

// v2.0.0
import { useRouterHistory, browserHistory } from 'react-router'

const createAppHistory = useRouterHistory(browserHistory)

const appHistory = createAppHistory({
  parseQueryString: parse,
  stringifyQueryString: stringify
})

<Router history={appHistory}/>

to
// v1.x
<Router
  parseQueryString={parse}
  stringifyQueryString={stringify}
/>

// v2.0.0
import { useRouterHistory } from 'react-router'
import createBrowserHistory from 'history/lib/createBrowserHistory'

const createAppHistory = useRouterHistory(createBrowserHistory)

const appHistory = createAppHistory({
  parseQueryString: parse,
  stringifyQueryString: stringify
})

<Router history={appHistory}/>

I'm happy to make a PR with the changes, but I just wanted to make sure I'm not off base here.
Thanks for all of your work!

You're exactly right. Nice catch.


jsrsasign: init at 10.8.6

Description of changes
Adds the NPM package jsrsasign to nodePackages.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

See https://github.com/NixOS/nixpkgs/pull/217084

The problem with packaging jsrsasign using buildNpmPackage is that there does not seem to be a package-lock.json file in the Github repo (https://github.com/kjur/jsrsasign). If you have any ideas or suggestions on how I could possibly package this, I'd love to hear them @linsui

It seems it has no deps so you don't even need buildNpmPackage.

I'll look into it & close this for now, thanks ^^


Refactor shared classpath for YaClientApp and devmode

Change-Id: If4737104b9d06b788a6d98133bac0a916f484b2e

This is actually 2 fixes in one. First, it refactors the shared components of the YaClientApp and devmode target classpaths into a single shared classpath. Second, it fixes a typo in one of the JavaScript files that causes the compiled superdevmode code to throw an error due to JS strict mode.


Romania is not showing cookie law

Hello,
I know you guys had some issues opened about Romania not showing cookie law, and from what I've seen you guys updated it in 2017 or 2018? Problem is that with "location: true", and with a Romanian IP, I don't see the cookie bar, and I should see it since it's an EU country.
Any ideas why it's not showing up?
Thanks,
Stephen

Hi, I can't help with your problem but could you let me know how you even setup geo location? I have this code

But I'm not sure how to setup geo targetting? Thanks

Hi, I have it like this:
`

`
Hope this helps.
Still, can someone maybe answer my OP question?

I think you have to host js on your server and Put 'RO' in the list manually.

I've tried it it works!

Hi, I have it like this:

Hope this helps.
Still, can someone maybe answer my OP question?

Great, thanks!


feat: liquation event details

Proposed solution for #103
I have a proposed solution, but have been unable to test due to the issues I have been having running the kinsugi testnet.
Also would you guys want to see the USD values in this entity as well?

Hey @daniel-savu @bvotteler, Here is what I currently have. We can get usdt without calls outside of squid, but lmk if the issue is more that we do not what to trust usdt specifically and I can remove it. Also @bvotteler I used the helper functions that the grants team created as it made this much simpler, but lmk if you want me to use the excahangeRate objects like we discussed in discord.
type LoanLiquidation @entity {     id: ID!     amountRepaid: BigInt!     amountRepaidHuman: BigDecimal!     amountRepaidToken: Currency!     seizedCollateral: BigInt!     seizedCollateralHuman: BigDecimal!     seizedCollateralToken: Currency!     liquidationCostBtc: Float!     liquidationCostUsdt: Float!     timestamp: DateTime! }

Also @bvotteler I used the helper functions that the grants team created as it made this much simpler, but lmk if you want me to use the excahangeRate objects like we discussed in discord.

If it works, then there's no reason to postpone the PR for that improvement.
I do lean towards using the exchange rates, but I'm happy for you to add that as a new issue instead that we can work on at a later stage.

@jtfirek , just checking in regarding your initial comments:

but have been unable to test due to the issues I have been having running the kinsugi testnet

Have you been able to test drive the changes and do a few sanity checks (that the values make sense) since then?

Ugh no, I am still having trouble with the kinsugi testnet and it so odd because I can run all the other networks just fine, but I feel like the kinsugi testnet would be the only network with liquidations I could test against.

I am still unable to run the kinsugi testnet on my machine.

@bvotteler could we merge this in with the next squid release please?

@bvotteler could we merge this in with the next squid release please?

I'm not strictly opposed, but to be clear, there is some remaining risk because it hasn't run a full scan against testnet yet. (Speed issues haven't been resolved on testnet afaict.)

Ok, then let's @ns212 on those speed issues and let's rerun this against testnet.

For reference, this is the spot where we get the error mentioned above:
https://github.com/interlay/interbtc-squid/pull/104/files#diff-96180e02130ffdf0db84cb80b8acff3480e612bebb402793a480dc20d8bf89aaR574

Closing due to lack of permissions to modify. Replacing it with #128


doesn't work on older browser

Hi i have a problem on my vue app, that it doesn't work on older browser (the one that i tested is ios 9/safari 9). At first i got error saying "fetch not found globally", so i've added unfetch, but still doesn't work.
Here's the code of my setup:
import { ApolloClient } from 'apollo-client';
import { HttpLink } from 'apollo-link-http';
import { InMemoryCache } from 'apollo-cache-inmemory';
import fetch from 'unfetch'

export default new ApolloClient({
  // Provide the URL to the API server.
  link: new HttpLink({ uri: '/api/', fetch: fetch }),
  // Using a cache for blazingly
  // fast subsequent queries.
  cache: new InMemoryCache(),
  connectToDevTools: true
});

My vue index file:
import apolloClient from '../utils/graphql'
import VueApollo from 'vue-apollo'

Vue.use(VueApollo);
const apolloProvider = new VueApollo({
  defaultClient: apolloClient,
  defaultOptions: {
    $loadingKey: "loading"
  }
})

......

apollo: {
      currentUser: {
	query: GET_LOGGED_USER,
	skipQuery() {
	 return this.skipUser
        }
      }
}

When i test it on the devices it self, this api doesn't get called.
I'm using 3.0.0-beta.25

Sorry i found the issue, it's turns out i need to add the babel-polyfill to fix this.
Closing this issue.


Revert "Disable macOS CI for now (#314)"

This reverts commit 768af17f9619c07d76dfdb8aecc1419329ee44c2.
Signed-off-by: Zhao Lufan<EMAIL_ADDRESS>

bors r+


func (e *Engine) WithRemote(r Remoter) should return error

Hi.
I think WithRemote should return an error. There are a few operations that can fail, mostly Listen() but also DRPCRegisterRemote.

#76 implements this.

This is now fixed. In the end actor.NewEngine() now returns an error if it can't active the supplied remote.


Error sending the contact-form

By sending the contact-form there occurs an error.
if (!$page) {
throw new RuntimeException('Display page not found. Please check the page exists.', 400);
}
The E-Mail is sending successfully. I think it is just the "Thank you" Message.

It seems there is a problem with your page's configuration, not a problem from the theme. You check to see if you forgot to create the Thank You page.

It seems there is a problem with your page's configuration, not a problem from the theme. You check to see if you forgot to create the Thank You page.

Is it possible to show a alert or replace the form into a message, instead of showing a new page. The theme is a singlepage so it would be great to keep everything on that side.

The theme doesn't support this at the moment. If you want to achieve this you need to override the theme's templates. Here is the instruction for how to submit Form plugin's form with AJAX: https://learn.getgrav.org/17/forms/forms/how-to-ajax-submission.


Vocabulary used for Pre-Trained BERT Model

Hi, I wanted to ask if it was possible to have information on the vocabulary you used for the pre-training of the Transformer-based model (ProteinBertModel).
I learned from your paper that you used the Pfam dataset and the masked-token technique, but I did not understand if you also use the IUPAC vocabulary or another (possibly larger) for the Transformer pre-training.
Thanks for your help.

It is the IIUPAC vocab defined in tape/tokenizers.py.


Fixed #30553 -- Moved (which is the default) from True to False.

Fixed misleading logging documentation about disable_existing_loggers default value
Moved  (which is the default)  from True to False in the paragraph.

"If the disable_existing_loggers key in the LOGGING dictConfig is set to True then all loggers from the default configuration will be disabled. Disabled loggers are not the same as removed; the logger will still exist, but will silently discard anything logged to it, not even propagating entries to a parent logger. Thus you should be very careful using 'disable_existing_loggers': True; it’s probably not what you want. Instead, you can set disable_existing_loggers to False (which is the default) and redefine some or all of the default loggers; or you can set LOGGING_CONFIG to None and handle logging config yourself."

https://code.djangoproject.com/ticket/30553

Made the suggested changes.


Implement v3 OAuth login method

Will also need to implement /session/refresh-token

There is a need for this to be implemented now. If you are a user of IG L2 Dealer, you must have your ISA or Share Dealing account specified as the default account. However, that would mean starting up an API session would fail with v1 or v2 , with the dreaded error.public-api.failure.stockbroking-not-supported error. The v3 OAuth /session endpoint works however. Once logged in you have to switch to your spread bet or CFD account. I'll have a crack at it, may need some help testing

@bug-or-feature Hello, thanks for all your great work. I am quite a beginner but would be happy to help you to test V3 OAuth login, if needed :)

There is a need for this to be implemented now. If you are a user of IG L2 Dealer, you must have your ISA or Share Dealing account specified as the default account. However, that would mean starting up an API session would fail with v1 or v2 , with the dreaded error.public-api.failure.stockbroking-not-supported error. The v3 OAuth /session endpoint works however. Once logged in you have to switch to your spread bet or CFD account. I'll have a crack at it, may need some help testing

I'll be happy to help testing also

Completed by release 0.0.10


'Error: ENOENT: no such file or directory' for mupc.json

Hi, Thanks for maintaining this repo. I'm running into this error after installing the latest version. I'm running node v8.9.0. Any ideas? Thanks in advance.
> mupc init                                                                                [103/4977]

Meteor Up: Production Quality Meteor Deployments
------------------------------------------------

fs.js:646
  return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
                 ^

Error: ENOENT: no such file or directory, open '/Users/chip/.nvm/versions/node/v8.9.0/lib/node_modules/mupc/example/mupc.json'
    at Object.fs.openSync (fs.js:646:18)
    at Object.fs.readFileSync (fs.js:551:33)
    at copyFile (/Users/chip/.nvm/versions/node/v8.9.0/lib/node_modules/mupc/lib/actions.js:233:22)
    at Function.Actions.init (/Users/chip/.nvm/versions/node/v8.9.0/lib/node_modules/mupc/lib/actions$
js:227:3)
    at Object.<anonymous> (/Users/chip/.nvm/versions/node/v8.9.0/lib/node_modules/mupc/bin/mupc:16:19)
    at Module._compile (module.js:635:30)
    at Object.Module._extensions..js (module.js:646:10)
    at Module.load (module.js:554:32)
    at tryModuleLoad (module.js:497:12)
    at Function.Module._load (module.js:489:3)

Yes, install a node manager and go back to node version 4.8.*

I ended up copying the files manually from the repo to get it working. I then ran into #15, which required your suggestion. I'm all good now. Thanks for the prompt help!


[BUG] allure-results folder will not appear inside docker image after docker run the image with a container

Wonder why its so? I tried the same code outside of the docker, it will successfully generate the allure-results folder. Is there specific setup I missed need for executing jest inside docker?

never mind, its an issue with docker, where I used CMD instead of RUN which for some reason, won't trigger the allure report generation...


wasm-mutate: replace instructions with constant values

This is an idea/suggestion for wasm-mutate. I'm not sure if I'll get an opportunity to implement something like this myself, but I definitely derived some value from applying a similar technique manually in the past.
As far as I know wasm-mutate will not attempt to simplify operations with dependencies on other items to simpler operations that are dependency free. This sort of transformation can enable removal of some of the definitions that aren't relevant to the test case more readily.
More generally such a transformation can be applied to an arbitrary WASM operation. The mutator would replace an operation with N operands and M results with N drop and M *.const rng operations.
For some operations that take as operands and produce as result values of the same type, fewer operations could be used to maintain the operand stack in the correct shape. e.g. i32.add could be replaced with a plain drop. I'm not entirely sure if that'd produce better or worse results. Either way it would seem that e.g. an obvious conversion to (drop) (drop) (i32.const 42) could be reduced to (drop) as a separate pass too.

We have a bunch of such rules already defined here:
https://github.com/bytecodealliance/wasm-tools/blob/5380b16c523ac3b56235d3254522504c2efedd3b/crates/wasm-mutate/src/mutators/peephole/rules.rs#L405-L538
I think we can close this issue?

Aha, okay, so there is something that prevents these from triggering in at least in some of the cases I had encountered.
An example I have on hand today is (elem (global.get 0) func) which I suspect may be related to the fact that we only have a rule for i32 and i64 today. In the past when I did a similar transformation by hand, I think it involved floating point types. I'll look into adding rules for funcrefs, floats and others and see if reduction capabilities improve.

I submitted 2 PRs that improve on what I'm suggesting here: https://github.com/bytecodealliance/wasm-tools/pull/469 https://github.com/bytecodealliance/wasm-tools/pull/470 but I don't believe these + the preexisting peephole rules are actually fully implementing what I'm suggesting.
In particular AFAICT transformation such as (i32.add) => (drop drop i32.const.rand), (local.set 10) => (drop), (call ... (...)) => ((params * drop) (result * T.const.rand)) etc. are not currently implemented. While we could start adding peephole “instcombines” that do this, that would result in a huge number of rules!
Another problem I've encountered as I was working on #469 and #470 is that currently “reduction” is required to produce a strictly smaller WASM encoding. In this case most such transformations would fail to decrease (and in many cases increase) the encoded size of WASM, even if ultimately they are objectively less complex and introduce opportunities for a further reduction.

In particular AFAICT transformation such as (i32.add) => (drop drop i32.const.rand), (local.set 10) => (drop), (call ... (...)) => ((params * drop) (result * T.const.rand)) etc. are not currently implemented. While we could start adding peephole “instcombines” that do this, that would result in a huge number of rules!

The rules I linked to before will do the first replacement. They match and replace whole expression subtrees, not just single instructions.
I don't believe we have any rules for call but it shouldn't be too hard to add support for.

Another problem I've encountered as I was working on #469 and #470 is that currently “reduction” is required to produce a strictly smaller WASM encoding. In this case most such transformations would fail to decrease (and in many cases increase) the encoded size of WASM, even if ultimately they are objectively less complex and introduce opportunities for a further reduction.

This is a known weakness of wasm-shrink's reduction approach, but is deemed an acceptable trade off regardless.
The problem is that more accurate/powerful metrics of Wasm module complexity are both more complicated to define and implement and also are more expensive to compute. (E.g. a test case reducer for a source language might want to count tokens rather than byte size, to avoid considering replacing descriptive variable names with single letter names a valid reduction, but that requires O(n) tokenizing every test case rather than an O(1) read of its length.
That said, we could loosen the invariants a little bit here to allow reductions that produce an identical byte size, but then we would

have to be careful to only go "one way" with e.g. replacing global-relative data segment offsets with constant offsets
add a debug_assert! to wasm-shrink that asserts that if we do "reduce" to the same size, then the contents of the test cases are different.

I think we can close this now with https://github.com/bytecodealliance/wasm-tools/pull/523 and https://github.com/bytecodealliance/wasm-tools/pull/470 and some other PRs being merged.


Added the sound feature

Not with neas sounds, but thats still fine I guess

Rerequest when fixed

Thanks lol, sorry for all the requests. If I'm gonna be annoying, I'm gonna be polite about it


Unique index not working properly? Duplicate record, broken index after update query.

Since I am very new to Orientdb, I am not sure if this is a bug ot if I am missing something.
I noticed this while using the Orientjs client, so I converted all queries to plain SQL in order to run them in Studio.
Create classes etc. on an empty database:
CREATE CLASS test_master EXTENDS V;
CREATE PROPERTY test_master.name STRING;
ALTER PROPERTY test_master.name MANDATORY true;

CREATE CLASS test_slave EXTENDS V;
CREATE PROPERTY test_slave.master_link link test_master;
ALTER PROPERTY test_slave.master_link MANDATORY true;
ALTER PROPERTY test_slave.master_link READONLY true;
ALTER PROPERTY test_slave.master_link NOTNULL true;
CREATE PROPERTY test_slave.slave_number SHORT;
ALTER PROPERTY test_slave.slave_number MANDATORY true;

CREATE INDEX unique_master_slave ON test_slave (master_link, slave_number) UNIQUE;

Create Vertices:
CREATE VERTEX test_master SET name='Master 1';
CREATE VERTEX test_slave SET master_link=#12:0, slave_number=1;
CREATE VERTEX test_slave SET master_link=#12:0, slave_number=2;
CREATE VERTEX test_slave SET master_link=#12:0, slave_number=3;

Now when I update one record to a duplicate slave_number
UPDATE #13:1 SET slave_number=1;

it fails with the following error
com.orientechnologies.orient.core.storage.ORecordDuplicatedException: Cannot index record #13:1: found duplicated key 'OCompositeKey{keys=[#12:0, 1]}' in index 'unique_master_slave' previously assigned to the record #13:0 RID=#13:0 [ONetworkProtocolHttpDb]

The record however was updated to the new value.
SELECT FROM [#13:0, #13:1]

{
    "result": [
        {
            "@type": "d",
            "@rid": "#13:0",
            "@version": 1,
            "@class": "test_slave",
            "master_link": "#12:0",
            "slave_number": 1,
            "@fieldTypes": "master_link=x,slave_number=s"
        },
        {
            "@type": "d",
            "@rid": "#13:1",
            "@version": 2,
            "@class": "test_slave",
            "master_link": "#12:0",
            "slave_number": 1,
            "@fieldTypes": "master_link=x,slave_number=s"
        }
    ],
    "notification": "Query executed  in 0.056 sec. Returned 2 record(s)"
}

If I try to rebuild the index
REBUILD INDEX unique_master_slave

it fails
com.orientechnologies.orient.core.index.OIndexException: Error on rebuilding the index for clusters: [test_slave]
--> com.orientechnologies.orient.core.storage.ORecordDuplicatedException: Cannot index record #13:1: found duplicated key 'OCompositeKey{keys=[#12:0, 1]}' in index 'unique_master_slave' previously assigned to the record #13:0 RID=#13:0 [ONetworkProtocolHttpDb]

Should the update query not just fail with an error, without changing the record and breaking the index?

What release?

Latest 2.1.1 as well as 2.1.0 (GA)

written the relative test case and fixed, in both 2.1 and develop

Hi,
the issue seems not be solved completely. In my environment (OrientDB 2.1.2) only remote protocol works as expected, if I test against plocal or memory the documents still get updated on index exception.


Change task background color to red

This PR will change Task input background color to red

Incorrect background color.


Convert CI pipeline to MicroBuild template

To be compliant, all production pipelines need to switch to the 1ES pipeline template. The MicroBuild template is an extension of that template that simplifies the installation of MicroBuild plugins.
This converts our CI to that template. I verified the CI and compliance builds pass.

您好，您的邮件我已经收到，我一定会尽快回复的！谢谢！【此为自动回复邮件】Hello, I've received your email. I will reply to you as soon as possible! Thank you! 【This is an automatic reply email】


Add new change log entry for deprecating shared use of old accounting datatypes

Description
Deprecation of the shared use of bankAccount and bankTransaction datatypes for banking data connections.
Type of change
Please delete options that are not relevant.

[x] New document(s)/updating existing
[ ] Fixes
[ ] Styling
[ ] Bug fix (non-breaking change which fixes an issue)

Not ready to be published yet. Thanks

This is ready to be merged. Thanks


[zk-sdk] Add error types and zero ciphertext instruction

Problem
The proof data and instructions for the new elgamal program in zk-sdk are not added yet.
Summary of Changes
I added the zero-ciphertext instruction to start. Once this PR is merged after some feedback, then I think I can add the rest of the proof data in slightly larger chunk PRs.

7c0e380: I added the ProofGenerationError and ProofVerificationError. These types remain largely the same as in zk-token-sdk and in particular, wraps RangeProofError and SigmaProofError.
d701214: I just copied the zero_balance instruction data module from zk-token-sdk verbatim to make the diffs in the next commit easier to view.
8468880: I updated the zero_balance instruction data. The changes are essentially updating ZeroBalance... to ZeroCiphertext... and re-organizing the imports.
25c3c88: Added the VerifyZeroCiphertext instruction in the elgamal proof program.

Fixes #

Codecov Report
Attention: Patch coverage is 36.69065% with 88 lines in your changes are missing coverage. Please review.

Project coverage is 82.7%. Comparing base (9fc8d17) to head (bcca0d3).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #1453     +/-   ##
=========================================
- Coverage    82.7%    82.7%   -0.1%
=========================================
  Files         871      872      +1
  Lines      370333   370472    +139
=========================================
+ Hits       306536   306597     +61
- Misses      63797    63875     +78


proposal: withLocalStorage/withSessionStorage

Hi Angular Architects!
Here's my proposal for a custom signalStoreFeature to be included within this library. I wanted to make my own library, but it makes much more sense to join forces. If you're open for contributions/PRs from outside...?
The idea behind this web storage synchronization is:

per store (granular, not global like)
redux-independent
only signals are taken into account, not computeds.
private withWebStorage implementation serves:  withLocalStorage/withSessionStorage

withLocalStorage(key: string, {
  validatorFn: (obj) => void, // required validator will be less troublesome in the long run
  serializer?: (obj) => string, // defaults to JSON.stringify
  deserializer?: (string) => obj // defaults to JSON.parse
})

which does the following:

onInit -> deserialize the storage content if it exists, iterate over signal properties and set them
whenever an update happens on a signal, it's being serialized and dumped to web storage: withHooks / onInit / effect -> reads the state, serializes it and pushes. This establishes synchronization

Please let me know what you think. If the idea is ok, I'll work on the PR.

Hiho, sure thing. To me, the only question is about the validatorFn.: What should that do, and why do you think the user has to provide an implementation?

@rainerhahnekamp form my experience, working with any web storage is vulnerable to issues with versioning serialized data structures sooner or later. Example:

a project starts with structure A
within some time, it's extended/refactored to structure B
some users could be left with structure A because of reasons... (e.g. they used the app before the release, whatever)
we don't want unexpected errors to pop up.

Anyway I'm open to suggestions - what is your take - make it optional? Or a different approach (dunno, migrating)?

@markostanimirovic: as far as I know the NgRx team has plans to come up with such a feature in the future as well. could you please comment here, how concrete those plans are?
I guess @ducin doesn't want to spend his time implementing it and - all of a sudden - an official pops up.

@ducin yeah, I'd say to get started very quickly that we provide a validator function by default. We can still discuss what that one should do.
I also think that the use cases you mentioned along different strategies should definitely show up in the documentation.

oh definitely!
If there's plan to make an official one, I'd be also happy to help with it @markostanimirovic.

@ducin  correct sorry pass in the object withStorage(localStorage)  but for types use the storage interface.

Hey 👋
That's true, we're considering introducing this plugin to the core library, but it's still not finally confirmed. https://github.com/ngrx/platform/issues/3997
Here is the implementation of the withStorageSync feature from the initial SignalStore prototype: https://github.com/markostanimirovic/ngrx-signal-store-playground/blob/main/src/app/shared/storage-sync.feature.ts
Note: To make this feature SSR-friendly, the storage factory needs to be passed as an input argument instead of a storage:
export const CounterStore = signalStore(
  withState({ count: 0 }),
  withStorageSync({
    key: 'counter',
    // if not passed, `localStorage` is used as the default
    storage: () => sessionStorage,
  })
);

When I created ngrx/plattform#4180 a little over motivated at the end of 2023 @rainerhahnekamp offered me opening a PR for ngrx-toolkit which can be found here ngrx-toolkit#7.
If adding this functionality to ngrx-toolkit is still on hold while an official plugin is being discussed - I'd close my pr.

I actually think the PR should get merged it seems perfect

@rainerhahnekamp I'm not in a hurry as I'm using the implementation as provided via the PR in a private project already.
Even if a merge will become obsolete in the future due to an official plugin a discussion about the API could offer some value somewhere.
(Well not everyone's having such a big bang birthday party 😂)

With version 0.0.7, the sync storage extension is now available. @ducin, if you want to add or modify some of the features... we would be ready.

Closing this issue since no further comments for 10 days and the version of @bohoffi is already available.


feat(api-server): TLS, mTLS support

This is not really something that we expect to be used in
production due to the prevalence of load balancers and
reverse proxies in most production scale web application
deployments, but it is a must have for testing and also
for us to be able to claim that we take the secure by
default design principle seriously.
Signed-off-by: Peter Somogyvari<EMAIL_ADDRESS>

but I would like to implement this with external server such as nginx.

@sfuji822 This change does not restrict you from doing that in any way. It's all opt-in by way of configuration flags that can turn TLS/mTLS on and off.

FYI: Did a rebase to resolve newly arisen conflicts after the other PRs got merged (then force pushed to keep the branch/PR tidy)

LGTM.  We may fix later as another item in terms of the above discussion points.

OK. We may fix it later. I will create a new issue as action item to myself.

LGTM. We may fix later as another item in terms of the above discussion points.

Thank you. Let's continue that discussion in #220 then.


[main] VMR sync failing with error 401 - Unauthorized

Failing build (internal Microsoft link)
fail: A '401 - Unauthorized' status was returned for a HTTP request. We'll set the retries amount to 0. Error details: {"message":"Bad credentials","documentation_url":"[https://docs.github.com/graphql","status":"401"}](https://docs.github.com/graphql%22,%22status%22:%22401%22%7D)
fail: Non-continuable HTTP 401 error encountered while making request against URI 'https://api.github.com/graphql'
      System.Net.Http.HttpRequestException: Response status code does not indicate success: 401 (Unauthorized).
         at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
         at Microsoft.DotNet.DarcLib.HttpRequestManager.ExecuteAsync(Int32 retryCount)
fail: Pushing to the VMR failed.
      Failure to authenticate: Response status code does not indicate success: 401 (Unauthorized).

A newer pipeline run has passed. This is resolved.


Is this project still alive?

Is the project still alive? I can see the last commit is a year ago.

I dont think so.. Also the angular version used in it is older one.. must be obsolete now..


Add support for ModelName EOLIS WIREFREE io

Did you read the instructions?

[X] I have read the README, including the advanced debugging section.

Device information
I have a Eolis Wirefree windsensor ( https://www.somfy.nl/producten/1816084/eolis-wirefree-io) but the app does not show the windsensor. I hooked it up directly on the motors (only way it seems) that is working correctly, but I would love to get the sensor working in HA so I can automate more stuff based on wind speed. Is this possible?
Home Assistant log
Logs
Copy/paste any log here, between the starting and ending backticks. The first log line must start with "Added device (Device...".

Device commands
Commands logs
No log available

Additional information
No response

Hi, please enable the debug logging https://github.com/iMicknl/ha-tahoma/blob/master/README.md#enable-debug-logging
Restart and report the log related to this component.


Unicode characters encoded as HTML entities, breaking iOS preheader text

If I send an email with HTML

Hi Andris 😀

at some point, 😀 gets converted to the HTML entity 😀: what is sent is

Hi Andris 😀

But iOS mail will not render HTML entities in preheaders:

If I use a plugin to unconvert the HTML after resolution (back to
Hi Andris 😀
), then the preheader renders the emoji just fine. And, both desktop Gmail and iOS Mail appear able to render Unicode just fine without entity-encoding.
Do you know of any reason not to undo Unicode entity conversion? Why are characters currently converted and where does this conversion take place?

Do you use a relay of some kind that modifies messages (SendGrid, mailgun etc.)? If I send a message like this:
{
    …
    html: '
Hi Andris 😀
'
}

then the generated rfc822 source is just like expected:
Content-Transfer-Encoding: quoted-printable
Date: Wed, 15 Jul 2015 08:41:26 +0000
Message-Id<EMAIL_ADDRESS>MIME-Version: 1.0

Hi Andris =F0=9F=98=80

As you can see there is no conversion made, the smiley is transmitted as raw bytes, not as an HTML entity.

Thanks for the quick response @andris9, and apologies on my part for improperly diagnosing the bug—the conversion happens on our end before submitting the message to Nodemailer, due to the way that we process HTML for sending. Keep up the good work :)


Can't SSH into an instance

Hi- I'm setting up a Flynn cluster for a staging environment.
I wanted to add some swap to the (only) instance I'm using, since it's on a relatively small box (1gb).
Per the documentation:

If you need to access cloud instances provisioned with the installer, you can connect to them with SSH. The installer will generate an SSH keypair, and will install the public key on all hosts it provisions. The keys are stored in ~/.flynn/installer/keys. The default key is called flynn.

Trying to do this results in a prompt for a password:
→ ssh -i ~/.flynn/installer/keys/flynn ubuntu@[removed]
ubuntu@[removed]'s password:

What cloud provider are you using? DigitalOcean uses the root account and Azure the flynn account. #3526 has been merged to fix the docs but an update hasn't bene pushed yet.

Aha- I'm on DO. Root worked perfectly. Thanks!


Add CHANGELOG entries for work included in 0.6.0

adds CHANGELOG entries for various items that are to be included in
0.6.0

@jtfmumm @aturley can you review and make sure the items in this change are under the section you expect them to be and that the phrasing is correct for any of the entries which relate to work you made?


XMLLoadingSample - particles only render in OpenGL and not in DirectX

Whilst writing the XMLLoading sample, using the XML Particles XNA sample as a base, I ran in to an issue.
The Shader only currently works for GL platforms and not for DX (e.g. WindowsDX / UWP / Etc)
I suspect it's an issue with the shader somewhere but haven't found the cause yet.
if anyone can help, please dig in and look.

Check the order of the items in the input struct to the VS in the .fx ... it needs to directly match the VertexDeclaration used in the game code. :)

I may have spoke too soon ... ;-P
See:  http://community.monogame.net/t/solved-xna-3d-particles-sample-port-on-monogame-3-4/6301
It may have to do with the multiple usages (usage indices) for "Position" ...
At the end of the link, you see that they re-arranged the order of the struct elements AS WELL AS the usages of each.

Thanks for the assist @WPCrasher
Turned out it was all to do with the vertex formats, seems GL is very forgiving but DX has none of that rubbish :D
Updated shader and the VertexFormat to match and it's all good now.


Support POSTGRES_16

Hello!
We would like to use this module to spin up an alloydb with POSTGRES_16, which is possible on the console but is not an option in the module.
Thank you!

Sorry, closing this in favor of: https://github.com/hashicorp/terraform-provider-google/issues/19565


ToDo: Modularize Functions Within page-template.js

Create a sub-directory to store similar functionality
Decrease the overall size of page-template.js (mainly to improve readability & create leaner code)

Update commit 39894c6
Portion of the page-template.js file has been migrated to a new template-utils page. As of this commit, these modularized functions are operational. The rest of page-template's functions will be turned into it's respective components

Update Commit 6ff90fc
Each component has been moved into their own file and are now imported into the page-template file.
Every change has been tested and as of this moment is working fine.


[tests] Make tests less dependent on fs time resolution

'touch' can be insufficient to change stat info on low resolution file
systems.
rdar://problem/48946422

@swift-ci please smoke test


Support Field(env=...) is submodels

As per https://github.com/pydantic/pydantic/issues/4982

I think it can be fixed by https://github.com/pydantic/pydantic-settings/pull/22

Closing as the original issue transferred into this repo


Localization & Hungfire, appreciate it can be included V2 as well.

Localization & Hungfire, appreciate it can be included V2 as well.
supporting multiple languages and log transactions are highly required.

Hangfire is already added. I will give a lower prioritization for localization for now.

appreciate it very much, thank you.


INCR-4 upgrade click log to 0.3.2

Please provide a new release of edx-lint to be made once it is merged.

Thanks for the pull request, @pjcrouse! I've created OSPR-2825 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.
We can't start reviewing your pull request until you've submitted a     signed contributor agreement     or indicated your institutional affiliation. Please see the CONTRIBUTING     file for more information.

@pjcrouse  Thank you for the contribution! When you have a chance, could you please send in a signed Contributor Agreement mentioned in the earlier comment? You only need to do this once and we can then review all your future code contributions. Thanks

@pjcrouse  Have you had a chance to sign contributors agreement? You only need to do this once and we can then review all your future code contributions. Thanks

@pjcrouse We have not heard from you for a while, so I am closing this request. Should you decide to pursue this again, please feel free to re-open this PR.

@pjcrouse Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.


CSHARP-582: Linq: Add ALLOW FILTERING on scalar Count() method

Includes an additional test for #310.

Rebased with master

I will squash those test commits before merge.

Looks good to me!
I'll do a merge commit to acknowledge the 3 different commits from 3 different authors :)
Thanks @francischie !


Extracts correct mco image based on managed cluster's arch

What this PR does / why we need it:
Handles manifest listed images and extracts correct mco image based on the managed cluster's arch. This allows you to create HyperShift hosted control planes on other architectures other than AMD64.
Which issue(s) this PR fixes :
Fixes ARMOCP-441
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[x] This change includes unit tests.

https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_hypershift/1716/pull-ci-openshift-hypershift-main-verify/1577692737967230976 complains.

/retest

dropped some more questions, happy to lgtm other than https://github.com/openshift/hypershift/pull/1716#discussion_r991208224

dropped some more questions, happy to lgtm other than #1716 (comment)

I think I addressed everything. Let me know if I missed something or if you see something new.

Rerunning e2e-aws to see if the redhat-operators-catalog pod restart was a one time thing or not. The rest of TestCreateCluster completely passed.
=== FAIL:  TestCreateCluster/EnsureNoCrashingPods (0.05s)
    util.go:354: Container registry in pod redhat-operators-catalog-7dd68db7cf-4zr5m has a restartCount > 0 (1)
    --- FAIL: TestCreateCluster/EnsureNoCrashingPods (0.05s)

/test e2e-aws

Rerunning e2e-aws to see if the redhat-operators-catalog pod restart fails again. The rest of TestNodepoolMachineconfigGetsRolledout completely passed.
{Failed  === RUN   TestNodepoolMachineconfigGetsRolledout/EnsureNoCrashingPods
    util.go:354: Container registry in pod redhat-operators-catalog-558ddc6b75-t4pzs has a restartCount > 0 (1)
    --- FAIL: TestNodepoolMachineconfigGetsRolledout/EnsureNoCrashingPods (0.05s)
}

/test e2e-aws

/lgtm

/lgtm

/lgtm

Rerunning e2e-aws to see if the failure was just a one off/flake.
=== CONT  TestUpgradeControlPlane
    util.go:262:
        failed waiting for image rollout

/test e2e-aws


Add the possibility to add proxy when lauching container

I'm behind a corporate proxy and I've setted the proxy where i can.
But when I'm trying to use
import os

from langchain.embeddings import OpenAIEmbeddings

os.environ["OPENAI_API_KEY"] = "rassndosssms-sssstrissnsg"

embeddings = OpenAIEmbeddings(openai_api_base="http://localhost:8444/v1")
text = "Prem is an easy to use open source AI platform."
query_result = embeddings.embed_query(text)
doc_result = embeddings.embed_documents([text],1)

I'm receiving the error
Retrying langchain.embeddings.openai.embed_with_retry.<locals>._embed_with_retry in 4.0 seconds as it raised APIError: Invalid response object from API: 'Traceback (most recent call last):\n  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 174, in _new_conn\n    conn = connection.create_connection(\n  File "/usr/local/lib/python3.10/site-packages/urllib3/util/connection.py", line 72, in create_connection\n    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):\n  File "/usr/local/lib/python3.10/socket.py", line 955, in getaddrinfo\n    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):\nsocket.gaierror: [Errno -3] Temporary failure in name resolution\n\nDuring handling of the above exception, another exception occurred:\n\nTraceback (most recent call last):\n  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 714, in urlopen\n    httplib_response = self._make_request(\n  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 403, in _make_request\n    self._validate_conn(conn)\n  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 1053, in _validate_conn\n    conn.connect()\n  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 363, in connect\n    self.sock = conn = self._new_conn()\n  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 186, in _new_conn\n    raise NewConnectionError(\nurllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7f1db722e260>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\n\nDuring handling of the above exception, another exception occurred:\n\nTraceback (most recent call last):\n  File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 489, in send\n    resp = conn.urlopen(\n  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 798, in urlopen\n    retries = retries.increment(\n  File "/usr/local/lib/python3.10/site-packages/urllib3/util/retry.py", line 592, in increment\n    raise MaxRetryError(_pool, url, error or ResponseError(cause))\nurllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host=\'openaipublic.blob.core.windows.net\', port=443): Max retries exceeded with url: /encodings/cl100k_base.tiktoken (Caused by NewConnectionError(\'<urllib3.connection.HTTPSConnection object at 0x7f1db722e260>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\'))\n\nDuring handling of the above exception, another exception occurred:\n\nTraceback (most recent call last):\n  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 162, in __call__\n    await self.app(scope, receive, _send)\n  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/cors.py", line 84, in __call__\n    await self.app(scope, receive, send)\n  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 79, in __call__\n    raise exc\n  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 68, in __call__\n    await self.app(scope, receive, sender)\n  File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__\n    raise e\n  File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__\n    await self.app(scope, receive, send)\n  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 718, in __call__\n    await route.handle(scope, receive, send)\n  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 276, in handle\n    await self.app(scope, receive, send)\n  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 66, in app\n    response = await func(request)\n  File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 237, in app\n    raw_response = await run_endpoint_function(\n  File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 163, in run_endpoint_function\n    return await dependant.call(**values)\n  File "/usr/src/app/routes.py", line 61, in embeddings_openai\n    encoding = tiktoken.model.encoding_for_model("text-embedding-ada-002")\n  File "/usr/local/lib/python3.10/site-packages/tiktoken/model.py", line 75, in encoding_for_model\n    return get_encoding(encoding_name)\n  File "/usr/local/lib/python3.10/site-packages/tiktoken/registry.py", line 63, in get_encoding\n    enc = Encoding(**constructor())\n  File "/usr/local/lib/python3.10/site-packages/tiktoken_ext/openai_public.py", line 64, in cl100k_base\n    mergeable_ranks = load_tiktoken_bpe(\n  File "/usr/local/lib/python3.10/site-packages/tiktoken/load.py", line 116, in load_tiktoken_bpe\n    contents = read_file_cached(tiktoken_bpe_file)\n  File "/usr/local/lib/python3.10/site-packages/tiktoken/load.py", line 48, in read_file_cached\n    contents = read_file(blobpath)\n  File "/usr/local/lib/python3.10/site-packages/tiktoken/load.py", line 24, in read_file\n    resp = requests.get(blobpath)\n  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 73, in get\n    return request("get", url, params=params, **kwargs)\n  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 59, in request\n    return session.request(method=method, url=url, **kwargs)\n  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 587, in request\n    resp = self.send(prep, **send_kwargs)\n  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 701, in send\n    r = adapter.send(request, **kwargs)\n  File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 565, in send\n    raise ConnectionError(e, request=request)\nrequests.exceptions.ConnectionError: HTTPSConnectionPool(host=\'openaipublic.blob.core.windows.net\', port=443): Max retries exceeded with url: /encodings/cl100k_base.tiktoken (Caused by NewConnectionError(\'<urllib3.connection.HTTPSConnection object at 0x7f1db722e260>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\'))\n' (HTTP response code was 500).

This error is generated by ghcr.io/premai-io/embeddings-all-minilm-l6-v2-cpu:1.0.2 which is started without proxy configuration and is not able to go online.
Can a proxy be added in setting and be concatenated as environment variable when launchinh these dockers?

Everything runs locally both Prem Daemon and Services, so weird you having problems, and you should not require a proxy as you calling all in localhost. Am I wrong?

Ciao @tiero !
At some point into the code, there is a call at openaipublic.blob.core.windows.net
File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 565, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host=\'openaipublic.blob.core.windows.net\', port=443): Max retries exceeded with url: /encodings/cl100k_base.tiktoken (Caused by NewConnectionError(\'<urllib3.connection.HTTPSConnection object at 0x7f1db722e260>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\'))
' (HTTP response code was 500).

this is the full stack trace:
Retrying langchain.embeddings.openai.embed_with_retry.<locals>._embed_with_retry in 4.0 seconds as it raised APIError: Invalid response object from API: 'Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 174, in _new_conn
    conn = connection.create_connection(
  File "/usr/local/lib/python3.10/site-packages/urllib3/util/connection.py", line 72, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "/usr/local/lib/python3.10/socket.py", line 955, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -3] Temporary failure in name resolution

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 714, in urlopen
    httplib_response = self._make_request(
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 403, in _make_request
    self._validate_conn(conn)
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 1053, in _validate_conn
    conn.connect()
  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 363, in connect
    self.sock = conn = self._new_conn()
  File "/usr/local/lib/python3.10/site-packages/urllib3/connection.py", line 186, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7f1db722e260>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 489, in send
    resp = conn.urlopen(
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 798, in urlopen
    retries = retries.increment(
  File "/usr/local/lib/python3.10/site-packages/urllib3/util/retry.py", line 592, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host=\'openaipublic.blob.core.windows.net\', port=443): Max retries exceeded with url: /encodings/cl100k_base.tiktoken (Caused by NewConnectionError(\'<urllib3.connection.HTTPSConnection object at 0x7f1db722e260>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 162, in __call__
    await self.app(scope, receive, _send)
  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/cors.py", line 84, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 79, in __call__
    raise exc
  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 68, in __call__
    await self.app(scope, receive, sender)
  File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__
    raise e
  File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 718, in __call__
    await route.handle(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 276, in handle
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 66, in app
    response = await func(request)
  File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 237, in app
    raw_response = await run_endpoint_function(
  File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 163, in run_endpoint_function
    return await dependant.call(**values)
  File "/usr/src/app/routes.py", line 61, in embeddings_openai
    encoding = tiktoken.model.encoding_for_model("text-embedding-ada-002")
  File "/usr/local/lib/python3.10/site-packages/tiktoken/model.py", line 75, in encoding_for_model
    return get_encoding(encoding_name)
  File "/usr/local/lib/python3.10/site-packages/tiktoken/registry.py", line 63, in get_encoding
    enc = Encoding(**constructor())
  File "/usr/local/lib/python3.10/site-packages/tiktoken_ext/openai_public.py", line 64, in cl100k_base
    mergeable_ranks = load_tiktoken_bpe(
  File "/usr/local/lib/python3.10/site-packages/tiktoken/load.py", line 116, in load_tiktoken_bpe
    contents = read_file_cached(tiktoken_bpe_file)
  File "/usr/local/lib/python3.10/site-packages/tiktoken/load.py", line 48, in read_file_cached
    contents = read_file(blobpath)
  File "/usr/local/lib/python3.10/site-packages/tiktoken/load.py", line 24, in read_file
    resp = requests.get(blobpath)
  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 73, in get
    return request("get", url, params=params, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 587, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 701, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 565, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host=\'openaipublic.blob.core.windows.net\', port=443): Max retries exceeded with url: /encodings/cl100k_base.tiktoken (Caused by NewConnectionError(\'<urllib3.connection.HTTPSConnection object at 0x7f1db722e260>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\'))
' (HTTP response code was 500).

@filopedraz I'm behind a corporate proxy without direct internet connection. Yep, the tiktoken library fetches some meta info. The workaround is download the source code of ghcr.io/premai-io/embeddings-all-minilm-l6-v2-cpu:1.0.2 set python environment proxy and rebuild.

Yep, if I understood correctly, you will need to re-run the containers.

Yes, to make the workaround works it's needed to rebuild prem-daemon with the above changes, relaunch docker-compose and then relaunch containers (in this case ghcr.io/premai-io/embeddings-all-minilm-l6-v2-cpu:1.0.2). Now it finds the proxy and is able to make the call to openaipublic.blob.core.windows.net .
I don't know if it could be useful a feature for configuring proxy conf or if this issue is enough.
I've solved my problems with the above workaround

Yes, good point, thanks. What we can do is support those ENV VAR in the official prem daemon image, so you can pass any value at runtime.


Add support for WORKSPACE.bazel

I developed these changes while attempting to use rules_bazel_integration_test in rules_rust, which names all workspace files WORKSPACE.bazel, rather than WORKSPACE.
Fixes #97

@jfirebaugh Thanks for the PR!
To be honest, I am not sure why I originally wrote the utilities to use BUILD_WORKING_DIRECTORY and not BUILD_WORKSPACE_DIRECTORY.  Good catch.
To fix the failing tests,

I adde parenthesis to the logical OR when looking for WORKSPACE or WORKSPACE.bazel. Without the parenthesis, the -print0 does not execute when WORKSPACE is found.
Added the appropriate export BUILD_WORKSPACE_DIRECTORY="..." to the tests.

To demonstrate support for WORKSPACE.bazel, I updated the PR to rename the workspace file in the custom_test_runner example.
I removed an obsolete Xcode selection, as I don't think that it is relevant with the latest Xcode and I did not have 13.1 still on my laptop. 🙂
Are you OK with these changes?

Looks good to me, thanks for the fixes!


HOCS-6588 : COMP - Directorate selection doesn't match the value disp…

JIRA Link : https://collaboration.homeoffice.gov.uk/jira/browse/HOCS-6588

Hi @sameerinfo2003 ,
I've tested this locally and it doesn't seem to have the desired effect unless I have missed something. When I have tested it, the original EUSSFP text can still be seen in the summary and accordion, it isn't being replaced, so there is potentially extra changes required here to alter the display value.
Could you have a look to try to replicate please? Happy to jump on a call if needed to show what I see,
Thanks,
Rob

@sameerinfo2003 It looks like hocs-casework is still configured to show directorate options based on OwningCSU, which was removed here: https://collaboration.homeoffice.gov.uk/jira/browse/HOCS-6542. The hocs-casework props should match the hocs-workflow props, so the same change will need to be made in hocs-casework.

@sameerinfo2003 It looks like hocs-casework is still configured to show directorate options based on OwningCSU, which was removed here: https://collaboration.homeoffice.gov.uk/jira/browse/HOCS-6542. The hocs-casework props should match the hocs-workflow props, so the same change will need to be made in hocs-casework.

They shouldn't always necessarily match, I don't know the context of the ticket, but as some cases have old values we need to support that within the accordion and summary.

@sameerinfo2003 It looks like hocs-casework is still configured to show directorate options based on OwningCSU, which was removed here: https://collaboration.homeoffice.gov.uk/jira/browse/HOCS-6542. The hocs-casework props should match the hocs-workflow props, so the same change will need to be made in hocs-casework.

They shouldn't always necessarily match, I don't know the context of the ticket, but as some cases have old values we need to support that within the accordion and summary.

Good point. In this case the mappings are still conditional on the OwningCSU value that has been removed. The hocs-casework props need to be changed to a single mapping of values to labels matching the current hocs-workflow props.


IPySheets test seems to fail consistently on raw non conda 3.9

Get this error.
Render IPySheets:1 | Widget did not render or Hello, World, 42.000 not in html =

Haven't looked further. Maybe ipysheets doesn't work without jupyter on 3.9.

Hmm, haven't seen this failing in my PRs. I'd say its flaky, but not consitent, don't see this failing here https://github.com/microsoft/vscode-jupyter/runs/6296118628?check_suite_focus=true

@rchiodo please could you point to the failing PR, i'd like to see the logs.

https://github.com/microsoft/vscode-jupyter/runs/6297915635?check_suite_focus=true