Tappable Test

Related to #15 -- I was writing a unit test for another component that uses react-tappable, with the following behavior (simplified to isolate react-tappable):
// __tests__/Tappable-test.js

"use strict";

jest.dontMock('react-tappable');
describe('Button', function() {
  it('fires callback when tapped', function() {
    var React = require('react/addons');
    var Tappable = require('react-tappable');
    var TestUtils = React.addons.TestUtils;

    // mock callback
    var callback = jest.genMockFunction();

    // Render a button with the mock callback
    var button = TestUtils.renderIntoDocument(
      <Tappable onTap={callback}></Tappable>
    );

    // Get the node
    var node = TestUtils.findRenderedDOMComponentWithTag(button, 'span').getDOMNode();

    // Fire the touch events
    TestUtils.Simulate.touchStart(node, TestUtils.nativeTouchData(0, 0));
    TestUtils.Simulate.touchEnd(node, TestUtils.nativeTouchData(0, 0));

    expect(callback).toBeCalled();

  });
});

The test case fails with the following trace:
  - Expected Function to be called.
        at Spec.<anonymous> (/vagrant/modules/javascript/www/components/Button/__tests__/Tappable-test.js:23:22)
        at Timer.listOnTimeout [as ontimeout] (timers.js:112:15)

What am I missing here about why the callback does not fire, specific to touch events? A similar test case with a mouseDown and mouseUp simulated events does pass:
// __tests__/Tappable-click-test.js

"use strict";

jest.dontMock('react-tappable');
describe('Button', function() {
  it('fires callback when tapped', function() {
    var React = require('react/addons');
    var Tappable = require('react-tappable');
    var TestUtils = React.addons.TestUtils;

    // mock callback
    var callback = jest.genMockFunction();

    // Render a button with the mock callback
    var button = TestUtils.renderIntoDocument(
      <Tappable onTap={callback}></Tappable>
    );

    // Get the node
    var node = TestUtils.findRenderedDOMComponentWithTag(button, 'span').getDOMNode();

    // Fire the touch events
    TestUtils.Simulate.mouseDown(node);
    TestUtils.Simulate.mouseUp(node);

    expect(callback).toBeCalled();

  });
});

(I'd be happy to contribute tests in a PR, but want to make sure I understand intended behavior with touch events)

Had a problem that somehow looks like this, I had a default prop value that was a funcion and tried to pass another function, but it wasn't replace. I guess renderIntoDocument doesn't pass functions as props, just serializable values.

@irvinebroque Thanks it ( TestUtils.Simulate.mouseDown(node), ... ) works :)


Converted WorkflowExecutionAlreadyStartedFailure handling to use RPC status

We ran into a condition where the error message changed slightly and we were unable to extract the Run ID. This changes moves to start using the RPC error in a similar way to the Go SDK.

@antstorm this should be ready for another review


Necromancer / too much mob

i rly dont nkow that is intended or what... but i guess there are too many mobs... and the config file doesn't seems like to change anything

wat


Team page refactor: make data driven, remove not used css classes.

This PR:

updates template for team page to use variables from controller to avoid duplicate markup and make code more readable;
fixes the bug with tooltip rendering (it was not centered when resized + weird behavior for the last person in a row); this PR makes tooltip render correctly on all screen sizes;
removed not used image for google groups;
delete empty guides.js.coffee file;
cleans up team page css + template to remove not used classes;
mover testimonials related images into dedicated folder for better project organization.

@jadefaerie this PR is mostly refactor to make code more readable and to remove not used files from project. It also fixes the bug with the rendering of the tooltip content for the team page on hover. I would love to merge this in before we deploy to production. Let me know if you have any questions!


Vulcan support

Hi,
I set up steam container to play but it complains for lack of vulcan support (not necessary to work, but seen it in log).
Readme also doesn't say anything about it so I guess it's missing.
I'm using this container, it uses --gpu and --hostdisplay

The image installs the mesa drivers with:
RUN apt-get install -y libgl1-mesa-dri:i386 libgl1-mesa-glx:i386 libc6:i386

It might help to install mesa-vulkan-drivers, too.
In my own images I just install mesa-utils mesa-utils-extra to get all free mesa drivers. Though, I am not sure yet if the vulkan driver package is  in this dependency chain, too.
For some video decoding support it helps to install libxv1 va-driver-all, too.
So I recommend to replace the line above with:
RUN apt-get install -y mesa-utils:i386 mesa-utils-extra:i386 libxv1:i386 va-driver-all:i386

If the vulkan driver is still missing, add mesa-vulkan-drivers:i386.

So basicly - x11docker itself has vulcan support?

I am not aware that there would be something special about Vulkan that x11docker would have to support.
With the open MESA drivers in general  (AMD, intel etc.)it is enough to have the drivers on host and in image and to share the files in /dev/dri. That's all.

@JuniorJPDJ  Did this work for you?
Is installing mesa-utils mesa-utils-extra enough to get the vulkan driver?

I opened issue in upstream container, I'll let you know if it worked.
I don't have time to debug it at my own ;/

Closing for now. Feel free to comment again if something comes up.


Add a method AssertRead to insert a key-version to the TX read-set

Some times there is a slow changing key-value that many following TXs depend upon. Instead of reading it on every TX, we can read it once, and then inject the key and version into the read set of future TXs, just using the read value. This saves a read on every future TX, saves RTT to the server.

Method signature:
AssertKey(db, key, version) error
To add to the tx context


Need documentation

Envoy is taking a dependency on protos from this repo. That means Envoy docs need references to the protos here in a better form than raw source code.

cc @markdroth @phlax
Do you have any thoughts on how to publish the protos here? Do we have a website? I think the tooling in Envoy is probably good enough.

I think initially the way I would do this is:

Have Envoy be able to run protodoc, etc. on this repo.
Allow protodoc annotations in this repo.

This will allow this repo to look like any other protos to Envoy which should make it "just work."
If/when other clients want to generate docs from here we can introduce the concept of client specific doc tags.

and linting for unused imports!

@phlax can you help with 1)? Maybe some pointer how to make envoy CI produce HTML for protos here?

Is there anything left to do here, or can this be closed?

I think the initial work is done. We probably need some CI to validate protodoc annotations, but that can be a separate issue.


mDNS stopped working in WSL2 after Windows 11 Pro update

Windows Version
Microsoft Windows [Version 10.0.22635.3061]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-1
Distro Version
Ubuntu 23.10
Other Software
No response
Repro Steps
ping okno-t320.local does not find okno-t320
Expected Behavior
jacek@okno-galaxy:~$ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=63 time=7.53 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=63 time=5.69 ms
64 bytes from <IP_ADDRESS>: icmp_seq=3 ttl=63 time=6.20 ms
^C
--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 2005ms
rtt min/avg/max/mdev = 5.689/6.472/7.533/0.777 ms
Actual Behavior
jacek@okno-galaxy:~$ ping okno-t320.local
ping: okno-t320.local: Name or service not known
jacek@okno-galaxy:~$
Diagnostic Logs
No response

I forgot to add that it happened on 2 windows computers after the same update.

When running from Windows Power Shell ping works ok. From WSL2 no more.

Are you sure this has anything to do with windows update? What do you have in /etc/nsswitch.conf?

Thank you @jtgoral. Can you try to put the following in %userprofile%/.wslconfig, then run: "wsl.exe --shutdown" to apply them:
[wsl2]
dnsTunneling=false

And see if that solves the issue ?

Yes, Adding the line to .wslconfig solved the problem:
jacek@okno-galaxy:/mnt/c/Users/jacek/public_html/beer/beersmith2$ ping okno-t320.local
PING okno-t320.local (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=1 ttl=63 time=14.9 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=2 ttl=63 time=16.4 ms

Yes, it was caused by a Windows update. All tries to use avahi connections stopped working right after the update on two Windows boxes.. Computers I connect from WSL  (*.local) are FreeBSD and EndeavourOS Linux.
Modifying .wslconfig as suggested fixed the problem and my git repository, ssh  and VNC servers are available again when using .local instead of IP addresses.

i also had this issue on the most recent version of Windows 11 Pro (as of 2024-06-14). the suggested fix worked for me as well.

I can confirm that the workaround from @OneBlue worked for me too.

I can also confirm that the upgrade broke this for me and what @OneBlue suggested fixed the issue.
I tried adding that configuration value inside the distribution's /etc/wsl.conf file and it didn't work.

Recently, I haven't been able to resolve .local domain names from WSL2, and I discovered this issue.
I can also verify the workaround fixed the issue.
But one more additional modification was necessary for me and I don't understand what happened with the recent update.
# /etc/nsswitch.conf
-hosts:          files mdns4_minimal [NOTFOUND=return] dns
+hosts:          files dns mdns4_minimal [NOTFOUND=return]

I used to be able to resolve names in the .local domain, but recently I encountered this issue.
For my case, the following fixes were necessary:
# the workaround from @OneBlue
[wsl2]
dnsTunneling=false

# /etc/nsswitch.conf
-hosts:          files mdns4_minimal [NOTFOUND=return] dns
+hosts:          files dns mdns4_minimal

(I am using WSL2 in NAT mode not Mirrored mode.)

I just re-discovered this problem and fix of disabling DNS tunneling. It seems like a pretty significant regression in WSL2, for .local domains to just stop working in the default configuration. Is this a temporary bug in the DNS tunneling implementation, will it eventually start working with mDNS again?

@CatalinFetoiu & @keith-horton - could you please take a look?

thanks for reporting the issue
we have update the docs to include details about how to resolve .local names in WSL. Please refer to https://learn.microsoft.com/en-us/windows/wsl/troubleshooting#resolve-local-names-in-wsl
let us know if you have any questions

and netgroup: nis, for old time's sake

@NelsonMinar if you use mirrored networking mode you can also keep DNS tunneling enabled
We only recommend turning off DNS tunneling if you are using NAT networking mode and want to resolve .local names

I will say that the description of dnsTunneling on https://learn.microsoft.com/en-us/windows/wsl/wsl-config#wslconfig leaves something to be desired. It's basically impossible to tell from the description what it does, such as "make mDNS resolution work", but ideally a more detailed description given that the audience for wsl2 docs likely skews both technically-savvy and curious

Changes how DNS requests are proxied from WSL to Windows

I've executed a wsl --shutdown after changing %UserProfile/.wslconfig and still no dice:

@shaver please make sure to follow the steps from https://learn.microsoft.com/en-us/windows/wsl/troubleshooting#resolve-local-names-in-wsl, section "networkingMode set to Mirrored"
(make sure to update WSL to the latest pre-release build (2.3.17) and do the configuration mentioned in the docs for enabling mdns in Linux)

thank you, I didn't realize I still had .14 installed!
so, it works now, but there's about a 10 second lag in resolution for either IPv6 or IPv4

@CatalinFetoiu should I open a separate bug for the lag? Happy to do so!

@shaver thanks for testing the fix and following up
yes, please feel free to open a separate issue for the delay you observed

I'm still unable to get this to work, despite following the instructions on the Microsoft docs site.

@s73v3r thanks for following up
can you please share the output of sudo strace -ff ping ender6printer.local?
please also collect networking logs using https://github.com/microsoft/WSL/blob/master/diagnostics/collect-networking-logs.ps1, using instructions at https://github.com/microsoft/WSL/blob/master/CONTRIBUTING.md#collect-wsl-logs-for-networking-issues

i ended up with custom code to find ip address. because i have wifi and wifi hot-spot both connected to another pc.  i tried mirror mode but i cannot make a ping to global network again. my laptop is window11 home.
sh IP=$(echo \"require('dnssd/lib/resolve').resolve4('xxxx.local', {interface: 'WLAN'}).then(console.log)\" | '/mnt/c/Program Files/nodejs/node.exe')

This solution fixed it for me.

I have the following in my .wslconfig file
[wsl2]
dnsTunneling=false

I even restarted my computer to be sure but I still have issues pinging .local addresses from WSL (it works from powershell).
Any other ideas I can try?
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.64
MSRDC version: 1.2.5326
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.26100.1-240331-1435.ge-release
Windows version: 10.0.22631.4037

@YAmikep in order to have IPv6 connectivity in WSL you need to enable mirrored mode - you need to add "networkingMode=mirrored" in your wslconfig file
if you do that, you no longer need to disable DNS tunneling in order to resolve .local names
but you need to follow the steps documented at https://learn.microsoft.com/en-us/windows/wsl/troubleshooting#resolve-local-names-in-wsl (install libnss-mdns and configure /etc/nsswitch.conf) to enable Linux to use mDNS queries
let us know know if you have any questions

Thank you @CatalinFetoiu I will try this but I'm also interested in making things work in the NAT mode. Any idea why disabling DNS tunneling does not fix it like it did for some people?

@YAmikep I see, thanks for following up. Please collect networking logs using https://github.com/microsoft/WSL/blob/master/diagnostics/collect-networking-logs.ps1 from the case with NAT mode and dnsTunneling=false and we can take a look

@CatalinFetoiu
Ok thanks, I am going to try to collect these logs.
To clarify though, what do I need to supposedly have .local names resolution work on WSL?
avahi-daemon and/or libnss-mdns as well?

@CatalinFetoiu
Ok thanks, I am going to try to collect these logs.
To clarify though, what do I need to supposedly have .local names resolution work on WSL?
avahi-daemon and/or libnss-mdns as well?

@YAmikep if you are in NAT networking mode, you don't need to install those, but you need to disable DNS tunneling
if you are in mirrored networking mode, you can install libnss-mdns and configure /etc/nsswitch.conf as shown in https://learn.microsoft.com/en-us/windows/wsl/troubleshooting#resolve-local-names-in-wsl

I use Ubuntu 24.04.1 as my distribution and have found the following solution without using avahi-daemon and libnss-mdns.
Make the following adjustments in the files mentioned
#.wslconfig
dnsTunneling=false
#/etc/wsl.conf
[boot]
systemd=true
[network]
hostname=wsl
generateResolvConf = false
Do not forget to restart the WSL with wsl.exe --shutdown for the adjustments to take effect.
In order to have systemd/resolved use mdns, two more steps are necessary:

multicastDNS=yes must be put into /etc/systemd/resolved.conf and resolved restarted to enable it in general.

it must be enabled per interface. On the command line this can be done with
resolvectl mdns eth0 yes

The activation of mDNS for the individual interfaces can then be checked with the command resolvectl mdns.

Same for me, got the issue after win11pro 24h2 udpate. Added the line in comment above and now it works again.

I just got a new machine with Windows 11. I had the same problem. WHAT A PAIN.  Thanks to finding this...hours...later...


I want my table to render without splitting words

This is a Bug Report

Problem:

I want my table to render without splitting words.
Rendering:
Source:

| `interactiveMode` Value | Meaning |
| ----------------------- | ------- |
| `Never` | This exec plugin never needs to use standard input, and therefore the exec plugin will be run regardless of whether standard input is available for user input. |
| `IfAvailable` | This exec plugin would like to use standard input if it is available, but can still operate if standard input is not available. Therefore, the exec plugin will be run regardless of whether stdin is available for user input. If standard input is available for user input, then it will be provided to this exec plugin. |
| `Always` | This exec plugin requires standard input in order to run, and therefore the exec plugin will only be run if standard input is available for user input. If standard input is not available for user input, then the exec plugin will not be run and an error will be returned by the exec plugin runner. |

Proposed Solution:

Eh, I dunno...give the user some way to control this?

Page to Update:

This comes from https://github.com/kubernetes/website/pull/28429#discussion_r664774943.

/label area/web-development

/area area/web-developmen

/area area/web-development

hmm, maybe
/area web-development

/triage accepted
/priority backlog

In CSS, it could be something like:
if a monospaced <tt> element is the first and last child of a <td> (ie, it's the whole cell) then try to avoid wrapping.
If there's no pure-CSS solution, we could improve the existing table shortcode to let you specify colgroups. Sounds trick though!


gplates while loading shared libraries: libSM.so.6

Describe the bug
Unable to launch gplates on NixOS. A shared object file appears to be missing.
Steps To Reproduce
Steps to reproduce the behavior:

nix-shell -p gplates
gplates
Output:
gplates: error while loading shared libraries: libSM.so.6: cannot open shared object file: No such file or directory

Expected behavior
gplates is expected to launch and be usable.
Screenshots
N/A
Additional context
N/A
Notify maintainers
No listed maintainers on Nix Packages but @wegank was the last to touch the source for this application.
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.74, NixOS, 23.11 (Tapir), 23.11.3376.1b64fc128799`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixos-23.11"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

Probably adding libSM to buildInputs and/or adding autoPatchelfHook to nativeBuildInputs might help, but I'm too tired to debug tonight. I'll look into it in the next few days.

Thank you, I was reading about the first option you suggested being a potential fix. I'll see if I can wrap my head around doing it on my own as well this week, though I don't have a lot of experience contributing to projects likes this.


Display battery gauge on all screens

Currently the battery gauge is only shown on the Song screen.
To make room for it on the Project screen the firmware build version can be removed.
To make room for it on the Device screen, the firmware build version can be moved to the bottom row of the screen layout.

Sample name will also be a problem in phrase screen


PHP or Composer are not correctly setting up

When I installed the atom-autocomplete-php an error appears to me, how can I specify the absolute path of composer and php?
Either PHP or Composer is not correctly set up and as a result PHP autocompletion will not work. Please visit the settings screen to correct this error. If you are not specifying an absolute path for PHP or Composer, make sure they are in your PATH. Feel free to look package's README for configuration examples

Thanks for reaching out!
Because we treat our issues list as the Atom team's backlog, we close issues that are questions since they don't represent a task needing to be completed. For most questions about Atom there are a lot of options, check out:

The Atom Flight Manual
The Atom FAQ on Discuss
Discuss, the official Atom and Electron message board
Join the Atom Slack team

On Discuss and in the Atom Slack team, there are a bunch of helpful community members that should be willing to point you in the right direction.


'Realm accessed from incorrect thread.' While accessing Realm via Siri Shortcuts.

Goals

Expected Results
I am expecting it to work it seamlessly with Siri Shortcut like it works with my main app.
Actual Results
e.g. the stack trace of a crash
2018-08-07 09:05:36.498102+0530 TrainsIntents[21484:1449214] *** Terminating app due to uncaught exception 'RLMException', reason: 'Realm accessed from incorrect thread.'
*** First throw call stack:
(
0   CoreFoundation                      0x000000010feea73b __exceptionPreprocess + 331
1   libobjc.A.dylib                     0x000000010f2dc735 objc_exception_throw + 48
2   Realm                               0x000000010dc23819 -[RLMRealm verifyThread] + 137
3   Realm                               0x000000010daeae37 _ZL18RLMVerifyRealmReadP8RLMRealm + 87
4   Realm                               0x000000010daea71a RLMGetObjects + 90
5   RealmSwift                          0x000000010e5699d5 $S10RealmSwift0A0C7objectsyAA7ResultsCyxGxmAA6ObjectCRbzlF + 277
6   TrainsIntents                       0x000000010bc66ec6 $S13TrainsIntents12RealmManagerC7objects6ofType0C5Swift7ResultsCyxGxm_tAF6ObjectCRbzlF + 38
7   TrainsIntents                       0x000000010bdc3f1c $S13TrainsIntents17TrainRouteManagerC5route4withAA0C6DetailCSgSS_tF + 92
8   TrainsIntents                       0x000000010bc3bbd7 $S13TrainsIntents20RunningStatusManagerC18fetchTrainSchedule4with9isRefresh10completionySS_SbySayAA0gH0CGSg_s5Error_pSgtctF + 135
9   TrainsIntents                       0x000000010bc3e3b6 $S13TrainsIntents20RunningStatusManagerC05fetchD04with9startDate9isRefresh43shouldShowLocationPermissionErrorIfRequired0L16ConsiderNTESData10completionySS_10Foundation0I0VS3byAA05TraincD0CSg_APSayAA0W8ScheduleCGSgARSgs0P0_pSgtctF + 1286
10  TrainsIntents                       0x000000010bc2cb01 $S13TrainsIntents26RunningStatusIntentHandlerC6handle6intent10completionyAA0cdE0C_yAA0cdE8ResponseCctF + 385
11  TrainsIntents                       0x000000010bc2cefb $S13TrainsIntents26RunningStatusIntentHandlerC6handle6intent10completionyAA0cdE0C_yAA0cdE8ResponseCctFTo + 139
12  Intents                             0x000000011069e371 __50-[INExtensionContext handleIntent:withCompletion:]_block_invoke.199 + 237
13  Intents                             0x000000011069fb81 __47-[INExtensionContext _deliverIntent:withBlock:]_block_invoke + 94
14  libdispatch.dylib                   0x000000011196152d _dispatch_call_block_and_release + 12
15  libdispatch.dylib                   0x0000000111962597 _dispatch_client_callout + 8
16  libdispatch.dylib                   0x0000000111969054 _dispatch_lane_serial_drain + 720
17  libdispatch.dylib                   0x0000000111969b97 _dispatch_lane_invoke + 401
18  libdispatch.dylib                   0x00000001119729c6 _dispatch_workloop_worker_thread + 645
19  libsystem_pthread.dylib             0x0000000111d44fd2 _pthread_wqthread + 980
20  libsystem_pthread.dylib             0x0000000111d44be9 start_wqthread + 13
)
libc++abi.dylib: terminating with uncaught exception of type NSException
Steps to Reproduce
The way to preproduce is access realm via siri shortcut.
Code Sample

Version of Realm and Tooling

Realm framework version: 2.10.2
Realm Object Server version: ?
Xcode version: 10.0 beta 5
iOS/OSX version: iOS 12
Dependency manager + version: Cocoapods

@kidsid-Ixigo You are not allowed to access a realm object from another thread than the one you obtained it from. Please see the section about threading here: https://realm.io/docs/swift/latest/#threading

@bmunkholm I am not doing any thread-specific things with Realm and its working fine in my main app. Its created on the main thread as usual I believe but accessing it in the same way with Siri Shortcuts throws this error. Now I am wondering that the way Realm instance is created and the way it's used is used is same in main app and Siri shortcut while it works fine with the main app but throwing an error with shortcuts.

@kidsid-Ixigo Ok, thanks for the update. The best way to expedite this would be if you could be helpful and try to replicate this in a minimal app so we can start to narrow it down.

Common practise is, that databases should be accessed on background thread. Realm, however, likes to be called on main thread only. If you call it only on 1-2 places in your app, you can get away with using background threads, but in larger apps not (according to my experience).
So - call realm on main thread only and you should be fine. It solved my problem

It's not a problem to call Realm from background threads, you just need to make sure to obtain a Realm instance for the background thread, rather than try and access the one you opened on the main thread.

@kidsid-Ixigo do you have a sample code or a sample app to send us? I haven't worked with Siri shortcuts yet, but I suspect you may simply have to allocate another realm instance.

Closing this issue due to age. If you have any other questions please open a new issue.
Thanks.


Commerce Orders & Customers

Wondering what your thoughts are to exclude certain sensitive fields from being imported or obfuscate the data.
I saw in an earlier issue that you prefer to have the whole database locally, does this apply to orders and customers as well?
Thanks!

So sure, if there is sensitive data that you're not in a position to handle in a secure manner, it'd make sense to not pull it down locally.


Set resource telemetry.sdk.* and scope otel.scope.name|version attributes for the OpenTelemetry tracer

Commit Message: Set resource telemetry.sdk.* and scope otel.scope.name|version attributes for the OpenTelemetry tracer
Additional Description:
Risk Level: Low
Testing: Unit tests added
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #36781]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

CC @wbpcode


Case insensitive in Filter query

Hi everyone!
The default prisma behavior without mode param is mode: sensitive, but I think the filter query should be case insensitive, like #5, to make the searching experience easier.
Why it has been removed? I think the ideal would be to make it configurable.

@VladyslavParashchenko this would probably require some runtime checks for database driver to make sure it supports mode: 'insensitive'. Ideally Prisma would just ignore this param if it's not supported instead of throwing error though.


Copilot insert code block keybinding overrides commit action in the scm view

Reported by @jooyoungseo

@jooyoungseo, pls lmk if this still an issue for you. testing on macos, cmd+enter commits while ctrl+enter inserts the code block


Issues with latest MPRester version

When I issue the command:
pydefect_vasp mp -e Mg Al O --e_above_hull 0.0005
i am getting error:
File "/home/user/Downloads/anaconda3/bin/pydefect_vasp", line 8, in
sys.exit(main())
File "/home/user/Downloads/anaconda3/lib/python3.9/site-packages/pydefect/cli/vasp/main_vasp.py", line 201, in main
args.func(args)
File "/home/user/Downloads/anaconda3/lib/python3.9/site-packages/pydefect/cli/vasp/main_vasp_functions.py", line 43, in make_competing_phase_dirs
query = MpQuery(element_list=args.elements, e_above_hull=args.e_above_hull)
File "/home/user/Downloads/anaconda3/lib/python3.9/site-packages/pydefect/util/mp_tools.py", line 27, in init
self.materials = m.query(criteria=criteria,
File "/home/user/Downloads/anaconda3/lib/python3.9/site-packages/mp_api/client/mprester.py", line 1188, in query
raise NotImplementedError(
NotImplementedError:
The MPRester().query method has been replaced with the MPRester().summary.search method.
Note this method also no longer supports direct MongoDB-type queries. For more information,
please see the new documentation.

Thanks for your suggestion.
I modified pydefect so that new MPRester can also be used.
Please  download ver 0.7.0.
Let me know if you have further issues.

Thanks. there is no error now.


Loading more than one replay buffer?

Greetings,
Is there any possibility of loading more than one replay buffer before training? For the time being, the second loaded buffer overwrites the first.

Hi,
We haven’t planned to add this feature for now. But the replay buffer is a dictionnary saved with pickle, you can easily load the two replay buffers and merge them into one replay buffer.

Good to hear, thank you!

Sorry to bother again. I have used this bit of code to merge three .pkl files,
import pickle

file = open('results/santorini/pickles/replays.pkl', 'ab')

with open('results/santorini/pickles/replay_buffer001.pkl', 'rb') as f:
    pickle.dump(pickle.load(f), file)
with open('results/santorini/pickles/replay_buffer002.pkl', 'rb') as f:
    pickle.dump(pickle.load(f), file)
with open('results/santorini/pickles/replay_buffer003.pkl', 'rb') as f:
    pickle.dump(pickle.load(f), file)

which did the job successfully. However, muzero loads the games of the first replay buffer only. I suspect this has to deal with common indices, shared between replay buffers. The loader might just load the elements with first unique indices.
Could you please recommend any solution?

Solution:
The replay_buffer dict has integer indices - which makes it nothing more than a basic list. One can just load list() files from .pkl, concatenate them together, convert them to dict() again and save as .pkl .
import pickle

out_list = list()

with open('results/santorini/pickles/replay_buffer001.pkl', 'rb') as f:
    d = pickle.load(f)
    out_list.extend( [v for k,v in d.items()] )
with open('results/santorini/pickles/replay_buffer002.pkl', 'rb') as f:
    d = pickle.load(f)
    out_list.extend( [v for k,v in d.items()] )
with open('results/santorini/pickles/replay_buffer003.pkl', 'rb') as f:
    d = pickle.load(f)
    out_list.extend( [v for k,v in d.items()] )

out_file = open('results/santorini/pickles/replays.pkl', 'wb')
pickle.dump(output, out_file)
out_file.close()


Azure Portal

Add a Note way up on the page:
Note: the Azure Portal still uses RESTv2, and therefore it will either not display entities created via v3 APIs (eg. Transforms), or will show an incomplete result (eg. Assets). Please do not use the Azure Portal to view or manage entities created via the v3 APIs.

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

ID: 8bbc0942-0a02-05f6-4519-bac2c64e7a18
Version Independent ID: 60d5482f-50b5-a68d-fc10-ce92789bd3fa
Content: Migrate from Azure Media Services v2 to v3
Content Source: articles/media-services/latest/migrate-from-v2-to-v3.md
Service: media-services
GitHub Login: @Juliako
Microsoft Alias: juliako

@anilmur Thank you for the feedback. It appears the doc author has added this to the doc.
As a result, we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


[Model] Add memory swap path property

MemorySwapPath property is added to define path for swap files.
If it's not defined, it creates files in crurent directory.
Signed-off-by: Jiho Chu<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2018. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @jihochu, nntrainer/tensor/cache_pool.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @jihochu, nntrainer/tensor/cache_pool.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md


S3 bucket with object lock enabled can't be used

When I try to save database on S3 bucket which has object lock enabled (compliance mode), it fails with error:

The reasoning for using object lock is to protect keepass database and all its previous versions from accidental or malicious deletion.

Is this still relevant or can this be closed?

Okay, thnaks for your reply, so I close this.


[NeuralChat] Fix backprop error for text only examples

Type of Change
Bug fix
Description
This PR fixes the problem reported in #1181. The problem only happens on HPU devices and can be recreated with this simple script:
import torch
import habana_frameworks.torch.core as htcore
import habana_frameworks.torch as htorch

a = torch.ones([10], requires_grad=True).to("hpu")
b = torch.ones([10], requires_grad=True).to("hpu")
c = torch.cat([a, b[0:0]], dim=0)
c.sum().backward()

which outputs:
Traceback (most recent call last):
  File "/intel-extension-for-transformers/intel_extension_for_transformers/neural_chat/examples/finetuning/multi_modal/test.py", line 9, in <module>
    c.sum().backward()
  File "/usr/local/lib/python3.10/dist-packages/torch/_tensor.py", line 502, in backward
    torch.autograd.backward(
  File "/usr/local/lib/python3.10/dist-packages/torch/autograd/__init__.py", line 251, in backward
    Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: Function SliceBackward0 returned an invalid gradient at index 0 - expected device hpu:0 but got cpu

which is the same error seen in the issue. Running this code on a CUDA device does not cause the issue which is why it's in the original LLaVA implementation. I removed creating the image embeddings and concatenating the empty array as it seems to no be doing anything. It was introduced in this PR but it's not clear to me why they needed to concatenate the cur_image_features[0:0] as it'd just be an empty list.
I've confirmed this works by running the finetuning script with DeepSpeed zero3 on only examples without images (to try and reproduce the error) and with images as well for a few iterations.
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

hi, good PR!! I had also met the issue when I enabled llava training on Gaudi2. I thought it was a network problem and some pictures were not downloaded completely. So I ignored the issue.
Now, I think your analysis and modifications are reasonable and correct!

Thanks @dillonalaird for your great contribution!


Downloading gretty-runner-jetty9-1.4.0.jar is quite slow in the process of 'gradle jettyRunDebug'.

Everytime after I executed 'gradle jettyRunDebug' in the terminal, the process was stuck in the "> Resolving dependencies ':simple-parent:simple-webapp:grettyRunnerJetty9" , and finally the webapp was deployed.
I don't know what was doing in the backend, and is there any way to speed up?

It seems that the process of :appRunDebug will be much fast if I change repository.

Could you, please, try running it with --info parameter? Does it give more information on the bottleneck with performance?

I can avoid the preceding problem with changing a new repository, but now I find another problem with the same project, as states below:
I  have multiple modules, three of them are: multi-web, multi-velocity, multi-jsp, namely, I move the velocity and jsp configurations to another two individual modules from multi-web module. The web app still works well after I deployed the war generated by gradle into jetty/tomcat, and now I want to start the web app with gretty plugin. But when I go into multi-web module and execute "gradle appRun", then error messages said that "...java.io.FileNotFoundException: Could not open ServletContext resource [/WEB-INF/velocity/velocity.properties]...". In fact, the file velocity.properties is located in multi-velocity module rather than multi-web module. Then I move the velocity.properties file and other necessary files to the /WEB-INF/velocity/ directory of multi-web, then gretty works, and the "gradle appRun" can start the web app.Other information:
velocity.properties path: (multi-app is my root module.)
multi-app/multi-velocity/src/main/resources/META-INF/resources/WEB-INF/velocity/velocity.properties
3 modules:
multi-app/multi-velocity
multi-app/multi-jsp
multi-app/multi-web


run and nothing

http://<IP_ADDRESS>:3003/

What is wrong?
GET http://<IP_ADDRESS>:3003/api/token 404 (Not Found)
I create callback with this url
http://<IP_ADDRESS>:3003/api/forge/callback/oauth

Did you make sure your Server.js and also the routes/oauth.js are uncommented ? Did you install all the dependencies and ran the server with the correct port, I remember I have it setup for 3000 but I'm sure you probably changed it.

/home/felipe/nodejs-forge-viewer-tutorial/server.js:

/home/felipe/nodejs-forge-viewer-tutorial/routes/oauth.js

cp credentials_.js credentials.js
/home/felipe/nodejs-forge-viewer-tutorial/credentials.js:

node_modules:

Did you input the URN in the index.js file correctly ? Can I see that file as well. Everything in the previous ones looked ok.

/home/felipe/nodejs-forge-viewer-tutorial/www/js/index.js:

Is possible hangout with your via skype or hangout or something, because only working models.autodesk.io locally:
http://bit.ly/2rEZ6zj
But that only useful for 60 min
Stay alert

Ok I have a meeting at the moment that goes until 3:30 EST Time, let's meet at 3:30 my time, where are you based ?
Cheers,

here gmt -5, then ping me in 2 hours


Towards a v3?

[ ] Externalize history logic.
[ ] Create separate package with persistent update helper.
[ ] Create separate package for data model (codename bonsai).
[ ] Rely on persistence for update only (pre-memoization scheme) (this means we drop non-persistent version of the tree).
[ ] The tree is cursor is tree paradigm (@jacomyal tmtc)

@jacomyal, could you develop here the use case and benefits you were seeing related to the one abstraction only paradigm please?


来自PyXLL作者的建议

以下内容来自PyXLL作者的邮件：

Hi!
I'm the founder/developer of PyXLL, and I came across your web page about using PyXLL for real time data from VeighNa here https://www.vnpy.com/docs/cn/excel_rtd.html
I noticed that in your description you do not mention adding your "vnpy_excelrtd" module to the pyxll.cfg file, and without that your module will not be loaded. I believe this could be a problem for some of your users who are not familiar with how PyXLL works.
Rather than have your users configure the pyxll.cfg file to include your module, did you know that you can add entry points to your Python package so that PyXLL will auto-discover it? This is what other packages do so they do not require the user to do any configuration, the user then simply has to pip install the package and PyXLL will find it.
Please see https://www.pyxll.com/docs/userguide/distribution.html#setuptools-entry-points for information about how to do this. For an example of this being used, you can take a look at these packages.
https://github.com/pyxll/pyxll-jupyter
https://github.com/pyxll/pyxll-openbb
The first specifies the entry points in the setup.py file, and the second uses the newer pyproject.toml method.
I hope this information is helpful, and please do let me know if anything is not clear or if I can be of any further help.
Kind regards,
Tony

计划调整setup.py来完成pyxll模块的自动识别加载。

PR merged. #5


Clicking on media in the gallery should never insert as background

Bug Description
Now that we have solid background masking, we'd like to change the behavior to image/video insertion, so that clicking an image or video will never set it as background, and always insert it to the page, consistently.
Expected Behaviour
When clicking media in the media gallery, said media should be inserted on top of the current elements - it should never become the background media by default.

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

noted in sprint 26 sheet under the "non alpha" section.

I couldn't agree with this ticket more!

#841


Multiple type support

Multiple types type: ["boolean", "null"] could be possible when widget is set for those.
Currently nothing will work with multiple types, but the actual implementation - if the ui is not selected by type - should be rather simple. The refactoring of existing usages may be the more problematic part.
The "used-for-validation" type can be get with the first plugin / after combination & referencing is executed. If not undefined and one of the specified types is valid, this type is used, otherwise "type-error" is set - but e.g. all further validators must do nothing then as no type can be used for "should the validator do something".

Implemented multiple type support, validators support JSON-Schema spec., but rendering needs either custom widgets, handling "which type should be entered". Or using simple only one-scalar types and null for nullables, e.g.: "type": ["string", "null"], as null is not counted as "input type".
Utility functions:

import {schemaTypeToDistinct} from '@ui-schema/ui-schema/Utils/schemaTypeToDistinct';
import { schemaTypeIs } from '@ui-schema/ui-schema/Utils/schemaTypeIs'
import { schemaTypeIsAny } from '@ui-schema/ui-schema/Utils/schemaTypeIs'
import { schemaTypeIsNumeric } from '@ui-schema/ui-schema/Utils/schemaTypeIs'


[Sema] Shepherd @manavgabhawala's changes to circular protocol diagnostics

What's in this pull request?

#2123's type checker changes were causing an infinite loop, but the rest of the changes related to circular protocol diagnostics are still valuable to have.  This will pull them in and leave the type checker alone for now.
h/t @rintaro

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci please test and merge.

@swift-ci please test.

@swift-ci please test OS X platform.

Let's get some sanity here.
@swift-ci please test OS X platform.

This passed linux tests before I futzed around with the crash tests.


Mobile Setup Portfolio

In this project I:

Added flexbox for responsiveness
Incorporated linters.
Followed Github flow principles

Hello,
I'm writing to enquiring about the feedback that you have given me. I would
like to say that the template that you have shown about the background
image had an issue and the color of the hamburger icon was when it was
reviewed the first time. I think there was a problem on your end because I
fixed the issues before asking for a review.
Kindly look into the matter
Regards Mark Yegon.
On Sat, 18 Feb 2023 at 16:01, Souad El Mansouri @.***>
wrote:

@.**** approved this pull request.
Hi @MarkYegon7 https://github.com/MarkYegon7 ,
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are
still some which aren't addressed yet.
Required Changes ♻
Check the comments under the review.
_also Please make sure to apply all changes required by previous reviewer
you have done a great work and you still have only one step to the approval
your body has a margin that prevents the background image from covering
the full width as it is in figma file you can give it a 100% width and
margin=0 then resize your sections inside
figma design
[image: background image]
https://user-images.githubusercontent.com/59707859/219867076-9e865268-63d8-4a3a-ae50-37c7f99b70c8.PNG
your design
[image: hamberger icon color]
https://user-images.githubusercontent.com/59707859/219867098-120f5edd-2f46-4fc0-9c90-0efd0dfb8c8f.PNG
_Also the hamberger icon has a diffrent color, you can export the icon
from figma template.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this
PR. However, I strongly recommend you to take them into account as they can
make your code better. Some of them were simply missed by the previous
reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In README.md
https://github.com/MarkYegon7/Portfolio-Website/pull/1#discussion_r1111003954
:

+> Describe 1 - 3 features you will add to the project.

[ OPTIONAL ] don't forget to remove all default text under each section
after using your own content as this text is there just to guide you and
should be removed to make your README look professional 😊
In index.html
https://github.com/MarkYegon7/Portfolio-Website/pull/1#discussion_r1111005196
:

@@ -0,0 +1,40 @@

+
+
+

Portfolio Website

+
+

well done, your logo is now a link as users are used to click the logo to
get back to the home page 👍
[ OPTIONAL ] don't forget to customize your logo using your name or icon .
In styles.css
https://github.com/MarkYegon7/Portfolio-Website/pull/1#discussion_r1111012004
:

height: 55px;

width: 50px;

margin-top: 1%;

margin-right: 11%;

margin-bottom: 0;

color: #6070ff;

+}

+.title {

margin-right: 20%;

width: 327px;

height: 270px;

font-family: 'Poppins', sans-serif;

font-style: normal;

font-weight: 700;

font-size: 300%;

you can use rem unit for font-size rather than percentage it's a scaling
unit and good to use for responsive design .
—
Reply to this email directly, view it on GitHub
https://github.com/MarkYegon7/Portfolio-Website/pull/1#pullrequestreview-1304589530,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUJ7GBDMY6PB25OXKAUQRXTWYDBZ7ANCNFSM6AAAAAAU7K7CPA
.
You are receiving this because you were mentioned.Message ID:
@.***>


[CSL-50] Check that more than 50% of stake participated in seed selection

In this PR “50% of stake” means “50% of richmen's stake”. However, @pva701 said

я во всех своих телегах про 749 брал больше половины всего стейка
и пацики меня не хэйтили

so maybe someone wants to look at his code as well :trollface:

@neongreen

In this PR “50% of stake” means “50% of richmen's stake”.

did you get approved from scientists?

@pva701 well, @gromakovsky said

Мне кажется более правильным сравнивать со стейком, принадлежащим богачам

but I'll go ask scientists just in case

Asked Bernardo, he said it's okay:

Since we assume that honest users controlling more than 50% of the total stake are online we can interpret the "stakeholder majority" either way.

TODO: test

Currently I can't get it to actually output the SSC couldn't compute seed: part even when I run 6 nodes locally and kill 3 of them.

Okay, I fixed it (and tested).


Request for a ping indicator in the world switcher menu

I believe it would be very useful, and hopefully easy to implement an indicator to show how much ping a world has when hopping.
Two potential design mock-ups:

I would say first one when u hold shift or alt

Will submit a PR for this tomorrow, with an option for [SHIFT] to display the pings in place of world activities

World Switcher Filter plugin inclueds ping
#2873


text_ process support

I just noticed this:
https://github.com/Open-EO/openeo-processes/blob/56b98ae3f2fd68cff3815c3885af3361e7c187fb/CHANGELOG.md#L15-L16
we actually added the text_ processes this week in VITO backend: https://github.com/Open-EO/openeo-python-driver/issues/105
(sidenote: we implement text_merge instead of text_concat at the moment)

Uhm... so what's the action to take on side of the processes?

Well, it's not really necessary any more to move the text_ processes to proposals.
Unless there are reasons to revise them.
To give a bit of background: we have an internal user that wants to do define a UDP with a year parameter and some processing along the lines of:
year = Parameter.integer("year", description="Year")
previous_year = subtract(year, 1)
start_date = text_merge([previous_year, "-10-01"])
end_date = text_merge([year, "-12-31"])
cube = con.load_collection("TERRASCOPE_S2_TOC_V2", temporal_extent=[start_date, end_date])

Okay. As it's also available on GEE, we can indeed move it back.
Would it be ok for you to rename to text_concat? @soxofaan

Would it be ok for you to rename to text_concat?

yes, I would also prefer text_concat or text_join  over text_merge


'1-start-here' branch shows no code

Hi :)
The ice_breaker.py is not suppose to be in this begining branch
the only files that should be:

cheeers


Jitsi on port 443

Hello,
We installed multiple jisti version, but we have been mandatory to install last version.
We want to all data go thru port 443, not on port 10000.
Our old jitsi version worked fine on port 443.
But all last version use port 10000.
Can someone help or explain us, how to force all jitsi flow on port 443 ?
Debian 10.6 x64
Jitsi <IP_ADDRESS>
Thanks a lot in advance
Any help will be appreciate

Use the community forum when you have questions or problems before opening new issues, thank you.
And this is the wrong repo.


[BUG] cuML/KNN ImportError

Describe the bug
Did nbconvert and ran the resulting script. Got the TB below.  Not sure if it's related to the other KNN issue #68 on API changes (seems like it could be)
(rapids) root@803553dd81fd:/rapids/notebooks/cuml# python /tmp/knn_demo-test.py
Traceback (most recent call last):
  File "/tmp/knn_demo-test.py", line 10, in <module>
    from cuml import KNN as cumlKNN
ImportError: cannot import name 'KNN' from 'cuml' (/conda/envs/rapids/lib/python3.7/site-packages/cuml-0.5
.1+596.gb05d87e-py3.7-linux-x86_64.egg/cuml/__init__.py)

Steps/Code to reproduce bug
nbconvert NB to get script, run script, see TB.
Expected behavior
no TB :)
Environment details (please complete the following information):
nightly container
Additional context
n/a

PR #72 should solve this

closing now that #72 merged. Please re-open if still an issue

Trying to import cuml.neighbors but got the import error.
Env.: Google colab.
import of cuml, and cudf are successful.
https://colab.research.google.com/drive/1rY7Ln6rEE1pOlfSHCYOVaqt8OvDO35J0


Fix using regex patterns when matching raw body value

When matching non-json or xml body with regex the expression gets
escaped. Because of that all standard patterns (number(), positiveInt())
will not work as well as any other regex involving escaped characters.
E.g.
response {   status OK()   body(value(stub("1"), test(number()))) }
Will generate invalid matcher:
assertThat(responseBody).matches("-?(d*.d+|d+)");
Valid version is:
assertThat(responseBody).matches("-?(\\d*.\\d+|\\d+)");

createBodyComparison methods in SpockMethodRequestProcessingBodyBuilder.groovy and JUnitMethodBodyBuilder.groovy as basically the same, but generate different code depending on framework (Spock, JUnit). I was thinking to push it to the base class, but it feels to me that here a bit of copy-paste is better that abstraction.

First we'll have to merge https://github.com/spring-cloud/spring-cloud-contract/pull/690 . I'll skip reviewing this PR until I merge that one. First, we'll need to branch the current master to 2.0.x branch and start a 2.1.0 branch.

Codecov Report

Merging #692 into master will increase coverage by 0.12%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #692      +/-   ##
============================================
+ Coverage     60.83%   60.96%   +0.12%
- Complexity     2017     2020       +3
============================================
  Files           220      220
  Lines          8350     8354       +4
  Branches       1302     1302
============================================
+ Hits           5080     5093      +13
+ Misses         2585     2573      -12
- Partials        685      688       +3

Impacted Files
Coverage Δ
Complexity Δ

...cloud/contract/verifier/util/RegexpBuilders.groovy
47.61% <ø> (ø)
6 <0> (ø)
:arrow_down:

...der/SpockMethodRequestProcessingBodyBuilder.groovy
75.6% <100%> (+1.25%)
27 <1> (+1)
:arrow_up:

...act/verifier/builder/JUnitMethodBodyBuilder.groovy
73.46% <100%> (+1.12%)
29 <2> (+1)
:arrow_up:

...r/builder/RestAssuredJUnitMethodBodyBuilder.groovy
86.36% <100%> (ø)
12 <1> (ø)
:arrow_down:

...loud/contract/stubrunner/AetherStubDownloader.java
64.15% <0%> (+0.94%)
26% <0%> (+1%)
:arrow_up:

...loud/contract/stubrunner/TemporaryFileStorage.java
48.38% <0%> (+25.8%)
5% <0%> (ø)
:arrow_down:

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 a8447cf...1bf34eb. Read the comment docs.

Cool, I'll keep an eye on this one and update all the tests to JUnit 5 if needed.

Congrats @alek-sys on a great PR :)


Added JsonCoverter for IFilterContainer

This pull request fixes an existing bug #1409
Please review :ghost:

[ ] @Mpdreamz / @gmarz

Happy to make any change from comments

LGTM :+1: this was a  grave omission not having these converters in place!


Experiment about using generics to catch version mismatches

This is an incomplete experiment to check what it means to use generics to catch version mismatches, using a "version token type", as proposed by @enikao .
To make this work, more work should be put in.
Most classes will need to be touched and I am not 100% sure we can make the generics always work.

One question: is it possible, with this approach, to introspect on the version at runtime? I see a LionWebVersionToken that has that info. btw: It is possible to reflect on type parameters at runtime, despite type erasure! :)

One question: is it possible, with this approach, to introspect on the version at runtime? I see a LionWebVersionToken that has that info. btw: It is possible to reflect on type parameters at runtime, despite type erasure! :)

Yes, but that means we need to both specify the type parameter and pass a value to the constructor

btw: It is possible to reflect on type parameters at runtime, despite type erasure! :)

Oh, good. But doesn't that work just with subclasses? Can you show what you had in mind?

btw: It is possible to reflect on type parameters at runtime, despite type erasure! :)

Oh, good. But doesn't that work just with subclasses? Can you show what you had in mind?

I'll try and add it to this branch ASAP

btw: It is possible to reflect on type parameters at runtime, despite type erasure! :)

Oh, good. But doesn't that work just with subclasses? Can you show what you had in mind?

I'll try and add it to this branch ASAP

Hi Meinte, I wanted to check with you if you can estimate when you will have time to look into this (as 2025 is sneaking on us and we are a bit behind with the release of LW Java 2024.1)

btw: It is possible to reflect on type parameters at runtime, despite type erasure! :)

Oh, good. But doesn't that work just with subclasses? Can you show what you had in mind?

I'll try and add it to this branch ASAP

Hi Meinte, I wanted to check with you if you can estimate when you will have time to look into this (as 2025 is sneaking on us and we are a bit behind with the release of LW Java 2024.1)

Sorry: forgot about this (or rather: I didn't put it on some kind of TODO queue) :(
Here's a link: https://stackoverflow.com/questions/1942644/get-generic-type-of-java-util-list
I intended to add an actually representative unit test to lionweb-java, but I still have to get around to that.


Add resume="auto"

The current resuming logic allows the user to pass --resume=True to find the latest checkpoint in the out_dir folder. However, this requires the checkpoint to exist, and is safe to ensure the user loads a checkpoint when intended. However, in a job where we auto-resume, we can't set resume=True because an initial checkpoint never exists. This PR adds --resume='auto' that does the same as --resume=True except it won't error if no checkpoint is found.
When it's reviewed, I'll add it to the other scripts.

@rasbt for final look


Trait subtyping error

When the Iterator interface is changed from an interface to a trait, the builtin package tests should compile as normal since all classes in builtin that should implement the trait declare is Iterator[..].
After making the change the compiler complains that an Array[String] is not a subtype of ReadSeq[Stringable] because ArrayValues[String, this->Array[String]] does not implement Iterator[Stringable] even though it does and is correctly identified when Iterator is marked as an interface rather than a trait. The full error is as follows:
Error:
/home/theodus/dev/ponyc/packages/builtin_test/_test.pony:619:34: argument not a subtype of parameter
    h.assert_eq[String]("_".join(["zomg"]), "zomg")
                                 ^
    Info:
    /home/theodus/dev/ponyc/packages/builtin/array.pony:511:17: ArrayValues[String val, this->Array[String val] ref] ref^ does not implement trait Iterator[this->Stringable box] ref^
      fun values(): ArrayValues[A, this->Array[A]]^ =>
                    ^
    /home/theodus/dev/ponyc/packages/builtin/array.pony:511:3: method result ArrayValues[String val, this->Array[String val] ref] ref^ is not a subtype of Iterator[this->Stringable box] ref^
      fun values(): ArrayValues[A, this->Array[A]]^ =>
      ^
    /home/theodus/dev/ponyc/packages/builtin/array.pony:511:3: Array[String val] ref is not a subtype of ReadSeq[Stringable box] box: method 'values' has an incompatible signature
      fun values(): ArrayValues[A, this->Array[A]]^ =>
      ^

I'd suggest trying to create a minimally reproducible example using new types that aren't entangled with the rest of the stdlib.  The problem may be more apparent once it is reduced to its barest elements.

The following shows an example of this without any builtin types where it compiles if Foo is an interface but not if it's a trait:
trait Foo[A]
  fun ref foo(): A

class Bar[A]
  let a: A

  new create(a': A) => a = consume a'

  fun apply(): this->A => a

interface box ReadBar[A]
  fun bar(): Foo[this->A]^

primitive Baz
  fun baz() => None

interface box Bazable
  fun baz()

class BazHolder
  let bazes: Bar[Baz] = Bar[Baz](Baz)

  fun bar(): BazBar[Baz, this->Bar[Baz]] =>
    BazBar[Baz, this->Bar[Baz]](bazes)

class BazBar[A, B: Bar[A] #read] is Foo[B->A]
  let _bazes: B

  new create(bazes: B) => _bazes = bazes

  fun ref foo(): B->A => _bazes()

actor Main
  new create(env: Env) =>
    apply(BazHolder)

  fun apply(rs: ReadBar[Bazable]) => None

Here's the error to go along with the code example:
/home/sean/pony-scratch/1146/main.pony:35:11: argument not assignable to parameter
    apply(BazHolder)
          ^
    Info:
    /home/sean/pony-scratch/1146/main.pony:35:11: argument type is BazHolder iso^
        apply(BazHolder)
              ^
    /home/sean/pony-scratch/1146/main.pony:37:13: parameter type requires Barable[Bazable box] ref^
      fun apply(rs: Barable[Bazable]) => None
                ^
    /home/sean/pony-scratch/1146/main.pony:23:14: FooBar[Baz val, this->Bar[Baz val] ref] ref does not implement trait Foo[this->Bazable box] ref^
      fun bar(): FooBar[Baz, this->Bar[Baz]] =>
                 ^
    /home/sean/pony-scratch/1146/main.pony:23:3: method result FooBar[Baz val, this->Bar[Baz val] ref] ref is not a subtype of Foo[this->Bazable box] ref^
      fun bar(): FooBar[Baz, this->Bar[Baz]] =>
      ^
    /home/sean/pony-scratch/1146/main.pony:23:3: BazHolder iso^ is not a subtype of Barable[Bazable box] ref^: method 'bar' has an incompatible signature
      fun bar(): FooBar[Baz, this->Bar[Baz]] =>

@jemc do you know why there is a traits pass but not interface pass? I'd never looked into implementation and I'm very confused why there's a traits pass. Can you elaborate on how traits and interfaces are handled in the compiler?

in subtype.c when we do nominal_provides_trait check, it appears to be incorrectly erroring.

my guess so far is that in is_fun_sub_fun that the reification is being done wrong for trait and not interface.
or there's something of a difference in:
is_nominal_sub_interface
that works where
is_nominal_sub_trait
doesn't.
Down in is_nominal_sub_trait

@SeanTAllen - the traits pass is about copying methods from is statements into a type. It applies to both trait and interface

@jasoncarr0 is this actually a bug? Or is this along the lines of the conversation recently in Zulip...
https://ponylang.zulipchat.com/#narrow/stream/189985-beginner-help/topic/Can.20unions.20implement.20traits.3F

Ah, yes that conversation applies here, and I wouldn't personally take this to be a compiler bug, although the behavior is surprising. Traits, being nominal currently behave the same way as generic classes. We could discuss this and the alternatives and look to the history of the feature to confirm that this is intended.

We talked about this over sync. This would not be a compiler bug, and is at the language design level. It is potentially surprising or difficult behavior but is expected/explainable because traits use nominal subtyping rather than structural subtyping.
A change here is a change to the meaning of generics and would require an RFC


[bug] signing in mobile wallet connect (metamask) pops the app store

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
0.4.1
wagmi Version
0.5.3
Current Behavior
see: https://github.com/cowprotocol/cowswap/issues/510 - there are some work arounds, but it gets complicated because rainbow is a wrapper around wagmi which is a wrapper around wallet connect.
Requesting the signer to do things, will open metamask, but then open the appstore.
Expected Behavior
Expected to pop the wallet, then return to the app
Steps To Reproduce
sign anything on mobile transactions or text. see: https://github.com/MetaMask/metamask-mobile/pull/4167 https://github.com/cowprotocol/cowswap/issues/510
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

So as the instructions say there if you do any http requests at all (including to rpc servers, etc) then you will get the app store, but if the very first request is to do the browser thing then this will work.

As you mentioned, this is an unavoidable issue with app linking via WalletConnect on iOS. I recently added some documentation to the WalletConnect site that goes into more detail about this issue and explains how to manage it: https://docs.walletconnect.com/mobile-linking#ios-app-link-constraints


Confused about pages !

Why does a page needs a category ?

Example:
Company (category)
  - History (page)
  - Mission (page)
  - Vision (page)
Services (category)
  - Service 1 (page)
  - Service 2 (page)
  - Service 3 (page)

i get it now, but can we make the category optional ?

Yes @DeeJaVu
Do you want to implement this and do a PR?


update docs to suit UNDRR context.

I have edited some portions of the docs to suit UNDRR context. Further updates will be needed as we go along.

Looks good. Want to merge this in now, or add a few more changes first?

Looks good. Want to merge this in now, or add a few more changes first?

I'm adding more today so that we merge all.


Autocomplete support

When a user enters tab  in terminal, begin autocompletion of available tabs.

A considerable amount of progress on this was made in #69 .
Tab now has autocomplete scripts for bash, zsh, fish, elvish, and powershell.  It will print the scripts with tab --completion.
Setup instructions as well, tested for bash/zsh/fish.

Even more progress on #83
The bash, fish, and zsh have full dynamic support (e.g. complete tab names for tab foo, and running tabs for tab -w foo)

The autocomplete installation scripts are complete.  I think this can be closed now, as setup is quick.


Create Issue Labels

[x] Defect
[x] Feature

added myself as assignee

created defect label, enhancement already existed


insert op chunk encode

Description
This PR adds support for chunk encoding in insert ops for trees.

There are currently some illegal exports from chunked-forest being made, but this PR should update those exports and should be fixed after those changes are merged into this PR


CVE-2020-8908 in Guava v.28.2-jre, should upgrade to v.30.1.1

Describe the bug
CVE-2020-8908 in Guava v.28.2-jre, should upgrade to v.30.1.1
To Reproduce
Please check vulnerability section in :
https://github.com/google/guava/issues/4011
Expected behavior
Upgrading to v30.1.1 will mitigate this vulnerability.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: all
Browser all
Version all

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@ssainz  Hello, It seems to be resolved. #2202

Unless there is any objection, I'm going to close this as it should be resolved in Drill 1.19.

Hello there @luocooong , @cgivre - defect not fixed. Please check this line:
https://github.com/vdiravka/drill/blob/master/pom.xml#L49
shared guava still refers to guava 28.2 , thus, CVE-2020-8908 remains in Drill 1.19.

@ssainz YES. Thanks for the reminder. @vdiravka has already started the process of updating Drill shaded Guava.

Can this be closed?

Can this be closed?

I think this is still valid, but not sure.  @vdiravka do you know the current status?

Hi - the shared guava seems is still 28.2, then not fixed yet ~
https://github.com/apache/drill/blob/master/pom.xml#L52
<shaded.guava.version>28.2-jre</shaded.guava.version>

https://github.com/google/guava/issues/4011 affects a deprecated method and it only affects Drill if it Drill uses the deprecated method in the shaded org.apache.drill.shaded.guava.com.google.common.io.Files class. I see no evidence that Drill uses this method.


onConfirm Function

Can you please add onConfirm(TimeOfDay startTime, TimeOfDay endTime)?

I don't think that this is necessary.
You can get the times like this:
             TimeRange result = await showTimeRangePicker(
              context: context,
            );
            print("result " + result.toString());

TimeRange contains the start and end times and the result will be null if the user did not click the confirm button.


Chaining JVPs results in infinite recursion

Hello,
I am currently rewriting the code of my paper on continuous-discrete filtering in JAX to make it more efficient and readable (at the moment it relies on SymPy to compote the chain rule...)
In order to to this, I essentially need to compute a chain of JVP evaluated at the same tangents everytime, informally: jvp(jvp(jvp(...,v), v), v) where v is the fixed tangent. However when doing it programatically, this results in an infinite recursion as illustrated by the code below.
import jax
import jax.numpy as jnp

def compose_jvp(fun, tangents, n):
    def res(x): return jax.jvp(fun, (x,), tangents)[1]
    for _ in range(n-1):
        def res(x): return jax.jvp(res, (x,), tangents)[1]
    return res

cfun = compose_jvp(jnp.sin, (1.,), 10)
cfun(0.1)  # infinite recursion

However doing the same thing manually (see below) works fine.
afun = lambda z: jax.jvp(jnp.sin, (z,), (1.,))[1]
bfun = lambda zz: jax.jvp(afun, (zz,), (1.,))[1]
bfun(0.1)

I can't see anything wrong with my code, what do you think is happening?
Thanks!

This isn't a JAX bug. The issue is that your Python program has infinite recursion.
Try:
import jax
import jax.numpy as jnp
from functools import partial

def compose_jvp(fun, tangents, n):
    def res(f, x): return jax.jvp(f, (x,), tangents)[1]
    for _ in range(n-1):
      fun = partial(res, fun)
    return fun

cfun = compose_jvp(jnp.sin, (1.,), 10)
print(cfun(0.1))

This isn't a JAX bug. The issue is that your Python program has infinite recursion.
Try:
import jax
import jax.numpy as jnp
from functools import partial

def compose_jvp(fun, tangents, n):
    def res(f, x): return jax.jvp(f, (x,), tangents)[1]
    for _ in range(n-1):
      fun = partial(res, fun)
    return fun

cfun = compose_jvp(jnp.sin, (1.,), 10)
print(cfun(0.1))

Thank you!


Release maintenance/flocker 1.0.1/disable restart policies floc 2467

Fixes https://clusterhq.atlassian.net/browse/FLOC-2467
Update of https://github.com/ClusterHQ/flocker/pull/1696 (changing branch name ... which matters ...)

Looks good if new tests don't turn up a bug. If they do resubmit, otherwsie merge when buildbot is hapy.

@itamarst Can't resubmit since the state is "Accepted".  The tests did turn up another bug though.

You can can click Merged and then resurrect and then clickclikclickickclckclck but sure. Looks good to me, merge if buildbot is happy (haha).


[QUESTION] Is there a way to create a storage account with blob versioning enabled?

Query/Question
The resourcemanager API allows the creation of storage accounts, but is it possible to enable versioning for the accounts? I didn't find such API nor any example doing that.
Why is this not a Bug or a feature Request?
If the feature does not exist, this would be a feature request. But I want to confirm first.
Setup (please complete the following information if applicable):

Library/Libraries: azure-resourcemanager 2.14.0

@felipeek Thank you for your question.
@weidongxu-microsoft Can you please comment on this?

@felipeek
Would you explain "enable versioning for the accounts"? Would you share and doc e.g. on Portal or CLI or MSDocs on the feature?

Hello @weidongxu-microsoft
In the portal, you can enable versioning by selecting the storage account, clicking on "Data Protection" and then checking the box "Enable versioning for blobs".
The feature is called blob versioning: https://docs.microsoft.com/en-us/azure/storage/blobs/versioning-overview

@XiaofeiCao
Please check and add the feature to convenient code (search for DeleteRetentionPolicy, we had that).
@felipeek
Meanwhile you can use this REST API
https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/resourcemanager/azure-resourcemanager/src/samples/java/com/azure/resourcemanager/storage/generated/BlobServicesSetServicePropertiesSamples.java#L91-L105

@weidongxu-microsoft I see, it's this property right?
https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/resourcemanager/azure-resourcemanager/src/samples/java/com/azure/resourcemanager/storage/generated/BlobServicesSetServicePropertiesSamples.java#L103

yes


Update ni-winioctl-ioctl_disk_set_partition_info.md

Remove duplicate head

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

#sign-off


Release new version to support redix 0.7

Currently redix_pubsub 0.4.2 requires redix ~> 0.6.0.
I've tried it in a project with redix 0.7.0, and everything seems to work fine.
Is it possible to release a new version to turn ~> 0.6.0 into ~> 0.6?
Thanks!

hi @whatyouhide, have you had a chance to see this?

I also have this issue and would be interested in an update.

Released v0.5.0 which requires Redix v0.8.0, so this can be closed.


License

Hello!
This small build script seems like nice and tidy way of getting the built set up, however it's not under any explicit license as far as I can find. So it makes it unusable as is without written permission for anything but a learning resource.
Any chance it could get a type of license allowing it to be used?
Best Regards
Björn

Hello Björn, thanks for pointing that out. Added a MIT license.


R to glasseye

I'll be happy to help here.  Wondering though, do you plan to completely redo the tufte css in R or just use htmlwidgets infrastructure for the d3.js components, etc on top of the RStudio tufte?

@timelyportfolio thanks. That would be great. I haven't thought it through too well yet but that sounds like a good route. I'm in the middle of a bit of work at the moment but should have some time free in the next couple of days. Will get back to you soon


feat(language-core): enhance single root nodes collection

This is an enhancement which enabling it to collect all single root nodes under the template branch, and selectively collect single root child node through specific components.
It will also infer the type of $el as the union of all single root nodes.

There is an upstream issue I caused that needs to be fixed :)


IOS build issue - dependencies import issue

Bug report
Current behavior:

Expected behavior:

Steps to reproduce:

Screenshots

Environment information

Cordova CLI version

cordova -v

Cordova platform version

cordova platform ls

Plugins & versions installed in project (including this plugin)

cordova plugin ls

Dev machine OS and version, e.g.

OSX

sw_vers

Windows 10

winver

Runtime issue

Device details

e.g. iPhone X, Samsung Galaxy S8, iPhone X Simulator, Pixel XL Emulator

OS details

e.g. iOS 12.2, Android 9.0

Android build issue:

Node JS version

node -v

Gradle version

ls platforms/android/.gradle

Target Android SDK version

android:targetSdkVersion in AndroidManifest.xml

Android SDK details

sdkmanager --list | sed -e '/Available Packages/q'

iOS build issue:

Node JS version

node -v

XCode version

Related code:
insert any relevant code here such as plugin API calls / input parameters

Console output

console output

// Paste any relevant JS/native console output here

Other information:

After fixing pod path getting this error

Please ensure you have updated Cocoapods as outlined in the documentation.
Please try building the example project to rule out other possible causes in your own project.

I tried the example project it's giving the same error for me

I just had the same issue. Someone on the cocoapods repo said this plugin needs to be updated to use the newer cocoapods style where it doesn't automatically sync the whole repo. Here's the issue I filed over there: https://github.com/CocoaPods/CocoaPods/issues/9228
I worked around this by uninstalling cocoapods, installing version 1.7.5, syncing the repo, then upgrading to 1.8.3.

This should be resolved by #173


CI: Cilium L4LB XDP - NAT46x64 test

CI failure
Error:
Run cd /home/runner/work/cilium/cilium/test/nat46x64 && sudo ./test.sh cilium 4f91a9ec07a4cb879bdd780a3e910b81ceda38de

...

Error: Cannot add/update recorder: [PUT /recorder/{id}][500] putRecorderIdFailure  loading eBPF collection into the kernel: program tail_lb_ipv6: load program: argument list too long: BPF program is too large. Processed 1000001 insn (2474 line(s) omitted)

Hit on two different PRs already:

https://github.com/cilium/cilium/actions/runs/5175665325
https://github.com/cilium/cilium/actions/runs/5175441653

Sysdump could not be retrieved due to an error:
Run sudo cilium sysdump --output-filename cilium-sysdump-out
🔍 Collecting sysdump with cilium-cli version: v0.14.5, args: [sysdump --output-filename cilium-sysdump-out]
failed to create sysdump collector: failed to detect Cilium namespace: Get "http://localhost:8080/api/v1/namespaces/kube-system": dial tcp [::1]:8080: connect: connection refused

Happens in the below as well
PR: https://github.com/cilium/cilium/pull/25711
Run: https://github.com/cilium/cilium/actions/runs/5161149871/attempts/2

seems like I am not lucky enough, 5 failures and counting
https://github.com/cilium/cilium/actions/runs/5161149871

Looks like a complexity regression in the verifier, I see
good run: 5.15.0-1036-azure
bad run: 5.15.0-1038-azure
And as GH is slowly updating their fleet, we now probably end up on bad kernels more often and fail consistently.


About page missing

There currently is a link to an About page in the footer, which does not yet exist.
Do we want a general About page to track some of the narrative history of the project? We could include all the GitHub contributors there as well, if we wanted to.

Potentially relevant links from other projects where I've collected the contributors, in case we want to do this.

https://github.com/chartgerink/ai-covenant/blob/main/.github/workflows/contributors.yml
https://github.com/epiverse-trace/epiverse-trace.github.io/blob/main/people.qmd


Add script to find case information

This script scrapes Virginia’s District Court case information system in an effort to deanonymize the case data in the cases directory.
It works by using the case information system’s hearing date search to look for all cases that were finalized on the anonymous case’s FINAL_DISP_DATE field. It then loads the details page for each case found. If the case shares CASE_TYPE, FILE_DATE, and OFFENSE_CLASS_CODE, it is considered a match for the anonymous case.
All matches are written to a new CSV file that looks just like the original with CASE_NUMBER and NAME fields added.

This script takes a while to run. Typically a few minutes per case. Here is the first bit of data I got from it. Let's verify that the data is good and the script is working correctly before the pull request is accepted.
FIPS,CASE_TYPE,ATTORNEY_TYPE,FILE_DATE,OFFENSE_CLASS_CODE,FINAL_DISP_DATE,CASE_NUMBER,NAME,MATCH
3,F,N,3/31/14,6,3/31/14,GC14003789-00,"RITTENHOUSE, AARON W.",1
9,F,N,2/24/14,O,4/3/14,GC14001574-00,"RAGLAND, ANTHONY WAYNE",1
11,F,N,5/29/14,5,7/30/14,GC14001544-00,"SANGIACOMO, MARY LOUISE",1
13,F,N,3/4/13,6,6/10/14,GC13000993-00,"LYONS, JONATHAN",1
13,F,N,10/7/13,5,5/2/14,,,
13,F,N,12/12/13,O,1/23/14,GC13005828-00,"JONES, DEMONTE ANTHONY",1
13,F,N,12/23/13,U,1/10/14,GC13005964-00,"BROWN, CORNELL ROBERT",1
13,F,N,12/23/13,6,4/17/14,GC13005979-00,"DEVITO PISCICELLI, DANTE",1
13,F,N,12/23/13,6,4/17/14,GC13005974-00,"OBERHEIM, ADAM DOUGLAS",2
13,F,N,12/23/13,6,4/17/14,GC13005975-00,"OBERHEIM, ADAM DOUGLAS",3
13,F,N,12/23/13,6,4/17/14,GC13005980-00,"OBERHEIM, ADAM DOUGLAS",4
13,F,N,12/23/13,6,4/17/14,GC13005979-00,"DEVITO PISCICELLI, DANTE",1
13,F,N,12/23/13,6,4/17/14,GC13005974-00,"OBERHEIM, ADAM DOUGLAS",2
13,F,N,12/23/13,6,4/17/14,GC13005975-00,"OBERHEIM, ADAM DOUGLAS",3
13,F,N,12/23/13,6,4/17/14,GC13005980-00,"OBERHEIM, ADAM DOUGLAS",4
13,F,N,12/23/13,6,4/17/14,GC13005979-00,"DEVITO PISCICELLI, DANTE",1
13,F,N,12/23/13,6,4/17/14,GC13005974-00,"OBERHEIM, ADAM DOUGLAS",2
13,F,N,12/23/13,6,4/17/14,GC13005975-00,"OBERHEIM, ADAM DOUGLAS",3
13,F,N,12/23/13,6,4/17/14,GC13005980-00,"OBERHEIM, ADAM DOUGLAS",4
13,F,N,1/2/14,2,1/2/14,GC13006123-00,"LINK, JAMES THOMAS",1
13,F,N,1/2/14,2,1/2/14,GC13006138-00,"MCNEAL, JAMES LARRY",2
13,F,N,1/2/14,2,1/2/14,GC13006124-00,"STODDARD, ALFONSO",3
13,F,N,1/2/14,2,1/2/14,GC13006123-00,"LINK, JAMES THOMAS",1
13,F,N,1/2/14,2,1/2/14,GC13006138-00,"MCNEAL, JAMES LARRY",2
13,F,N,1/2/14,2,1/2/14,GC13006124-00,"STODDARD, ALFONSO",3
13,F,N,1/2/14,2,1/2/14,GC13006123-00,"LINK, JAMES THOMAS",1
13,F,N,1/2/14,2,1/2/14,GC13006138-00,"MCNEAL, JAMES LARRY",2
13,F,N,1/2/14,2,1/2/14,GC13006124-00,"STODDARD, ALFONSO",3
13,F,N,1/27/14,U,2/3/14,GC14000325-00,"GILMARTIN, LAWRENCE PATRICK",1
13,F,N,2/21/14,U,3/4/14,GC14000716-00,"MORRISON, CHARMECE NICOLE",1
13,F,N,3/4/14,U,3/11/14,GC14000841-00,"HICKMAN, ERIC DEWAYNE",1

I'm also wondering if we want to post the results to the repo or leave them out to keep the data publicly anonymous.

This script takes a while to run. Typically a few minutes per case.

Is that accurate?! If so, it's going to take a really long time to get every case in, then. For example if there are 10,000 cases and each case takes 3 minutes to load, then it will take three weeks of nonstop crunching to get every case downloaded.

@fj Yep. It's accurate. Smaller courts finish in just a few seconds, but the larger courts take a while. They can hear hundreds of cases in a day. We have to make a request to the server for each case that gets finalized and we have to rate limit our requests to one a second or the system locks us out. Luckily, there are less than 2,000 cases in the data in this repo. We should be able to get that done in a couple days. And the script only needs to run once a year.

I accepted the pull request despite not knowing if the script is working properly because I like to live dangerously. ;)
I'm going to give this a whirl!

I got an immediate error:
Partial case 1 of 486
	Searching Albemarle General District Court for cases on 03/31/2014
Traceback (most recent call last):
  File "./process-cases.py", line 94, in <module>
    cases = reader.get_cases_by_date(fips_code, search_date)
  File "~/attorney-analysis-master/courtreader/readers.py", line 34, in get_cases_by_date
    cases.extend(districtcourtparser.parse_hearing_date_search(soup))
  File "~/attorney-analysis-master/courtreader/districtcourtparser.py", line 24, in parse_hearing_date_search
    'status': list(cells[6].stripped_strings)[0]
IndexError: list index out of range

I assume that means that the HTML doesn't match what is expected.

Yeah. That's strange. Let me add some better debug stuff so I can't take a
look at your issue.
On Saturday, March 7, 2015, Waldo Jaquith<EMAIL_ADDRESS>wrote:

I got an immediate error:
Partial case 1 of 486
Searching Albemarle General District Court for cases on 03/31/2014
Traceback (most recent call last):
File "./process-cases.py", line 94, in
cases = reader.get_cases_by_date(fips_code, search_date)
File "~/attorney-analysis-master/courtreader/readers.py", line 34, in get_cases_by_date
cases.extend(districtcourtparser.parse_hearing_date_search(soup))
File "~/attorney-analysis-master/courtreader/districtcourtparser.py", line 24, in parse_hearing_date_search
'status': list(cells[6].stripped_strings)[0]
IndexError: list index out of range
I assume that means that the HTML doesn't match what is expected.
—
Reply to this email directly or view it on GitHub
https://github.com/openva/attorney-analysis/pull/4#issuecomment-77725293
.

Just submitted #5. Give that a try. I fixed one thing in the parser, but I doubt it will fix your problem. If you get the exception again, send me the unexcepted_output.html and the stack trace (if its any different).

Stack trace is the same. Here's the output.
https://gist.github.com/waldoj/1e8c71003c5546875235

Ah, now it's working better. :)

These results certainly look good to me!
FIPS,CASE_TYPE,ATTORNEY_TYPE,FILE_DATE,OFFENSE_CLASS_CODE,FINAL_DISP_DATE,CASE_NUMBER,NAME,MATCH
3,F,N,3/31/14,6,3/31/14,GC14003789-00,"RITTENHOUSE, AARON W.",1
9,F,N,2/24/14,O,4/3/14,GC14001574-00,"RAGLAND, ANTHONY WAYNE",1
11,F,N,5/29/14,5,7/30/14,GC14001544-00,"SANGIACOMO, MARY LOUISE",1
13,F,N,3/4/13,6,6/10/14,GC13000993-00,"LYONS, JONATHAN",1

What does it look like for a case with no attorney?
On Sun, Mar 8, 2015 at 10:22 PM, Waldo Jaquith<EMAIL_ADDRESS>wrote:

These results certainly look good to me!
FIPS,CASE_TYPE,ATTORNEY_TYPE,FILE_DATE,OFFENSE_CLASS_CODE,FINAL_DISP_DATE,CASE_NUMBER,NAME,MATCH
3,F,N,3/31/14,6,3/31/14,GC14003789-00,"RITTENHOUSE, AARON W.",1
9,F,N,2/24/14,O,4/3/14,GC14001574-00,"RAGLAND, ANTHONY WAYNE",1
11,F,N,5/29/14,5,7/30/14,GC14001544-00,"SANGIACOMO, MARY LOUISE",1
13,F,N,3/4/13,6,6/10/14,GC13000993-00,"LYONS, JONATHAN",1
—
Reply to this email directly or view it on GitHub
https://github.com/openva/attorney-analysis/pull/4#issuecomment-77790940
.

You've done it! I can't thank you gentlemen enough. Let me know when you can send me a complete list.

What does it look like for a case with no attorney?

In the HTML, it's just a blank cell. I assume @bschoenfeld is detecting those to output the N value for an attorney.
Ben, do you have this running somewhere, or should I fire up an EC2 Micro instance and leave it running for a few days? :)

Rad; once you get the other half of this data I'll be happy to finish things up in #1!

What does it look like for a case with no attorney?

@fj @waldoj I'm not doing anything with the attorney field. It's always N in data in this repo and I see a field on the case information site for attorney, but I've never seen anything in it. Should I be doing something with it? I don't understand the original question.
@waldoj Because the site tends to kick me out every night (and sometimes during the day), I usually just run locally so I can keep an eye on it. I've got about 20% of 2014 done. You can try to do another year on EC2 - you have to change the date in the script here.

Great—I'm running 2013 now.

84 of 322 cases matched.

248 of 322 cases matched.

Done with 2014. Starting 2012.


Stackable Parameter Radar Upgrade

Current Parameter Radar implementation only supports a single data point using Radar Sector plotting approach:

[ ] Upgrade so that the parameter radar is "stackable", ie.. when engaged each subsequent new data point is added to the chart instead of replacing so users can visualize differences.
[ ] Add toggle button to engage stacking feature
[ ] Add clear all button to clear any existing stacked vectors
[ ] Upgrade Parameter chart labels to draw from new Dimension labeling system when available
[ ] Fix plotting so that Radar polygon options work. (may need to upgrade Charts library version)
[ ] Upgrade Projections3D pane so Parameter RADAR is enabled for this view. Allow user to click on projected points and see the hyper-dimensional data associated with it.

Latest Charts release is 17.1.43
https://github.com/HanSolo/charts


update appveyor

Use VS 2019, build in x64 (only)
Use ITK 4.13.3

Might fix #59

I've also updated the MacOS github-action here, but that build still fails due to #64

I've also updated the MacOS github-action here, but that build still fails due to #64


Unable to use request options with response model enabled

Hi,
what I am trying to do: re-use openai client and change its options by passing it in the request options section with response_model enabled
unfortunately the TS throws error saying we cant use response_model
how to use request options with response_model enabled ?
sample code:
import { openai } from "../utils/openai";

const instructorJsClient = Instructor({
  client: openai,
  mode: "MD_JSON",
});

const UserSchema = z.object({
  age: z.number().describe("The age of the user"),
  name: z.string(),
});

async function callOpenAiWithInstructor() {
  const response = await instructorJsClient.chat.completions.create(
    {
      messages: [{ role: "user", content: "Jason Liu is 30 years old" }],
      model: "gpt-3.5-turbo",
      response_model: {
        schema: UserSchema,
        name: "User",
      },
    },
    {
      timeout: 30000,
    }
  );

  return response;
}

error screenshot:

it works if we not pass the request options (here: timeout )

I am experienceing the same issue

👀


Added the OpenSSF Scorecard Badge

This PR adds OpenSSF Scorecard Badge in the README file.
Hi, I'm Harshita. I’m working with CNCF and the Google Open Source Security Team for the GSoC 2024 term. We are collaborating to enhance security practices across various CNCF projects. The goal is to improve security for all CNCF projects by both using OpenSSF Scorecards and implementing its security improvements.
The project already has the Scorecard GitHub Action running, so adding the scorecard badge would be quite handy. So that the results can be shown as a badge in the repository's README. This badge serves as a quick indicator of the project's security posture, helping users and contributors evaluate the project's security practices quickly.

We didn't have this added previously because the scoring system was buggy. It seems much of that is fixed but still see a couple errors in the score based on https://scorecard.dev/viewer/?uri=github.com/containerd/containerd. We have already followed the guidance for "Token-Permissions" yet the score is 0 and does not match the documentation remediation steps. How is it supposed to check branch protection, we have that in place but not something that would be accessible to view.

Hi

We have already followed the guidance for "Token-Permissions" yet the score is 0 and does not match the documentation remediation steps

The issue here is that a single workflow https://github.com/containerd/containerd/blob/main/.github/workflows/links.yml doesn’t have a top-level read-only permissions block like present in the other workflows. Scorecard is quite severe in this check: a single workflow without top-level permissions gets a 0/10 for the check.

How is it supposed to check branch protection, we have that in place but not something that would be accessible to view.

Branch-Protection doesn’t work when running the Scorecard Action. Or, rather, you’d need to feed it a GitHub Personal Access Token (PAT), and the risks should the PAT ever leak are greater than the benefit they’d get from having the Action run the check, so Scorecard itself recommends against giving a PAT.
Currently, the score is 7.3. As my next step, I will create a separate scorecard score improvement issue and fix each check where the score is dropping, thus improving the project's security posture.

Somewhat orthogonal to the PR itself, but I had a look at the ScoreCard results; most notably "pinning dependencies"; I went through some of the warnings it printed - summary below
Actions
I see it flags actions from within the containerd org as "third-party";
Warn: third-party GitHubAction not pinned by hash: .github/workflows/ci.yml:56: update your workflow using https://app.stepsecurity.io/secureworkflow/containerd/containerd/ci.yml/main?enable=pin

I wonder if such cases could be considered allowed, as they are part of the same organisation (and likely to be considered "trusted", or scored differently);
https://github.com/containerd/containerd/blob/919beb1cf7d4edda26a3b95bf1b0d83b178e995e/.github/workflows/ci.yml#L56
Not sure how these are scored, but I'm still a bit on the fence on these; or at least I would generally consider GitHub-owned actions to be "trusted". Pinning those dependencies would introduce the risk of running outdated (unpatched) versions - more so taking release branches into account - or introduce code-churn (dependabot updating the digest, and very unlikely for maintainers to review changes in-depth for a trusted (GitHub-owned) workflow);
Warn: GitHub-owned GitHubAction not pinned by hash: .github/workflows/nightly.yml:23: update your workflow using https://app.stepsecurity.io/secureworkflow/containerd/containerd/nightly.yml/main?enable=pin

Go commands
This seems like a bug; or at least, I would not expect the scorecard to include issues found in dependencies it should probably be made aware of vendor/ directories are external to the project (maybe some .gitattributes if needed?);
Warn: goCommand not pinned by hash: vendor/github.com/json-iterator/go/build.sh:10
Warn: goCommand not pinned by hash: vendor/google.golang.org/grpc/regenerate.sh:35

It looks like it complains about these two dependencies being installed without a patch version specified;
Warn: goCommand not pinned by hash: script/setup/install-dev-tools:27
Warn: goCommand not pinned by hash: script/setup/install-dev-tools:28

The warning looks to be legit (so :+1:) on that part, but is a bit confusing as it mentions "hash", but it looks like it's ok if a full version (including patch version) is used, or at least, it doesn't complain about other lines in the same file thatt include a minor version https://github.com/containerd/containerd/blob/919beb1cf7d4edda26a3b95bf1b0d83b178e995e/script/setup/install-dev-tools#L26

I opened a PR to address those: https://github.com/containerd/containerd/pull/10604

Dockerfiles
Detection currently doesn't support non-Linux Dockerfiles, and is failing to handle those correctly (it's logged as Info so probably doesn't affect score);
Info: Possibly incomplete results: error parsing shell code: a command can only contain words and redirects; encountered (: integration/images/volume-copy-up/Dockerfile_windows:31-33
Info: Possibly incomplete results: error parsing shell code: a command can only contain words and redirects; encountered (: integration/images/volume-ownership/Dockerfile_windows:32-35

It looks like detecting images in Dockerfiles is a bit of a hit-and-miss;
Warn: containerImage not pinned by hash: .devcontainer/Dockerfile:16: pin your Docker image by updating mcr.microsoft.com/devcontainers/base:1-ubuntu-22.04 to mcr.microsoft.com/devcontainers/base:1-ubuntu-22.04@sha256:f8cfe39b16ce648d49af3739e449668baaec8e20b1c1ca21fc6912f41a673fa5
Warn: containerImage not pinned by hash: .github/workflows/release/Dockerfile:20
Warn: containerImage not pinned by hash: .github/workflows/release/Dockerfile:23
Warn: containerImage not pinned by hash: .github/workflows/release/Dockerfile:36
Warn: containerImage not pinned by hash: .github/workflows/release/Dockerfile:37
Warn: containerImage not pinned by hash: .github/workflows/release/Dockerfile:42

It has trouble understanding build-args, and therefore not able to determine if an image is actually pinned (or not); https://github.com/containerd/containerd/blob/919beb1cf7d4edda26a3b95bf1b0d83b178e995e/.github/workflows/release/Dockerfile#L20
It also considers some stages as images, e.g. these two are referencing another stage. Technically it may be correct that that stage isn't pinned, but not sure if it would be able to detect that case https://github.com/containerd/containerd/blob/919beb1cf7d4edda26a3b95bf1b0d83b178e995e/.github/workflows/release/Dockerfile#L36-L37
I don't know how these are scored example but overall, the same applies here as to trusted (GitHub-owned, or other official ones) workflows; I think pinning official images would risk running unpatched images, or otherwise cause code-churn with little value to pin to new digests.

Oh! I should mention that I briefly discussed the Dockerfile parsing with a colleague (currently on vacation though), to see if we could possibly contribute changes to improve the parsing.

oh, nevermind; looks like "passing" is a level of passing, and could be "silver" and "gold" (confusing though!); https://github.com/ossf/scorecard/blob/230f5a76c9d301a9037b990d1a4c653e759a9f28/docs/checks.md#cii-best-practices


Doesn't respond to current directory changes

Hi. Good package, and things mostly work well, but I've noticed that the rebuilding and/or restarts do not occur if the files changed are in the current directory (i.e. from which templier was run). I have my config referencing ./, but templier doesn't react to any edits to files in ./. This includes the example config change custom builder... not just Go files. If I move things into a subdirectory, then changes to them are picked up.

I did a little debugging which led me to the add() function: https://github.com/romshark/templier/blob/main/internal/watcher/watcher.go#L279
It doesn't watch dir that is passed in... only the child directories.
As a hacky experiment I added this to the top of the function:
	d, err := filepath.Abs(dir)
	if err != nil {
		return err
	}
	if err := w.watcher.Add(d); err != nil {
		return err
	}
	w.watchedDirs[d] = struct{}{}

Then I saw what I think is correct behavior, with edits to root directory files triggering rebuilds, etc. But there is quite a bit going on in the tool, so I may not be understand this all correctly.

I think I found the issue. I used the example config to get started, and it has the line:
  exclude:
    - .* # all hidden files and directories

The issue is that when processing the directories, the relative path for the root directory is ".", which matches .* as an excluded directory. When I changed the exclusion to .?* (mandate at least one character after the dot), things seem to work OK. It may be worth updating the example config if the code is otherwise correct.

Thanks for going the extra steps debugging the issue!
There indeed was a bug and Templier shouldn't have excluded the working directory based on ".*".
This is fixed now https://github.com/romshark/templier/commit/86d16df680a9c80f383ff6b9d1772cbc5ef057cb and released under v0.7.1


[Mystical Worlds] Mob drops not listed in JER

believe JER only works on vanilla mobs from what i have seen

I guess I'm diving into JER to see how difficult that would be to patch in...


Update README.md

Solving an issue with a header.

Does it not duplicates https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/396 ?
Why adding spaces before the snippet?

To be honest, I'm not sure. I couldn't find a reason why the current header doesn't render...

Closing as it duplicates https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/396