The "log" function in "dart:developer" package isn't working in version 3.10.1

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
1- Created new fresh app using the latest Flutter version
2- Created a button to log something in the console
3- Built and ran the app on both iOS and Android and the "log" wasn't working at all
Expected results
The "log" should just work as normal.
Actual results
There's no output when using the "log" or anything else that indicates something related to it.
Code sample
Code sample
import 'package:flutter/material.dart';
import 'dart:developer';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatelessWidget {
  final String title;

  const MyHomePage({super.key, required this.title});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(title),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () => log('Hello World!'),
        tooltip: 'Log Something',
        child: const Icon(Icons.print),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
None

Logs
Logs
None

Flutter Doctor output
Doctor output
❯ flutter doctor -v
[✓] Flutter (Channel stable, 3.10.1, on macOS 13.1 22C5050e darwin-x64, locale en-IQ)
    • Flutter version 3.10.1 on channel stable at /Users/apple/Library/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d3d8effc68 (7 days ago), 2023-05-16 17:59:05 -0700
    • Engine revision b4fb11214d
    • Dart version 3.0.1
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/apple/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • ANDROID_HOME = /Users/apple/.android
    • ANDROID_SDK_ROOT = /Users/apple/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.78.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.64.0

[✓] Connected device (4 available)
    • RMX3085 (mobile)           • 75EE6HHIQ4T89D9D                     • android-arm64  • Android 12 (API 31)
    • iPhone 14 Pro Max (mobile) • 0D139FBF-7232-49FB-AF0E-CBE92FD5C24F • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)            • macos                                • darwin-x64     • macOS 13.1 22C5050e darwin-x64
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome 113.0.5672.126

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi @devmuaz
I checked this on Android but I can see the log displaying as expected. Verifying this on VSCode. Can you tell me which IDE are you testing?  And does it always happen even after reopening IDE/clear cache if any?

Hi @huycozy,
I'm using VSCode IDE and I reopened the editor multiple times but the issue is still the same. Maybe this is something related to DevTools? I'm not quite sure

It doesn't seem to be DevTools. I can see you are using the same version as me DevTools version 2.23.1.
Can you confirm if this option has been checked or not?

And please check this answer that may be helpful.
Moreover, have you tried this on Terminal or Android Studio?

Yes @huycozy, I have that option checked in my settings

And I have tried to run the app on Web from the terminal and literally not logging anything to the terminal

Something isn't right but I can't figure it out!

I check out the answer you provided, and I remembered that I'm running the app without debugging!
It's kinda weird because I used to run apps without debugging and the log() function was working, now it only works when debugging the app.
Thank you so much mate, for the help and time really appreciated 🙏🏻
I'm gonna be closing the issue since I believe it's been solved.


raise JSONDecodeError

Hi, my code is just simply as follows:
import googletrans
from googletrans import Translator
translator = Translator()
translator.translate('bonjour')
I would like to test the package and the text is so short within the limit. But it raise error:
raise JSONDecodeError("Expecting value", s, err.value) from None
and the error message is
JSONDecodeError: Expecting value
Can anybody help me? It is really urgent and I don't know how to deal with it since the text is so short.
Thanks a lot!

@ssut

@josemoralesp @Paethon @vuolter @universeroc

@VionaWang Check out #77, the lib pretty much won't work until #78 is merged. You'll have to make a fork and apply the PR there. (could also borrow mine since it is same thing) https://github.com/stephwag/py-googletrans

Error trace:
/root/py-googletrans/googletrans/client.pyc in translate(self, text, dest, src)
    170
    171         origin = text
--> 172         data = self._translate(text, dest, src)
    173
    174         # this code will be updated when the format is changed.

/root/py-googletrans/googletrans/client.pyc in _translate(self, text, dest, src)
     79         r = self.session.get(url, params=params)
     80
---> 81         data = utils.format_json(r.text)
     82         return data
     83

/root/py-googletrans/googletrans/utils.pyc in format_json(original)
     60         converted = json.loads(original)
     61     except ValueError:
---> 62         converted = legacy_format_json(original)
     63
     64     return converted

/root/py-googletrans/googletrans/utils.pyc in legacy_format_json(original)
     52             text = text[:p] + states[j][1] + text[nxt:]
     53
---> 54     converted = json.loads(text)
     55     return converted
     56

/usr/lib/python2.7/json/__init__.pyc in loads(s, encoding, cls, object_hook, parse_float, parse_int, parse_constant, object_pairs_hook, **kw)
    337             parse_int is None and parse_float is None and
    338             parse_constant is None and object_pairs_hook is None and not kw):
--> 339         return _default_decoder.decode(s)
    340     if cls is None:
    341         cls = JSONDecoder

/usr/lib/python2.7/json/decoder.pyc in decode(self, s, _w)
    362
    363         """
--> 364         obj, end = self.raw_decode(s, idx=_w(s, 0).end())
    365         end = _w(s, end).end()
    366         if end != len(s):

/usr/lib/python2.7/json/decoder.pyc in raw_decode(self, s, idx)
    380             obj, end = self.scan_once(s, idx)
    381         except StopIteration:
--> 382             raise ValueError("No JSON object could be decoded")
    383         return obj, end

ValueError: No JSON object could be decoded

same error, but yesterday it was worked

I just switched to googletrans
Does the same (but just for the google translator) and works

same error, any fix on this?

@stephwag , Hi , Thanks for your advise. But I have hanged and it is the issue that happens after the change. I have tried and found that if I am using WiFi for a long time and it does not work, when I change to other network like ethernet it works again, vice versa. It is strange. Could you help me?

Hey there, have you tried change your network? I don't know why it works for a while but if you change your network it works again. That's the way I fixed it.

it worked for 10 queries, after that it started giving json error

@stephwag is there any fix for json decode error yet?

What I do is to give alternative addresses to use for querying. The system then uses all of them and you don't get blocked as quickly.
translate_urls = ["translate.google.com", "translate.google.co.kr",
                      "translate.google.at", "translate.google.de",
                      "translate.google.ru", "translate.google.ch",
                      "translate.google.fr", "translate.google.es"]
tr = Translator(service_urls=translate_urls)

Fixed library are here https://pypi.org/project/py-translator/


Research Page

Create a page that has links to professor's pages who are accepting undergraduate research assistants.

I made major changes to the research page and added a general outline of what should be on there.
To-Do:

Integrate REU summaries effectively (either docs / HTML versions of the PDFs, integrating PDFs directly looks hard)
Check for typos
Distribute information across multiple pages (?)

Add: Poster information and tips
Otherwise, done.

Finished


golang kubernetes provider is erroneously rendering to yaml directory if directory is unset

Steps to reproduce

Try to create a kubernetes provider and create a resource with it:
	kubeconf := createKubeConfig(gkeCluster)
	clusterProvider, err := kubernetes.NewProvider(d.ctx, "gke-cluster", &kubernetes.ProviderArgs{
		Kubeconfig: kubeconf,
	})
	if err != nil {
		return err
	}

	ns, err := corev1.NewNamespace(d.ctx, "kube-ready", &corev1.NamespaceArgs{
		Metadata:   metav1.ObjectMetaArgs{
			Name:   pulumi.String("kube-ready"),
		},
	}, pulumi.Provider(clusterProvider))
	if err != nil {
		return err
	}
// and so on

When I run pulumi up, instead of applying to the cluster, it creates yaml files in null/1-manifest folder. The problem is the RenderToYamlDirectory check is not considering a nil value as unset. As a workaround, I can set RenderToYamlDirectory to "" which fixes it.
On pulumi v3.10.3.
Expected: kubernetes provider works by just setting the kubeconfig field
Actual: it erroneously renders to yaml in the local filesystem

Sorry! Wrong repo.


IPython support for jupyter-notebook gitignore files

Should be jupyter-notebook gitignore files have IPython support?

@gvvynplaine this seems reasonable, given the information on the site:

As of IPython 4.0, the language-agnostic parts of the project: the notebook format, message protocol, qtconsole, notebook web application, etc. have moved to new projects under the name Jupyter. IPython itself is focused on interactive Python, part of which is providing a Python kernel for Jupyter.

@shiftkey I think that it is reasonable too. Thank you for your cooperation.


Request List : Columns needs to be wrapped

Subject of the issue
Columns is not getting wrapped, not fitting in one view
Your environment

version of your python
which browser and its version
OS Type (iOS/Android/Linux/Windows)

Steps to reproduce
Tell us how to reproduce this issue. Please provide a working demo as a GIF, if possible.

Expected behaviour
Should be able to view without scrolling horizontally.
Actual behaviour
Need to scroll to view important  details

This issue can be closed I believe.


dial tcp <IP_ADDRESS>:443: i/o timeout when upgraded to 1.23 with containerd

I upgraded eks to 1.23 today and updated ami with containerd runtime --container-runtime containerd.
However, my pods can't be startup, one of my pod reports error like this:
dial tcp <IP_ADDRESS>:443: i/o timeout

But I can telnet 443 port from this node:
ubuntu@ip-10-120-28-180:~$ telnet <IP_ADDRESS> 443
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.

Please help, thanks.
EKS version: 1.23
Tag of AMI repo referenced: v20230217
VPC CNI Plugin version: v1.12.5-eksbuild.1
AWS Region: cn-northwest-1

ok, I fixed this.
Previously I used Ubuntu 22.04 and containerd version 1.6.19, I rolled back to Ubuntu 20.04 and containerd 1.6.6, everything works now.


fibjs fails to build on Ubuntu 14.04

See https://gist.github.com/500ef570e019404f9ad6
Message:

Error: No such file or directory - bin/Darwin_Release/fibjs

I'm not planning on working on this right this second, but I might get to it sooner or later.

Actually, looks like the fix is probably trivial. Will submit it as soon as the build's done.


能否增加一个保存原神设置的功能?

Checklist

[X] I have already read docs/Tips.md, but my feature is not implemented.
[X] My suggested feature was not mentioned by others, and it is not a duplicate feature.

Summary
原神重新安安装后按键什么的都需要再次设置一次,能不能保存原神的设置,再次安装后释放原神的设置?
Solution or Design

游戏设置都存储在注册表中，你只需要卸载游戏时取消勾选清理注册表即可。


Reduce the namespace manager interface to the essentials

All other methods are moved into utility methods, as their implementations are shared

Closed in favor #581


Android issues after switching to a different Priority Plan

I have successfully been building and running an iOS and android app, using a Priority Plan tied to my personal Expo user.
After creating an Expo organisation and adding it as expo.owner in app.json, I was unable to use EAS Build, as there was no Priority Plan tied to the organisation.
I then created a Priority Plan on the organisation, and was able to successfully create iOS and android builds to the organisation using EAS Build. After doing so, the iOS app works, but the android app doesn't, it crashes on startup.
If I remove the expo.owner from app.json and build to my personal Expo user, both apps still work.
Do EAS have some sort of cache I need to clean?

Just to expand a bit on the errors I get in adb logcat, which I cant wrap my head around.
I'm running a "bare" app, or "generic" as it is called in the eas config. I can build and run the android app locally, and as mentioned previously, I can also run it, if I build it on my personal Expo account. The issue only occurs when I add an expo.owner to app.json.
My initial error was about a library that wasn't linked correctly. Removing it completely from the app made the next build complain about some other library.
Adding them all back and removing the expo.owner from app.json fixes everything, but I don't want to build to a personal account.
Comparing the output of the failing and succeeding builds, this is the first difference in the failing build:
...
06-17 01:14:19.299 18707 18707 V fb-UnpackingSoSource: deps mismatch on deps store: regenerating
06-17 01:14:19.299 18707 18707 V fb-UnpackingSoSource: so store dirty: regenerating
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libc++_shared.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libfb.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libfolly_futures.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libfolly_json.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libglog.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libglog_init.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libimagepipeline.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libjsc.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libjscexecutor.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libjsijniprofiler.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libjsinspector.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libnative-filters.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libnative-imagetranscoder.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libreactnativeblob.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libreactnativejni.so: deferring to libdir
06-17 01:14:19.308 18707 18707 D ApkSoSource: not allowing consideration of lib/arm64-v8a/libyoga.so: deferring to libdir
...

hi there! sorry i didn't see this sooner. the owner field certainly does not have an impact on the build or link phase of your app. could it be that your app is somehow coupled to the username? or perhaps there was an unrelated change that you forgot about when removing the owner field?

Closing because of the inactivity.


Deploy failing

We're suddenly seeing an error when deploying - here's the output from GHA:
(node:1943) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)
/home/runner/work/_actions/deployphp/action/v1/index.js:1
import core from '@actions/core'
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Object.compileFunction (node:vm:352:18)
    at wrapSafe (node:internal/modules/cjs/loader:1031:15)
    at Module._compile (node:internal/modules/cjs/loader:1065:27)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:17:47

I guess it was caused by this commit: https://github.com/deployphp/action/commit/ba36add8b14331e5c5f614b5c41bd11cfb0d6eb4
I can supply more our workflow / deployment script if needed.

Should be fixed.

hot fix:

 - name: Deploy
   uses<EMAIL_ADDRESS>

@antonmedv latest release pointing to a non-working version 1.0.18 (as 1.0.12)

Should be fixed.

Now getting another error;
Run deployphp/action@v1
node:internal/errors:464
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'zx' imported from /home/runner/work/_actions/deployphp/action/v1/index.js
    at new NodeError (node:internal/errors:371:5)
    at packageResolve (node:internal/modules/esm/resolve:884:9)
    at moduleResolve (node:internal/modules/esm/resolve:929:18)
    at defaultResolve (node:internal/modules/esm/resolve:1044:11)
    at ESMLoader.resolve (node:internal/modules/esm/loader:422:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:222:40)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:76:40)
    at link (node:internal/modules/esm/module_job:75:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}

Will try @bes-internal's suggestion of locking to an earlier specific version for now.

Should be fixed now.

We need some tests)

Same for me. Exisiting deployment isn't running anymore. Seems like the $'deploy prod' isn't processed (correct)

Is it possible that you set https://github.com/deployphp/action/releases/tag/v1.0.17 as the latest release again instead 1.0.12

Same here :(

Same here :(

Can't tell from this error message what is the problem.

@antonmedv
$ sudo chmod +x deployer.phar
Invalid JSON in options
$ php deployer.phar --no-interaction --ansi -v $'deploy prod'

In Application.php line 709:

  [Symfony\Component\Console\Exception\CommandNotFoundException]
  Command "deploy prod" is not defined.

Exception trace:
  at phar:///home/runner/work/Project/Project/deployer.phar/vendor/symfony/console/Application.php:709
 Symfony\Component\Console\Application->find() at phar:///home/runner/work/Project/Project/deployer.phar/vendor/symfony/console/Application.php:259
 Symfony\Component\Console\Application->doRun() at phar:///home/runner/work/Project/Project/deployer.phar/vendor/symfony/console/Application.php:171
 Symfony\Component\Console\Application->run() at phar:///home/runner/work/Project/Project/deployer.phar/src/Deployer.php:317
 Deployer\Deployer::run() at phar:///home/runner/work/Project/Project/deployer.phar/bin/dep:96
 require() at /home/runner/work/Project/Project/deployer.phar:4

Error: Failed: dep deploy prod

I suggest @malteschlueter I change version to 1.0.17 and it's working as previously

Hello, had to revert to 1.0.17
This was the error
Run deployphp/action@v1
with:
private-key: ***
dep: deploy dev
skip-ssh-setup: false
ansi: true
verbosity: -v
$ ssh-agent -a /tmp/ssh-auth.sock
SSH_AUTH_SOCK=/tmp/ssh-auth.sock; export SSH_AUTH_SOCK;
SSH_AGENT_PID=1729; export SSH_AGENT_PID;
echo Agent pid 1729;
$ ssh-add -
Identity added: (stdin) (admin)
Error: Cannot read properties of undefined (reading 'version')

@ashwin-nath-m I can confirm this same problem none of our actions are working anymore because of v1.0.12 being tagged as v1

Same as for @ashwin-nath-m happens to me:
$ ssh-agent -a /tmp/ssh-auth.sock
SSH_AUTH_SOCK=/tmp/ssh-auth.sock; export SSH_AUTH_SOCK;
SSH_AGENT_PID=1903; export SSH_AGENT_PID;
echo Agent pid 1903;
$ ssh-add -
Identity added: (stdin) (*******)
Error: Cannot read properties of undefined (reading 'version')

I have the same problem as for @ashwin-nath-m
Error: Cannot read properties of undefined (reading 'version')
Revert to 1.0.17 help for me

Any updates on this problem? im having the same issue

@knewitzgui I noticed that you are running deployphp/action@master. Do you still have this issue when running deployphp/action@v1? v1 is now pointing to a release that works for us.

thanks for replying, yes, i tried with v1, master, v1.0.12, and others, nothing seems to work.

i will leave the config of the deploy so u can see what i am doing, and if u have any sugestions on how to solve this.


feat: add "from" field in the transaction_executed event

Feature Request
Describe the Feature Request
In the Kakarot RPC track, we need the "from" field accessible easily.
It saves a lot of storage (DB) and compute to include "from" (the Ethereum address of the sender of a tx) in the transaction executed event
Just add a field in the event, and it is "evm_address.read()"

All good! Can use msg_hash and ecrecover using r,s,v


Issue with max_buffer_bytesize & Kafka::MessageSizeTooLarge

If this is a bug report, please fill out the following:

Version of Ruby: ruby 2.6.2p47 (2019-03-13 revision 67232) [x86_64-linux]
Version of Kafka: https://www.apache.org/dist/kafka/2.2.0/kafka_2.12-2.2.0.tgz
Version of ruby-kafka: master

Please verify that the problem you're seeing hasn't been fixed by the current master of ruby-kafka.
Steps to reproduce
Sorry about the odd setup, but I took the number + size of messages from an error we ran into in production and dropped them in the test and scrubbed them to recreate the specific issue we ran into in production.
$:.unshift File.join(File.dirname(__FILE__), 'lib')

require "kafka"

lengths = [3064, 2424, 2700, 2660, 2468, 2660, 2532, 2488, 3044, 2444, 2660, 2424, 2660, 3364, 2508, 3532, 2532, 3300, 2488, 2572, 2700, 3276, 2508, 2424, 2788, 2508, 2424, 2468, 2532, 3404, 2596, 3084, 2596, 2828, 2444, 2488, 2660, 2212, 3940, 3556, 2636, 2660, 2424, 2424, 2424, 2596, 2424, 2444, 2444, 4004, 3940, 3320, 3128, 2872, 2744, 2572, 2828, 2680, 2468, 2468, 2532, 2744, 2936, 2660, 2488, 2552, 2636, 2872, 2660, 2424, 2508, 2724, 2488, 2552, 2468, 2424, 2744, 2468, 3256, 2552, 2552, 2468, 2508, 2444, 2444, 2252, 3064, 2552, 2444, 2532, 3064, 2468, 2508, 2572, 2488, 2724, 2636, 2744, 2552, 2532, 3640, 2552, 2468, 2444, 2468, 2572, 2872, 2916, 2892, 2980, 2744, 2596, 2424, 2636, 2488, 3064, 4132, 2488, 2680, 2552, 3084, 2764, 3256, 2596, 2616, 2744, 2444, 2616, 7544, 2488, 2444, 2532, 2552, 2444, 2444, 2892, 2340, 2424, 2852, 2552, 2788, 2572, 2572, 2552, 2700, 2444, 2444, 2232, 3340, 2508, 2660, 3852, 2808, 2488, 3084, 3468, 2828, 2680, 2360, 2808, 2424, 2636, 2616, 2552, 3020, 2572, 2596, 2424, 2468, 2680, 2508, 2596, 2424, 2552, 3064, 3064, 3364, 3428, 2532, 2444, 2468, 2488, 2552, 2828, 2724, 2424, 2468, 3064, 3364, 3320, 2508, 2188, 2700, 2744, 2532, 3084, 2424, 2616, 2660, 2552, 2468, 2424, 2468, 2700, 2596, 2424, 2444, 2724, 2724, 3108, 2276, 2660, 2424, 2444, 3128, 5644, 2508, 3128, 2424, 2468, 2552, 2444, 2040, 2724, 4044, 2828, 2700, 2532, 2552, 2424, 2468, 2488, 2724, 3128, 2744, 2596, 2552, 2444, 2532, 3148, 2788, 2660, 2764, 2764, 2596, 2636, 2444, 2424, 2532, 2488, 2764, 2444, 2660, 2148, 2660, 2636, 2508, 2700, 2552, 3428, 2616, 2444, 3084, 2188, 2424, 2764, 2508, 3620, 2488, 2508, 3300, 3340, 2596, 2508, 2552, 2040, 3064, 2956, 2488, 2532, 2552, 2552, 2424, 2468, 2468, 2444, 2444, 2828, 2808, 2572, 2488, 2572, 2444, 2552, 2508, 2872, 2424, 2660, 2916, 2532, 2488, 3064, 2660, 2572, 2488, 2488, 2424, 2616, 2636, 2764, 2744, 2700, 2552, 2724, 2424, 2444, 2636, 2660, 2660, 2488, 2424, 2788, 4172, 2424, 2660, 2700, 3108, 2852, 3148, 2572, 2488, 2444, 2444, 2596, 2596, 2636, 2424, 3404, 2572, 3512, 2532, 2680, 2444, 2508, 3108, 3108, 2572, 2956, 2424, 2424, 2468, 3212, 2700, 2828, 2572, 2424, 2872, 2744, 2508, 2572, 2508, 2552, 2552, 2488, 2872, 2700, 2488, 2616, 2660]

messages = lengths.map {|length| "0" * length}

kafka = ::Kafka.new(seed_brokers: "kafka://<IP_ADDRESS>:9092")
producer = kafka.producer(max_buffer_bytesize: 1_000_000)

messages.each do |message|
  producer.produce(message, topic: "topic", partition: 0)
end

puts "producer.buffer_bytesize: #{producer.buffer_bytesize}"

producer.deliver_messages

Expected outcome
What you thought would happen when running the example.
-> % ruby issue.rb
Traceback (most recent call last):
	4: from issue.rb:12:in `<main>'
	3: from issue.rb:12:in `each'
	2: from issue.rb:13:in `block in <main>'
	1: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/producer.rb:210:in `produce'
/home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/producer.rb:520:in `buffer_overflow': Cannot produce to topic, max buffer bytesize (1000000 bytes) reached (Kafka::BufferOverflow)

Actual outcome
What actually happened.
-> % ruby issue.rb
producer.buffer_bytesize: 998072
Traceback (most recent call last):
	20: from issue.rb:18:in `<main>'
	19: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/producer.rb:249:in `deliver_messages'
	18: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/instrumenter.rb:23:in `instrument'
	17: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/producer.rb:256:in `block in deliver_messages'
	16: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/producer.rb:393:in `deliver_messages_with_retries'
	15: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/producer.rb:393:in `loop'
	14: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/producer.rb:405:in `block in deliver_messages_with_retries'
	13: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/produce_operation.rb:53:in `execute'
	12: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/instrumenter.rb:23:in `instrument'
	11: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/produce_operation.rb:62:in `block in execute'
	10: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/produce_operation.rb:105:in `send_buffered_messages'
	 9: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/produce_operation.rb:105:in `each'
	 8: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/produce_operation.rb:133:in `block in send_buffered_messages'
	 7: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/produce_operation.rb:144:in `handle_response'
	 6: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/protocol/produce_response.rb:34:in `each_partition'
	 5: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/protocol/produce_response.rb:34:in `each'
	 4: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/protocol/produce_response.rb:35:in `block in each_partition'
	 3: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/protocol/produce_response.rb:35:in `each'
	 2: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/protocol/produce_response.rb:36:in `block (2 levels) in each_partition'
	 1: from /home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/produce_operation.rb:153:in `block in handle_response'
/home/rbriggs/src/github.com/zendesk/ruby-kafka/lib/kafka/protocol.rb:160:in `handle_error': Kafka::MessageSizeTooLarge (Kafka::MessageSizeTooLarge)

Explanation
We have hit Kafka::MessageSizeTooLarge in the past because of partition imbalance.  There is a kafka configuration message.max.bytes which limits the maximum byte size per produce request, the default of which is 1000012 according to the documentation.  To try and prevent this we set our max buffer size to 1_000_000 to get under this kafka server limit.
We recently received this error again because the producer.buffer_bytesize does not match the actual byte size that is submitted to kafka.  It took some time to figure this out because almost all the time compression kept us under this limit, we just happened to hit a combination of data that did not compress well enough.  I took the compression out of the example to keep things simpler.
Ideally the buffer_bytesize would match the byte size of the message that would go to kafka.  Right now we trap Kafka::BufferOverflow errors and flush, but since the byte size in the example is already too large we cannot successfully flush the buffer.  We have fixed this temporarily by reducing max_buffer_bytesize by the max_buffer_size and a constant, but it is not a great heuristic.

Can you take a stab at a more accurate heuristic for the actual message size?


[@azure/openai] onResponse hook does not fire for listChatCompletions

Package Name: @azure/openai
Package Version: 1.0.0-beta.8
[ ] nodejs

version: v21

Describe the bug
The onResponse hook is not fired when requesting a streamed response (via server sent events) with listChatCompletions. It works correctly for getChatCompletions. The intent is to inspect the rate limiting headers of Azure OpenAI responses.
To Reproduce
Steps to reproduce the behavior:

Follow the guidance here to generate a streaming chat completion
Add an onResponse callback to the listChatCompletions call that logs something to the console
Nothing is logged to the console

Expected behavior
The onResponse callback is fired and logs to the console. If you change the code above to getChatCompletions instead of listChatCompletions it will work. For my use case, it is ok for the onResponse to fire when the connection is closed
Code example
Working:
  const events = await client.getChatCompletions(
    deploymentId,
    messages,
    {
      maxTokens: 128,
      onResponse: (rawResponse) => {
         console.log(rawResponse);
      }
    });

Not Working:
  const events = await client.listChatCompletions(
    deploymentId,
    messages,
    {
      maxTokens: 128,
      onResponse: (rawResponse) => {
         console.log(rawResponse);
      }
    });

Additional context
Open to other ideas on how to view these header values when streaming responses. Do I need to write a custom pipeline policy? Headers of interest include:
x-ratelimit-remaining-requests
x-ratelimit-remaining-tokens
Azure-Openai-Deployment-Utilization

v1.0.0-beta.10 has been released and it has a fix to this issue. Please let us know if you have any other feedback!


Little endian/big endian

Hello
I'm a  noob in hashing function. but i was wondering  that it seems in your implementation you have some operations for changing the little-endian/big endian,  I was wondering if it was only to be compatible with murmur2, or if it was absolutely for the hash function to work as expected (i.e if not necessary, would be it to possible to have a hash function without them,  just that it will not be "murmur2" )

it seems in your implementation you have some operations for changing the little-endian/big endian

Which operations are you referring to? Point me to the line of code.
Thanks!

Yup sorry to not have included that ^^
https://github.com/aviddiviner/go-murmur/blob/master/murmur2.go#L38
but actually now that I ready it carefully , it's not what I thought (i thought it was byte reordering, as in C implementation one can directly cast at his own risk the 4 bytes into one uint64 )

This might give you some clarity:
data := []byte("Hello, playground")
fmt.Printf("%b %b %b %b\n", data[3], data[2], data[1], data[0])
// 1101100 1101100 1100101 1001000

k := uint32(data[0]) | uint32(data[1])<<8 | uint32(data[2])<<16 | uint32(data[3])<<24
fmt.Printf("%b\n", k)
//<PHONE_NUMBER>011000110010101001000

https://play.golang.org/p/8i6cSEObrP
Just making a 32-bit int by shifting and OR-ing four 8-bit bytes.
HTH


Question, username - password, please help..

Please, can someone tell me how can setup in this script to generate user with username and password when loging to openvpn.
because created client now only ask for password, thanks a lot!

You can't :)


Expand examples so they actually write files

For the "download" order types, expand the examples to they create and save the required files. They can be stored locally or on a storageLocation by specifying it in the bank's configuration file.
Signed-off-by: Herman van Hazendonk<EMAIL_ADDRESS>

@nanov Feel free to shoot at these. I just tried to make the examples a bit easier to work with, so they can be used in real life scenarios without many adjustments. Might help new users to get started easier as well. I'll also try to do something similar for the "upload" type of orders and also to update the README with a short "step plan" on how to get started, so it's easier to get started with setting up a connection etc.

@nanov I'm not very happy with the amount of duplication in the code for the various examples. I'm considering simplifying it into way fewer small .js files (1 for regular download, so C52, C53, G52, G53, STA, VMK and some of the other ones, 1 for zipped downloads (C52/C53/G52/G53/G02 zipped) and 1 for uploads such as payment files and direct debit files (CDD, XCT, XG1 etc)). However, I would need some more fancy command-line options and parsing. For example to provide
"-orderType=C52" and -fileName="my fancy filename.xml". I was looking at yargs (https://yargs.js.org/docs/). Would that be OK to add as an dependancy, or you have an alternative solution?

@nanov I did some initial stab at more generic example code files in https://github.com/node-ebics/node-ebics-client/pull/58

Closing this one, I believe the solution in https://github.com/node-ebics/node-ebics-client/pull/58 will be the better way forward.


(DO NOT MERGE) (PUP-5308) Bump beaker dependency

We were using an old beaker version that relied on deprecated vmpooler
behavior (see BKR-572), which will be removed real soon now. So bump
beaker to the same version we're using in puppet#stable.
[skip ci]

Waiting on 3.8.4 release

Release is out, running some manual CI runs on linux and windows to make sure newer beaker still works with 3.x puppet

Real soon now :tm:

win2012r2-rubyx64.yaml
$ env PAGER= git show                                                                                ±[A20●][3.x]
commit f5c5658e211a1451a74e7a5f7dfac8cf3216b700
Merge: d638596 94d93e8
Author: Josh Cooper<EMAIL_ADDRESS>Date:   Wed Nov 4 10:33:57 2015 -0800

    Merge remote-tracking branch 'origin/ticket/3.x/PUP-5308-bump-beaker-2.23' into 3.x

    * origin/ticket/3.x/PUP-5308-bump-beaker-2.23:
      (PUP-5308) Bump beaker dependency

$ bundle exec rake ci:test:git SHA=3.8.4 CONFIG=config/nodes/win2012r2-rubyx64.yaml
...
              - Test Case Summary for suite 'tests' -
       Total Suite Time: 2988.45 seconds
      Average Test Time: 11.11 seconds
              Attempted: 269
                 Passed: 219
                 Failed: 0
                Errored: 1
                Skipped: 48
                Pending: 1
                  Total: 269

      - Specific Test Case Status -

Failed Tests Cases:
Errored Tests Cases:
  Test Case tests/external_ca_support/apache_external_root_ca.rb reported: #<Beaker::Host::CommandFailure: Host 'lgjfmwmwirg6c78.delivery.puppetlabs.net' exited with 1 running:
 mv --force /etc/httpd/conf/httpd.conf{.orig,}
Last 10 lines of output were:
	mv: cannot stat `/etc/httpd/conf/httpd.conf.orig': No such file or directory>

The one failure is a known issue do to upstream removing mod_passenger packages.

Using beaker 2.28.0 on windows:
q5dmrb8effl6lut.delivery.puppetlabs.net (agent-2012r2-x86_64-rubyx64) 23:25:57$ cmd.exe /c puppet module install --version '< 1.7.0' pmtacceptance-java
  Notice: Preparing to install into C:/ProgramData/PuppetLabs/puppet/etc/modules ...
  Notice: Created target directory C:/ProgramData/PuppetLabs/puppet/etc/modules
  Notice: Downloading from https://forgeapi.puppetlabs.com ...
  Notice: Installing -- do not interrupt ...
  C:/ProgramData/PuppetLabs/puppet/etc/modules
  └─┬ pmtacceptance-java (v1.6.0)
    └── pmtacceptance-stdlub (v1.0.0)

Using rhel6:
hydcrett5pm7oxx.delivery.puppetlabs.net (master) 23:36:00$ puppet module install --version '< 1.7.0' pmtacceptance-java
  Notice: Preparing to install into /etc/puppet/modules ...
  Notice: Downloading from https://forgeapi.puppetlabs.com ...
  Notice: Installing -- do not interrupt ...
  /etc/puppet/modules
  └─┬ pmtacceptance-java (v1.6.0)
    └── pmtacceptance-stdlub (v1.0.0)

Ready to merge now?

Yep, should be good


support deprecated magic __toString() in echo statement

with this change phpstan is able to detect cases, where object values are used in echo statements and implicitly call a magic and deprecated __toString method.
In case this new rule gets accepted, I would be willing to implement the same for the print statement.
refs https://github.com/phpstan/phpstan/issues/4899

Hi, this is rather incomplete and does not have rules for other situations.
What I recommend in your situation:

Delete the deprecated __toString() method from your code.
Run PHPStan to see all the usages.
Remove the usages.

Thx for your feedback.
I am not sure how I should take it.
I am aware there are a lot of more things to take care of, but just want to get in contact early, since the very basic case which is important to me is already covered.
Was your suggestion about how I should proceed to get this rule into shape so you might consider merging it, or was your intention to give me a hint of how I should handle a possible deprecated __toString() method in my real world project?
Thanks again

I want ahead and added more coverage for more complex expressions.
In case someone knows a expression I am missing feel free to sent me an example.


Superagent

Refactor engine to use superagent due to issues with memory leaks

For cfpb/hmda-pilot#265


oh-my-posh: 8.36.6 -> 8.36.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jandedobbeleer/oh-my-posh/releases.
meta.description for oh-my-posh is: "A prompt theme engine for any shell"
meta.homepage for oh-my-posh is: "https://ohmyposh.dev"
meta.changelog for oh-my-posh is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.36.7 with grep in /nix/store/nqzs77l3x1waavanbp2l0n3zfsj8m186-oh-my-posh-8.36.7
found 8.36.7 in filename of file in /nix/store/nqzs77l3x1waavanbp2l0n3zfsj8m186-oh-my-posh-8.36.7

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
oh-my-posh

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nqzs77l3x1waavanbp2l0n3zfsj8m186-oh-my-posh-8.36.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A oh-my-posh https://github.com/r-ryantm/nixpkgs/archive/ccdc8f3162aa5ec41ae2c6852ce21cc64c69c25f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nqzs77l3x1waavanbp2l0n3zfsj8m186-oh-my-posh-8.36.7
ls -la /nix/store/nqzs77l3x1waavanbp2l0n3zfsj8m186-oh-my-posh-8.36.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

oh-my-posh

Maintainer pings
cc @lucperkins for testing.

Result of nixpkgs-review pr 189286 run on x86_64-linux 1

1 package built:

oh-my-posh


Set SkipDetect appropriately when staging with multiple buildpacks

Fixes https://github.com/cloudfoundry-incubator/eirini-release/issues/149

LGTM, thank you for your contribution @stoyanr!


[Cirrus Polls] Add tip to polls repository

See https://app.clickup.com/t/emytav.
This PR fixes intermittent "Invalid Signature" errors that can pop up when running the node after being offline during a federation change.
It implements a tip for the polls repository and makes the processing of blocks atomic by allowing the various operations to be called in the context of a single transaction, which culminates in the updating of the tip. The tip now rigorously specifies how up-to-date the repository is in relation to the consensus chain. The current assumption has been that a signal-based update mechanism is adequate. Unfortunately we don't have atomic operations over the various repositories so this can never be guaranteed.
Changes include:

The tip allows us to be more specific about when the federation is known or not and as a result the PoAHeaderSignatureRule could be improved with the use of a new CanGetFederationForBlock method. Peers will no longer be banned due to the current node having a lagging polls repository.
Improvements to caching of federation history.
Combining federation (polls repository) reconstruction into the BAU code-flow. The polls repository is now reconstructed from block 0 to ensure that ALL polls are reconstructed. This takes about 15 minutes now. Further optimisation does not seem possible as the node (during re-sync) is spending 95% of its time in the bouncy castle math now and we're doing no repeated work (due to caching).

@fassadlr , I've tested this with CirrusMinerD and its synching and mining correctly. The console display of mining history is snappy.
Will do some more tests with fed peg.

I ran the master version of CirrusMinerD today and got an "Invalid Signature" error. I then ran the code from this PR and the error went away. So its definitely fixing some stability issues for us.


Troubles creating "View Repository" button

Hi,
Coming from Crawl's extension because this one starts up in half the time (thanks!), and I'm having trouble enabling the "view repository" button.
I looked at the Feature Contributions, and I tried to set vscord.status.buttons.button1.git.active.enabled to true, since it seemed that it would do what I want, but that didn't do anything.
I also tried to add this config, to no avail:
{
   "vscord.status.buttons.button1.active.enabled": true,
   "vscord.status.buttons.button1.active.label": "View Repository",
   "vscord.status.buttons.button1.active.url": "{git_url}",
}

So I'm asking, how would I go about doing this, since it seems this feature is suported?
Maybe this issue is related to #151. For me, a more intuitive system would have a configuration like so:
"vscode.status.buttonActive": "myButtonA",
"vscode.status.buttons": {
  "myButtonA": { ... },
  "myButtonB": { ... },
}

That way, an arbitrary amount of buttons can be added without cluttering the "Feature Contributions" extension tab. It would also be really simple to have have multiple builtin buttons and then be able to specify through buttonActive. Maybe you can make the image parts of the config object-based too Of course this would have to be documented somewhere...

why did we never think of that @leonardssh @nick22985 XD
it might also be a bug too, i will have to talk with two contributors I mentioned first.

That is a better way of formatting it xD and hanling lol

nick, can try it and see if the settings menu show "Edit in settings.json" or easy to edit gui

Dont have much time atm will get to this when I can unless someone else wants to

man.. got the same problem, but this issue closed quickly. Now i hope i appeared outta nowhere so i can contribute. And i think i know why its realy messy tho 😕


post-release fixes shortlsit

cardano-wallet-selector__menu__item__text set text color to # 333
spinning loader icon when loading
cardano-notification__content color to # 333
cardano-toaster__content # 333
make notifier to either not be fixed, easily configurable, or dissapear on scroll
positioning of toaster and notifier

z-index on the elements

I'm thinking

remove notification
when wallet selected, show i symbol, and info when clicking wallet selector instead, with disconnect

The example component embedded on the documentation site doesn't look quite right, let's find out why!

done, moving tailwind remark to own issue


add post_cmd_args() interface to jobs

Coverage decreased (-0.03%) to 89.222% when pulling 324ef5d9d88e935c3b7d85db43e17968c6243ff8 on irontable:master into 9dd9786018fda92463b4fb6e06705dfdc15f7bb2 on Netflix:master.


Customize tooltip for different elements of a mark?

Just a question: I'm using the new Graph mark.
tt = Tooltip(fields=node_attrs_list)

graph = Graph(node_data=node_data,
              link_data=link_data,
              link_type='line',
              colors=color_array,
              tooltip=tt,
              directed=False)

Instead of adding the same tooltip to all nodes in the graph, I would like to display different tooltip for different nodes (ie, some nodes might have a 'size' attribute, others might not.) Is there any way to do this? Or if not, is there a way to get the tooltip to not display empty attributes?

You can observe the hovered point and change the attributes of the tooltip in the function. it would look something like:
def check(*args, **kwargs):
    if condition_check_point(graph.hovered_point):
        tt.fields = node_attrs_list_1

graph.observe(check, 'hovered_point')

Though, that's a good point, we should allow the user to decide not to display attributes when theyre None for the tooltip

Perfect, thank you so much!!

@dustinmichels Great, Im closing this issue. FYI, you can send an entire Figure as the tooltip, and use the hovered_point to modify it. The examples for the MarketMap detail this quite well, this should allow you to display an arbitrary chart for each node.


"Liked by 0" message on ratings

On the Rating section of the resource landing page, seeing that a resource is liked by no one is a little discouraging. I think when the number of likes is 0 the message should be changed to "Be the first to +1 this resource" or something similar.

Would this work?

I like that text. Friendly and positive.
On Mon, Aug 3, 2015 at 2:03 PM Maurier Ramirez<EMAIL_ADDRESS>wrote:

[image: ss]
https://cloud.githubusercontent.com/assets/2448568/9046538/37040852-39f9-11e5-8d76-890d55044773.png
Would this work?
—
Reply to this email directly or view it on GitHub
https://github.com/hydroshare/hydroshare/issues/530#issuecomment-127390528
.

This fix has been merged and appears to be working in develop.

Checked on alpha


Poor Temporal

The temporal action annotations of Activitynet are so poor that I can not trust them. I believe the data needs to be relabelled. For example, temporal annotations in video 'OiL6Aj0gC14', 'onBAyGhqubg' and 'optJ47P_5Ys'. Is there any reason for that?

Thanks for the feedback about the quality of temporal annotations. While we have conducted two rounds of cleaning already, we are open to contributions from the community to improve the dataset quality further. Please send your feedback, and potential annotation fixes, to the dataset maintainers via email.


Relationship between BEM and atomic/functional CSS?

Sorry if this question could be somewhat silly, I'm just a new learner.
I can see that applying BEM to styling will overally improve code quality in many aspects.
However, the current trend is seemingly moving towards atomic/functional CSS.
In my understanding, it is the so called global modifier once addressed in the FAQ section here: http://getbem.com/faq/#can-i-create-global-modifier. Perhaps in a different context?
I wonder what the most recent viewpoint on relationship between these two powerful CSS methodology is?
Perhaps some experienced user can elaborate on some actual way to combining BEM and atomic/functional CSS?
Thanks and regards.

@iamstarkov Please help, thanks.


fixed bad hyphen encoding

https://github.com/Tivix/django-common/issues/31

Ping. If there are any questions on the change please let me know.

Thank you! 👍


No imperial options/settings or settings at all

Testing your app out and I don't see any settings (at all) to change from metric to imperial. I'm sure plenty of your users will need this option. Fairly easy to implement.

Yeah I can add that option to it.


Foundation sec members sec contact casc

Description

I would like to propose our help with regards to security maintenance.
Our team at CloudBees will be able to react on security issue reports, by definition time-sensitive. This way we can be notified on the event of a security report to help on the resolution, collaboration with the security team, synchronization, etc.
We have done this already for a number of plugins:
https://github.com/jenkins-infra/repository-permissions-updater/search?q=foundation_security_members&unscoped_q=foundation_security_members
If anybody sees a problem with this, please let me know.
Thank you.

Repo implied: https://github.com/jenkinsci/configuration-as-code
CC @praqma @jetersen @oleg-nenashev @timja for approval from an existing maintainer.
CC @daniel-beck as the security officer since this PR is changing security contacts.

Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Merging given @jetersen approved. We'll adjust if anything is needed anyways.
Thanks all.


Several go packages fail for lack of perl in path

I cannot identify clear culprit, but some time in the last 48 hours or so, something seems to have changed such that a range of go packages have begun failing for want of perl (which is, incidentally, hilarious :):
building path(s) ‘/nix/store/3j14b2xc0h3kni3c1mp4myawlk9x1raj-go1.5-govers-2015-01-09’, ‘/nix/store/695d3iyl9rr8xrs02hngvz6szgpq70ls-go1.5-govers-2015-01-09-bin’
unpacking sources
unpacking source archive /nix/store/sx0z80s66yq7pjfphrqfkqm7m3xj57pg-govers-3b5f175f65d601d06f48d78fcbdb0add633565b9-src
source root is govers-3b5f175f65d601d06f48d78fcbdb0add633565b9-src
patching sources
configuring
building
sh: perl: command not found
parallel: Error: No more processes: cannot run a single job. Something is wrong at github.com/rogpeppe/govers.
builder for ‘/nix/store/p5w8d92gbpr5bd9c2g9iv9n8p4a6srg0-go1.5-govers-2015-01-09.drv’ failed with exit code 255

I don't know how widespread the issue is.

@pSub A git bisect identified your commit, ed2b30dc283f2ec326fe80dc5259682d1f0f4fb3 as the commit that broke this.  Your change to get a new version of parallel is fine, but the other changes in the code fail.

Fixed by https://github.com/NixOS/nixpkgs/pull/11283.


error: unable to load plugin '../afl-llvm-pass.so': '../afl-llvm-pass.so: undefined symbol: _ZNK4llvm10ModulePass17createPrinterPassERNS_11raw_ostreamERKSs'

hello, when i build afl to llvm_mode, creat some error. like below:
➜  llvm_mode make
[] Checking for working 'llvm-config'...
[] Checking for working 'clang'...
[] Checking for '../afl-showmap'...
[+] All set and ready to build.
[] Building 32-bit variant of the runtime (-m32)... failed (that's fine)
[*] Testing the CC wrapper and instrumentation output...
unset AFL_USE_ASAN AFL_USE_MSAN AFL_INST_RATIO; AFL_QUIET=1 AFL_PATH=. AFL_CC=clang ../afl-clang-fast -O3 -funroll-loops -Wall -D_FORTIFY_SOURCE=2 -g -Wno-pointer-sign -DAFL_PATH="/usr/local/lib/afl" -DBIN_PATH="/usr/local/bin" -DVERSION="2.52b"  ../test-instr.c -o test-instr
error: unable to load plugin '../afl-llvm-pass.so': '../afl-llvm-pass.so: undefined symbol: _ZNK4llvm10ModulePass17createPrinterPassERNS_11raw_ostreamERKSs'
make: *** [test_build] Error 1
=============================
my llvm verion is :
➜  llvm11 llvm-as --version
LLVM (http://llvm.org/):
LLVM version 11.0.0git
Optimized build with assertions.
Default target: x86_64-unknown-linux-gnu
Host CPU: cascadelake
➜  llvm11 clang --version
clang version 11.0.0
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /usr/local/llvm11/bin
➜  llvm11

I ran into this as well.

update your clang with
apt install clang

I am having the same problem on a Gentoo Linux installation. I have LLVM 12.0.0 installed and I'm still seeing the problem.

I've been doing some investigation, and I'm also posting what I'm finding here in the Gentoo issue tracking this problem.
Looks like at least on my system a whole swath of LLVM symbols can't be resolved at build-time. A quick sample of two symbols:
ld.lld: error: undefined symbol: llvm::ModulePass::~ModulePass()
>>> referenced by afl-llvm-pass.so.cc:52
>>>               /tmp/afl-llvm-pass-8f42cb.o:((anonymous namespace)::AFLCoverage::~AFLCoverage())
>>> referenced by afl-llvm-pass.so.cc
>>>               /tmp/afl-llvm-pass-8f42cb.o:(vtable for (anonymous namespace)::AFLCoverage)

ld.lld: error: undefined symbol: llvm::Type::getInt8Ty(llvm::LLVMContext&)
>>> referenced by afl-llvm-pass.so.cc:77
>>>               /tmp/afl-llvm-pass-8f42cb.o:((anonymous namespace)::AFLCoverage::runOnModule(llvm::Module&))

My llvm and clang installations seem fine otherwise, as I can build other applications using clang, and LLVM is working fine as I can build and use rust just fine with the system llvm.

Even if most of the symbols I found that are missing are somehow loaded at runtime, it still appears that createPrinterPass isn't found.
Here's the build output with verbose output:
# unset AFL_USE_ASAN AFL_USE_MSAN AFL_INST_RATIO; AFL_QUIET=1 AFL_PATH=. AFL_CC=clang ../afl-clang-fast -O3 -funroll-loops -Wall -D_FORTIFY_SOURCE=2 -g -Wno-pointer-sign -DAFL_PATH=\"/usr/lib64/afl\" -DBIN_PATH=\"/usr/bin\" -DVERSION=\"2.57b\"  ../test-instr.c -o test-instr -v -fuse-ld=lld
clang version 12.0.0
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/lib/llvm/12/bin
Selected GCC installation: /usr/lib/gcc/x86_64-pc-linux-gnu/11.1.0
Candidate multilib: .;@m64
Candidate multilib: 32;@m32
Selected multilib: .;@m64
Found CUDA installation: /opt/cuda, version 11.0
 "/usr/lib/llvm/12/bin/clang-12" -cc1 -triple x86_64-pc-linux-gnu -emit-obj --mrelax-relocations -disable-free -disable-llvm-verifier -discard-value-names -main-file-name test-instr.c -mrelocation-model static -mframe-pointer=none -fmath-errno -fno-rounding-math -mconstructor-aliases -munwind-tables -target-cpu x86-64 -tune-cpu generic -fno-split-dwarf-inlining -debug-info-kind=limited -dwarf-version=4 -debugger-tuning=gdb -v -resource-dir /usr/lib/llvm/12/bin/../../../../lib/clang/12.0.0 -D _FORTIFY_SOURCE=2 -D "AFL_PATH=\"/usr/lib64/afl\"" -D "BIN_PATH=\"/usr/bin\"" -D "VERSION=\"2.57b\"" -D __AFL_HAVE_MANUAL_CONTROL=1 -D __AFL_COMPILER=1 -D FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION=1 -D "__AFL_LOOP(_A)=({ static volatile char *_B __attribute__((used));  _B = (char*)\"##SIG_AFL_PERSISTENT##\"; __attribute__((visibility(\"default\"))) int _L(unsigned int) __asm__(\"__afl_persistent_loop\"); _L(_A); })" -D "__AFL_INIT()=do { static volatile char *_A __attribute__((used));  _A = (char*)\"##SIG_AFL_DEFER_FORKSRV##\"; __attribute__((visibility(\"default\"))) void _I(void) __asm__(\"__afl_manual_init\"); _I(); } while (0)" -internal-isystem /usr/local/include -internal-isystem /usr/lib/llvm/12/bin/../../../../lib/clang/12.0.0/include -internal-externc-isystem /include -internal-externc-isystem /usr/include -O3 -Wall -Wno-pointer-sign -fdebug-compilation-dir /tmp/portage/app-forensics/afl-2.57b/work/AFL-2.57b/llvm_mode -ferror-limit 19 -funroll-loops -fgnuc-version=4.2.1 -fcolor-diagnostics -vectorize-loops -vectorize-slp -load ../afl-llvm-pass.so -o /tmp/test-instr-8f1575.o -x c ../test-instr.c
error: unable to load plugin '../afl-llvm-pass.so': '../afl-llvm-pass.so: undefined symbol: _ZNK4llvm10ModulePass17createPrinterPassERNS_11raw_ostreamERKNSt3__112basic_stringIcNS3_11char_traitsIcEENS3_9allocatorIcEEEE'

Demangling the symbol:
llvm::ModulePass::createPrinterPass(llvm::raw_ostream&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&) const

Interesting, if I try building the plugin manually but add -lLLVM-12 to the command, the only symbol missing is:
ld.lld: error: undefined symbol: llvm::PassManagerBuilder::addGlobalExtension(llvm::PassManagerBuilder::ExtensionPointTy, std::__1::function<void (llvm::PassManagerBuilder const&, llvm::legacy::PassManagerBase&)>)
>>> referenced by PassManagerBuilder.h:244 (/usr/lib/llvm/12/include/llvm/Transforms/IPO/PassManagerBuilder.h:244)
>>>               /tmp/afl-llvm-pass-dc9876.o:(llvm::RegisterStandardPasses::RegisterStandardPasses(llvm::PassManagerBuilder::ExtensionPointTy, std::__1::function<void (llvm::PassManagerBuilder const&, llvm::legacy::PassManagerBase&)>))

ld.lld: error: undefined symbol: llvm::ModulePass::createPrinterPass(llvm::raw_ostream&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&) const
>>> referenced by afl-llvm-pass.so.cc
>>>               /tmp/afl-llvm-pass-dc9876.o:(vtable for (anonymous namespace)::AFLCoverage)
clang-12: error: linker command failed with exit code 1 (use -v to see invocation)

So it looks like for some reason this symbol is not in LLVM-12???

tried updating your clang?

My clang is at latest, I think, at 12.0.0.

Interesting. I did find a library in the LLVM system directory with a similar symbol defined, but it's not the same... here's the symbol (and its demangled version) that's being asked for by afl:
_ZNK4llvm10ModulePass17createPrinterPassERNS_11raw_ostreamERKNSt3__112basic_stringIcNS3_11char_traitsIcEENS3_9allocatorIcEEEE
llvm::ModulePass::createPrinterPass(llvm::raw_ostream&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&) const

And here's the symbol I found in my system LLVM:
_ZNK4llvm10ModulePass17createPrinterPassERNS_11raw_ostreamERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE@@LLVM_12
llvm::ModulePass::createPrinterPass(llvm::raw_ostream&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) const@@LLVM_12

Hm, looks like the difference is C++11 string linking? and the @@LLVM_12 tag.

After investigating a bit more what the sources of those namespaces are, turns out that what appears to be happening is that my LLVM links against libstdc++ while clang by default tries to link against libc++. If I force clang to use libstdc++ when building afl, I'm no longer getting missing symbols. I think this is a consequence of AFL trying to build an out-of-tree pass for LLVM, and not knowing how LLVM was built. I'm not exactly sure how to go about fixing this.

Before you start your own afl fork, just switch to https://github.com/aflplusplus/aflplusplus
Alternatively, there are probably working docker files around with prior clang versions

I did some experimentation last night, and I can get AFL to build if I disable Gentoo's clang support for linking against libc++ by default. But I'll likely switch to aflplusplus anyway (just found out about it last night) which builds just fine regardless (at least on Gentoo).


Add vector rendering

Same as #69, but from my fork of xarray-leaflet.

@jfoster17 I made some optimizations that should hopefully fix the issues you reported.
I also changed the API so that you can choose to make a vector map dynamic, and pass an external rasterize function.

Thanks @davidbrochart -- as we discussed offline, I'm having trouble getting a new vector layer to serve through voila on a remote server. You suggested the following:
Pass get_base_url to df.leaflet.plot with:
def get_base_url(window_url):
    return window_url

and launching the application as:
jupyter server --ServerApp.default_url="voila/render/examples/vector.ipynb"
I haven't been able to get this working on my server, which I configured following the directions here. Specifically, since your instructions suggest launching voila via jupyter server, I'm sure there is some extra steps to make this work. I don't really understand the distinction...
I did try updating the nginx configuration for the proxy_pass to http://localhost:8888 since that is the port jupyter server was trying to serve to, and that got me as a login page for jupyer server, but then a 404 for the actual page.
Is there not a way to get this running with the voila command? Or is there some documentation on how to serve voila via jupyter server?

I also get this odd behavior when I am running this locally on my Mac versus (remotely) on Ubuntu. Serving the notebook notebook from Linux I get the expected behavior, but locally I get some sort of jumbled tiles where the color scales of buildings changes radically on zoom/pan and sometimes I can see obvious tile boundaries.
https://user-images.githubusercontent.com/3639698/191332677-dc5dd0f2-3840-4431-955a-7327151971d1.mov

Anything I can do to help review this?

Thanks for the help @raybellwaves!
The code is a bit of a mess, I'll probably refactor later, but it would be great if you could try this PR and give your feedback.
@jfoster17 I'll answer your comments tomorrow, it's late here :)

I did try updating the nginx configuration for the proxy_pass to http://localhost:8888/ since that is the port jupyter server was trying to serve to, and that got me as a login page for jupyer server, but then a 404 for the actual page.

@jfoster17 You can pass --port=8866 or whatever port number you want. If you land on a login page, then you need to enter the token jupyter-server showed in the terminal. Or you can disable authentication with --ServerApp.token='' --ServerApp.password=''.
Also, if your server has a base URL, e.g. https://yourdomain.com/my/base/url, then you need:
def get_base_url(window_url):
    return "https://yourdomain.com/my/base/url"

I get some sort of jumbled tiles where the color scales of buildings changes radically on zoom/pan and sometimes I can see obvious tile boundaries.

@jfoster17 I was finally able to track down the issue. It is a regression that has been reported here. For now, a workaround is to install pandas<1.5, e.g. pip install pandas==1.4.

it would be nice to be able to fiddle with the size of point-like data

@jfoster17 I think it would be nice to be able to apply transformations to the data, as with the raster data flow. For instance, the rivers in this notebook could not be revealed without "thickening", because rivers are represented with one-pixel wide streams in the original array. This is achieved by applying scipy.ndimage.maximum_filter(array, footprint=circle) on each tile.
I'm going to try and support transformation functions in the vector data flow.

Making some notes for reviewing this (mostly for myself).
git clone https://github.com/xarray-contrib/xarray_leaflet.git
cd xarray_leaflet
git fetch origin pull/75/head:vector
git checkout vector
mamba create -n xarray_leaflet python=3.10 --y
conda activate xarray_leaflet
mkdir -p ~/miniforge3/envs/xarray_leaflet/etc/jupyter/jupyter_notebook_config.d
mkdir -p ~/miniforge3/envs/xarray_leaflet/etc/jupyter/jupyter_server_config.d
cp etc/jupyter/jupyter_notebook_config.d/* ~/miniforge3/envs/xarray_leaflet/etc/jupyter/jupyter_notebook_config.d/
cp etc/jupyter/jupyter_server_config.d/* ~/miniforge3/envs/xarray_leaflet/etc/jupyter/jupyter_server_config.d/
mamba install jupyterlab pyarrow
pip install -e .
jupyter lab
# run examples/vector.ipynb and vector_points.ipynb

Notebooks work great.
TODO try it on a month of the NYC taxi cab data inspired by https://examples.pyviz.org/nyc_taxi/nyc_taxi.html as an example of big point data.
!wget https://d37ci6vzurychx.cloudfront.net/trip-data/yellow_tripdata_2022-01.parquet
!wget https://d37ci6vzurychx.cloudfront.net/misc/taxi_zones.zip

Thanks @raybellwaves.
Note that if you install JupyterLab before, then you don't need the mkdir, and you only need:
cp etc/jupyter/jupyter_server_config.d/* ~/miniforge3/envs/xarray_leaflet/etc/jupyter/jupyter_server_config.d/

Hey @davidbrochart -- here are the issues I was talking about offline.

Fails to deal with no data in the viewport

If the user moves the map outside of the region where there is data, xarray-leaflet hangs. For instance, starting the map in vector.ipynb so that NYC is not in the viewport:
m = Map(center=(45.74427755695474, -73.96981238541562), zoom=11, basemap=basemaps.CartoDB.DarkMatter, interpolation='nearest')
will cause the spinner to spin endlessly and nothing will ever display, even if you zoom over to New York City. You can get the same behavior moving away from a region where data has previously been rendered (perhaps a more likely scenario for most users).

Fails to deal with E/W longitude swap

Again, on the vector.ipynb notebook if you specify the starting map location as positive longitude
m = Map(center=(40.74427755695474, 286), zoom=11, basemap=basemaps.CartoDB.DarkMatter, interpolation='nearest')
the map will be center on NYC but xarray-leaflet will fail to plot any data, probably because the data is in negative longitude (i.e. -73) and thus not in the range of the viewport (again, leading to infinite spinning). So this is really two problems, lack of wrapping longitude leading to the above-specified problem.

Thanks Jonathan for reporting these issues.

issue 1: I cannot reproduce it, maybe my recent commits fix it and you are not up to date?
issue 2: I fixed it in https://github.com/xarray-contrib/xarray_leaflet/pull/75/commits/9ee6b34e44559060f69feba48c475bc9e5385117, but it also needs a fix in ipyleaflet.

I released ipyleaflet v0.17.2 with the fix and require that version in xarray-leaflet.

Thanks for your quick response David -- indeed Issue 1 goes away in a new up-to-date env, and Issue 2 goes away for me too with the new ipyleaflet version!


Improve the Button-Look

I'd suggest to improve the Button-Look since it is hard to identify which pipeline has passed or failed if we only see the GitLab logo and the status.
Here's a small mockup I made, the left button would be the easiest solution which only adds a title, to quickly identify the status of each pipeline I'd suggest to also add red/green indication and also parse the status:

The StreamDeck, at least at this moment, have a few limitations:

The action only can display one text, to help to identify projects I recommend you to customize the action image with the Stream Deck Key Creator
It is not possible to change the color of text, only font, size and position (top, center, bottom).
The action only can have two images, but I like the idea of having two images: success and failure.

Adding the option to use two images in case of fail/pass would be a great idea, this way the user could add a title and will still see the result :)

@tuc0w Sorry for the delay, but other projects taken out all my free time, if you feel that you can work on any of the open issues, please feel free to assign this issue to you and send a PR with the code.

I just send the update with multiple images for success and failed states

Sorry I wasn't able to get back to you before you did the coding, it looks great!


[HybridApp] Remove jcenter from Android codebase

Problem
We've seen Android HybridApp builds fail with a 502, this is because jcenter is being officially turned off August 15th. https://jfrog.com/blog/jcenter-sunset/
Solution
We need to migrate to mavenCentral, however, we need to also upgrade a few dependencies to their latest versions since they did not back port all versions.

Example: https://github.com/Expensify/Mobile-Deploy/actions/runs/10361798303/job/28682826522
* What went wrong:
Could not determine the dependencies of task ':buildReleasePreBundle'.
> Could not resolve all task dependencies for configuration ':releaseRuntimeClasspath'.
   > Could not resolve io.realm:realm-android:0.87.4.
     Required by:
         project :
      > Could not resolve io.realm:realm-android:0.87.4.
         > Could not get resource 'https://jcenter.bintray.com/io/realm/realm-android/0.87.4/realm-android-0.87.4.pom'.
            > Could not GET 'https://jcenter.bintray.com/io/realm/realm-android/0.87.4/realm-android-0.87.4.pom'. Received status code 502 from server: Bad Gateway
   > Could not resolve io.branch.sdk.android:library:2.19.3.
     Required by:
         project :
      > Could not resolve io.branch.sdk.android:library:2.19.3.
         > Could not get resource 'https://jcenter.bintray.com/io/branch/sdk/android/library/2.19.3/library-2.19.3.pom'.
            > Could not GET 'https://jcenter.bintray.com/io/branch/sdk/android/library/2.19.3/library-2.19.3.pom'. Received status code 502 from server: Bad Gateway
   > Could not resolve pl.charmas.android:android-reactive-location2:2.0.
     Required by:
         project :
      > Could not resolve pl.charmas.android:android-reactive-location2:2.0.
         > Could not get resource 'https://jcenter.bintray.com/pl/charmas/android/android-reactive-location2/2.0/android-reactive-location2-2.0.pom'.
            > Could not GET 'https://jcenter.bintray.com/pl/charmas/android/android-reactive-location2/2.0/android-reactive-location2-2.0.pom'. Received status code 502 from server: Bad Gateway


Pipeline summary endpoint

The Kontinuous UI needs to get summary details of a pipeline for display on the main page. Querying the builds endpoint is slow (since it returns all data), an additional endpoint should be added to return high level information about the pipeline and latest build.

Alternatively additional metadata could be added to the pipelines endpoint

Added in v0.1.2


Please update npm repo

Actual behaviour:
The tmi.js repo on npm says the last update was a year ago.
the dependency ws is on version 1.0.1 in npm instead of the most current 5.1.1 on github, 1.0.1 has a lot of security flaws
The same goes with the dependency request. It is outdated and insecure in the npm repo
Expected behaviour:
the npm repo should be updated to reflect the changes made in the github repo
If any help is needed please let me know and I will assist. Thank you for your time.

Please use twitch-js for now.


Format function head match assigments consistently

Ebert has finished reviewing this Pull Request and has found:

10 fixed issues! 🎉

You can see more details about this review at https://ebertapp.io/github/pinterest/elixir-thrift/pulls/80.


Gh 875 stream utils remove double iteration

Codecov Report

Merging #876 into develop will decrease coverage by 0.87%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             develop     #876      +/-   ##
=============================================
- Coverage      62.61%   61.73%   -0.88%
  Complexity       869      869
=============================================
  Files            444      444
  Lines          12827    12825       -2
  Branches        1480     1476       -4
=============================================
- Hits            8031     7917     -114
- Misses          4204     4325     +121
+ Partials         592      583       -9

Impacted Files
Coverage Δ
Complexity Δ

...java/uk/gov/gchq/gaffer/commonutil/StreamUtil.java
20% <0%> (+0.59%)
6 <0> (ø)
:arrow_down:

...chq/gaffer/rest/mapper/GenericExceptionMapper.java
0% <0%> (-100%)
0% <0%> (ø)

...hq/gaffer/rest/serialisation/RestJsonProvider.java
0% <0%> (-100%)
0% <0%> (ø)

...ain/java/uk/gov/gchq/gaffer/rest/SystemStatus.java
0% <0%> (-100%)
0% <0%> (ø)

...chq/gaffer/rest/application/ApplicationConfig.java
0% <0%> (-97.06%)
0% <0%> (ø)

.../java/uk/gov/gchq/gaffer/rest/FactoriesBinder.java
0% <0%> (-71.43%)
0% <0%> (ø)

...gov/gchq/gaffer/rest/service/OperationService.java
0% <0%> (-67.4%)
0% <0%> (ø)

.../gaffer/rest/mapper/ProcessingExceptionMapper.java
0% <0%> (-50%)
0% <0%> (ø)

.../gchq/gaffer/rest/factory/DefaultGraphFactory.java
14.89% <0%> (-40.43%)
0% <0%> (ø)

...v/gchq/gaffer/rest/factory/UnknownUserFactory.java
66.66% <0%> (-33.34%)
0% <0%> (ø)

... and 6 more

Continue to review full report at Codecov.

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


Remove link types, since they are excessive and channels actually seem to cover all the required functionality

Should be tested (#104)


Update services dialog to allow free-text names for service providers

This supports a common SST meeting use case of specific staff that work closely with the school and students but are not district employees (eg., outside counseling providers), or for cases of tutoring programs where the individual providing services is different from session to session.
This path seemed like a good balance between keeping the UI simple and not getting into the challenges of customizing for every different use case, while still allowing some ability to capture provider information for these important edge cases.  We also trade off the ability to perform more precise queries for which staff are providing services, but this capability isn't being used now and isn't an immediate priority.
Separately, but relatedly we also briefly considered updating this UI to support capturing finer-grained interventions like the difference between 1:1 Wilson, regular small group reading intervention, and occasional after school groups.  We didn't move forward with this since it increases challenges with data consistency and training across users and we wanted to focus on successful adoption at the current level of granularity first.

I think using free-text for service provider names will present challenges down the road.
The data that educators input during SST meetings ultimately gets used in two ways:

It becomes part of a qualitative, narrative story in the Student Profile
It becomes part of a quantitative view of the whole school in the School Overview

If we make this a free-text field, it will be difficult to surface in the information in form (2).
If we use a drop-down instead it will be easy to surface in form (2).

Ah okay yeah I completely agree so I think flubbed capturing this in writing. :)
What I took away was that there'd be a dropdown, but that we allow any arbitrary text to be entered as well.  I think I went right to "free-text," thinking about adding a new database column, and migrating the educator_ids to their text values.  We could also add a provided_by_text field and keep the provided_by_educator_id, and enforce that only one is set and let clients branch on that when querying or displaying.
Does that match what you're saying, or am I still flubbing it? :smile:

Let's say we have provided_by_text. Won't we run into problems like
LIST OF SERVICE PROVIDERS AT HEALEY SCHOOL

+ Kevin Robinson
+ Robinson, Kevin
+ Kevin Robinsoon
+ Kevin R

@alexsoble cool, thanks for chatting this out in Slack, and let's chat more in person next week, since I think I might have misunderstood this and we've got other important things to finish first :)

@kevinrobinson  👍

Started down the denormalizing path on the backend.
Will finish this up later tonight or tomorrow.
Was thinking about the front-end. I think to save time, perhaps we should have two separate input fields.
Is this service being provided by a #{school.name} staff member?
{{ educator dropdown }}
Is this service being provided by someone outside #{school.name}?
{{ free text input }}
No messing around with combo boxes...

@kevinrobinson and I talked about this feature over Slack and IRL.
Here's an overview of what we discussed:

Currently, the drop-down list of staff providers is a blocker for capturing accurate data during SST meetings. When the SST notetaker can't find the right staff member, s/he assigns herself, which makes the data inaccurate.

One approach would be to try to add a bunch of options for all of the service providers who aren't directly employed by Somerville Public Schools. But this creates a new data field of outside service providers and a big new chunk of work. It might help us avoid the human-entered data problem outlined above, but we wouldn't be able to ship the feature quickly and it would add non-trivial complexity to the service-recording process.

So the plan is to capture a provided_by_educator_name for each service, denormalizing services from the educators table. This will let SST / MTSS note-takers enter any name, supported by a dropdown of previously-entered educator names, so they can quickly capture provider data.

It's possible that the provided_by_educator_name will suffer from some human-entered data errors, but this is not a pressing issue right now because we are not querying for service providers anywhere in the app. If we decide in the future that querying by service provider is an important feature, we can double-check the data, do some cleanup, and run migrations to fix any issues.

As part of this change, I think we should make provided_by_educator_name optional. If a note-taker can't remember the provider name, or the service is provided by a group of people instead of one person, or some other complicating factor we can't think of arises — we don't want to run into the same issue we faced earlier of forcing note-takers to enter incorrect data in order to satisfy strict validation.

One side effect of denormalization within the profile page:
The renderStaff function outputs a list of staff who are supporting the student, with mailto links for contacting each staff person.
We'll need to either renormalize the provided_by_educator_name into a provided_by_educator_id when possible for a student who is receiving services, or drop features that require normalization / other data from X2 such as email address.

After doing some exploratory poking around, I'm going to break this into a series of smaller chunks of work as opposed to one big PR that changes the data structure and the UI at the same time.
At least three separate, non-interdependent, merge-able pieces of work that will reduce complexity in the Services sections of the Student Profile and make denormalizing simpler:

Make provided_by_educator optional for Services
Remove deprecated Intervention model
Clarify "Support staff" summary list in student profile

@alexsoble This is amazing, just caught up on your incredible notes and explanation here.  I added it to the contributing doc as an example of what awesome communication on a pull request looks like! 😄

@kevinrobinson Aw, thanks Kevin!


[RFC] config: Add config package to handle cluster configuration

New configuration package is added in order to handle
the yaml cluster configuration provided to the
scheduler.
Implements part of issue #66
Signed-off-by: Simental Magana, Marcos<EMAIL_ADDRESS>

-1
Missing code coverage.
PR for early code review purposes.

Coverage decreased (-0.7%) to 53.856% when pulling b2c0d43f4a432069632146257176107a8aec8ba6 on mrkz:config_package into 5d7d8d022b9ee6d667d83513585d88b37649fe86 on 01org:master.

Version 2 of the patchset looks pretty good, besides the few nitpicks that I commented about.
With the next version, could you please also include unit testing ?

@sameo @mrkz  what do you think about adding an etc/ciao/ciao.yaml sample config file with the available variables that can be manipulated from configuration file?

@obed I'd prefer to add one to the the configuration/README.md file where we will have to document all possible fields within that file, rather than having an out-of-context sample file.
Also, I expect unit tests to be build on top of sample files embedded into the code as strings.

BTW when you add unit testing don't forget to update the .travis.yml file.  You need to add your new package to the first call to test-cases so that your unit tests get run, i.e., this line
   - test-cases -text -coverprofile /tmp/cover.out -short github.com/01org/ciao/ciao-launcher github.com/01org/ciao/ciao-scheduler github.com/01org/ciao/ciao-controller/... github.com/01org/ciao/payloads

I get a 89.7% coverage with your unit tests, this is pretty good.

Coverage increased (+0.3%) to 57.389% when pulling aa85b0e81b0a294a0435cef63e54d06cdbb4db26 on mrkz:config_package into d577bfd287d3deaf21a9e4c8d1e452a03d7fdfd8 on 01org:master.


We should make it clear that NPM is the only package manager officially support

Link to the Page
https://www.skeleton.dev/guides/install
Describe the Issue
PNPM, Yarn, etc may work, but we cannot promise they will do so without issue. We need to discourage bug tickets for unsupported features right now and instead make it clear that if people want support for these technologies they will need to contribute it.
Are you able to create a Pull Request with the fix?
Yes

Done:


tweaking dcgan tutorial

@SsnL @yf225 @jspisak doing a PR for the 3rd iteration of fixes to make sure it builds properly

Looks like the dcgan built properly, the chatbot tutorial is causing the fail. I am merging this now.


Define SHELL environment variable

Summary of the new feature / enhancement
Currently, it is very difficult for processes to determine whether or not they were launched from PowerShell. Workarounds include inspecting the process tree or relying on assumptions about the contents of PSModulePath.
Many (most) unixy shells define SHELL in the environment which points to the executable path of the currently running shell. This would make it very easy and predictable for programs to detect powershell.
Proposed technical implementation details (optional)
No response

@sztomi  could you please explain a little more about the scenario? Are you talking about executables rather than scripts? Have you tried mocking this up in your profile, it should be fairly trivial to do something like$env:SHELL = "${PSHOME}/pwsh". Is there a particular application that is having issues? On my Mac, for example, $SHELL is set to /bin/zsh even if I run bash -l

It is dup #12150 which we concluded to close.

unixy shells define SHELL in the environment which points to the executable path of the currently running shell

Environment is the key word. In Windows no such environment variable, just $env:COMSPEC, that is nearest to $SHELL in unix OSes, where you could set pwsh as logon shell.

could you please explain a little more about the scenario? Are you talking about executables rather than scripts?

@JamesWTruher  In my case, I'm talking specifically about a python script, but from that perspective it might as well be an executable. My script needs to distinguish between powershell and cmd to call a powershell script or batch script, respectively.

It is dup #12150 which we concluded to close.

@237dmitry it is similar, but I'm not proposing this only for non-windows, I'm proposing this for all operating systems. Current solutions for detecting powershell are hacky at best: https://stackoverflow.com/a/55598796

Current solutions for detecting powershell are hacky at best

Checking PSModulePath environment variable is not universal. For example, in my Linux environment there is not this variable. PSModulePath is available only inside pwsh when it running.

@237dmitry That's kinda my point: the detection without something that PS provides is hacky, brittle. Hence the issue. I don't think you addressed my answer as to why this isn't a dupe, yet closed this issue as such. I can live with a WONTFIX, though I don't think anything said here really explained why this shouldn't happen or provide any meaningful official way of detecting the shell.


chore(e2e-tests): fix lint errors

Introduction :pencil2:
There are some lint errors that appear in e2e-tests due to the cypress unsafe-to-chain-command rule.
Resolution :heavy_check_mark:
By wrapping the rule-breaking functions in cy.wrap, we ensure that a chainable command is returned and therefore this rule is no longer broken.

Codecov Report
Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (6da84ce) 21.88% compared to head (92fc1c7) 22.57%.
Report is 164 commits behind head on main.

Files
Patch %
Lines

...1/controllers/tfm/deal/tfm-get-deals.controller.js
0.00%
3 Missing :warning:

...lers/tfm/facility/tfm-get-facilities.controller.js
0.00%
1 Missing and 1 partial :warning:

dtfs-central-api/src/drivers/db-client.js
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2687      +/-   ##
==========================================
+ Coverage   21.88%   22.57%   +0.68%
==========================================
  Files        1132     1136       +4
  Lines       27048    27255     +207
  Branches     4447     4486      +39
==========================================
+ Hits         5920     6153     +233
+ Misses      18403    18388      -15
+ Partials     2725     2714      -11

Flag
Coverage Δ

api-test
0.48% <ø> (ø)

ui-test
65.66% <ø> (+0.52%)
:arrow_up:

unit-test
4.63% <0.00%> (+0.85%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Respect freedesktop/XDG spec for configuration

Reference: https://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html
$XDG_CONFIG_HOME defines the base directory relative to which user specific
configuration files should be stored. If $XDG_CONFIG_HOME is either not set or
empty, a default equal to $HOME/.config should be used.

I retained fall-back behavior for ~/.ansiweatherrc for backward compatibility.

This looks reasonable, I will try to review your diff soon.

@fcambus any luck? I'm cleaning up my old forks

So I had a look at your diff, but I couldn't find anything about the following part in the XDG specification:
elif [ -s "$HOME"/ansiweather/config ] ; then
	config_file="$HOME"/ansiweather/config

Is there any valid reason to add it?

That should be "$HOME"/ansiweather/.config:

$XDG_CONFIG_HOME defines the base directory relative to which user specific configuration files should be stored. If $XDG_CONFIG_HOME is either not set or empty, a default equal to $HOME/.config should be used.

I'll upload a fix.

@fcambus how's that?

It all makes sense with this change, thanks for your contribution!
I will tag a new version soon.


Read existing custom tags from databricks config template

In order to create a new data bricks cluster, we are required to set a series of custom tags. It did not seem like this was directly configurable.
This was less than 20 lines, so I wanted to share what I did to get it working. It seems like there are a few different ways to do this and I would love to learn of an existing way I missed or contribute a PR to add functionality.
I will post this in slack and we can discuss there.

https://feathrai.slack.com/archives/C036HEUQM08/p1659720184974569?thread_ts=1659720184.974569&cid=C036HEUQM08
This thread should have more context


[DOC] Misc Doc improvements

Report incorrect documentation
Location of incorrect documentation
docs/additional-functionality/rapids-shuffle.md file:
References versions that are Spark snapshot versions, since we don't release them we should remove them from the table.
download.md file:
The plugin is tested on the following architectures mention V100, A100, etc

maybe a nit and again fixed for later, but aren't the "architectures" really Ampere, Volta, Turing vs GPU models?

Requirements mention 	Apache Hadoop 2.10+ or 3.1.1+ (3.1.1 for nvidia-docker version 2)

do we want to qualify this as its not really required if not using YARN and really these are only required if using docker/cgroups?

Version info mentioned Azure Synapse is 21.10 but not 21.12:

we put Azure Synapse here, but they are in progress of upgrading did we want to put something for Azure and EMR that say see their pages for current version?

@tgravescs @nvliyuan
I submitted https://github.com/NVIDIA/spark-rapids/pull/4472 for this to address item:

docs/additional-functionality/rapids-shuffle.md
The plugin is tested on the following architectures mention V100, A100, etc

For item:
3. Requirements mention Apache Hadoop 2.10+ or 3.1.1+ (3.1.1 for nvidia-docker version 2)
What is our suggestion for supported Hadoop version?

Version info mentioned Azure Synapse is 21.10 but not 21.12.
Currently the Azure Synapse only ships with 21.10. I talked with them and once 21.12 is updated officially, we need to remove it from 21.10 and add it to 21.12 download page.

for 3. we should either quantify it saying, for yarn deployments that want to use GPU scheduling requires Hadoop 2.10+....  Or alternatively we could remove it form here and say see the documentation for any requirements based on the deployment used.

do we really want to have to keep updating our docs based on this or could we just point to some Azure docs?

For 4, currently there is no Azure Doc mentioning the version of the plugin.  Maybe once it is there, we can point it to Azure doc. However if we do this for Azure Synapse, we may need to do this for EMR as well?

it might be a call for Sameer or Karthik, but to me it would be nice just to point to any third party docs since we don't control the versions.

For #3 https://github.com/NVIDIA/spark-rapids/pull/4512 is filed.
For #4, i will update it once Azure Synapse release new version of jars later.

I think we can close this one now.


Simple Voter Class Error - TypeError: Cannot cast array data from dtype('O') to dtype('int64') according to the rule 'safe'

After creating a pipeline with my best models and including the feature-transformer I used to create the training data... I get the following error:
TypeError: Cannot cast array data from dtype('O') to dtype('int64') according to the rule 'safe'
My feature pipeline makes use of StandardScaler, OneHotEncoder, HashingVectorizer and uses make_column_transformer to make a Pandas column transformer.
ensemble_pipeline = make_pipeline(
    feature_pipeline,
    SimpleVoter(
    best_models,
    classes=best_models[0][1].classes_, voting="hard"
    ),
)

Any idea the reason behind this error?
Thanks!

Can you post the full traceback error?
I'd expect SimpleVoter to work with sklearn Pipelines but I'm not sure it's been explicitly tested. Does your code work if you use one of the best models in place of the SimpleVoter?

I've submitted a PR with a quick example of using SimpleVoter inside of a Pipeline(https://github.com/Ibotta/sk-dist/pull/19). I'd expect this to work generally assuming that the pipeline would otherwise work with all of the individual models in the SimpleVoter as the last step of the pipeline.

@denver1117

Thank your submitting the example. :)

# train sgd model
sgd2 = DistGridSearchCV(
    sgd,
    sgd_dict, sc,
    scoring=scoring, cv=val_split
    )
sgd2.fit(train_feat,y_train)

# train pac
pac2 = DistGridSearchCV(
    pac,
    pac_dict, sc,
    scoring=scoring, cv=val_split
    )
pac2.fit(train_feat,y_train)

This works.... (with both models)
model = Pipeline(
    steps=[("vec", feature_pipeline), ("clf", sgd2)]
    )
pred = model.predict(X_test)

This does not...
# assemble voter and pipeline
voter = SimpleVoter(
    [("sgd2", sgd2),("pac2", pac2)],
    classes=sgd2.classes_,
    voting="hard"
    )

model = Pipeline(
    steps=[("vec", feature_pipeline), ("clf", voter)]
    )
pred = model.predict(X_test)

I can't seem to upload the trackback file :(
Here it is:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<command-1349179484758455> in <module>()
      9     steps=[("vec", feature_pipeline), ("clf", voter)]
     10     )
---> 11 pred = model.predict(X_test)

/databricks/python/lib/python3.6/site-packages/sklearn/utils/metaestimators.py in <lambda>(*args, **kwargs)
    114
    115         # lambda, but not partial, allows help() to work with update_wrapper
--> 116         out = lambda *args, **kwargs: self.fn(obj, *args, **kwargs)
    117         # update the docstring of the returned function
    118         update_wrapper(out, self.fn)

/databricks/python/lib/python3.6/site-packages/sklearn/pipeline.py in predict(self, X, **predict_params)
    420         for _, name, transform in self._iter(with_final=False):
    421             Xt = transform.transform(Xt)
--> 422         return self.steps[-1][-1].predict(Xt, **predict_params)
    423
    424     @if_delegate_has_method(delegate='_final_estimator')

/databricks/python/lib/python3.6/site-packages/skdist/postprocessing.py in predict(self, X)
     76                 lambda x: np.argmax(
     77                     np.bincount(x, weights=self._weights_not_none)),
---> 78                 axis=1, arr=predictions)
     79         maj = self.le_.inverse_transform(maj)
     80         return maj

/databricks/python/lib/python3.6/site-packages/numpy/lib/shape_base.py in apply_along_axis(func1d, axis, arr, *args, **kwargs)
    378     except StopIteration:
    379         raise ValueError('Cannot apply_along_axis when any iteration dimensions are 0')
--> 380     res = asanyarray(func1d(inarr_view[ind0], *args, **kwargs))
    381
    382     # build a buffer for storing evaluations of func1d.

/databricks/python/lib/python3.6/site-packages/skdist/postprocessing.py in <lambda>(x)
     75             maj = np.apply_along_axis(
     76                 lambda x: np.argmax(
---> 77                     np.bincount(x, weights=self._weights_not_none)),
     78                 axis=1, arr=predictions)
     79         maj = self.le_.inverse_transform(maj)

TypeError: Cannot cast array data from dtype('<U4') to dtype('int64') according to the rule 'safe'

Thanks. This looks like a bug in sk-dist. I'll work on a fix.

This has been addressed by this PR: https://github.com/Ibotta/sk-dist/pull/28 and is now reflected in version 0.1.5.
Thanks for the bug report.


JS CORS policy question

When I try to deploy a JS bot to the gov env I see this error:

It's not clear to me if the issue is JS, the SDK, gov, or yes.

@Virtual-Josh Is nodeparentbot2 a Web App Bot or a Bot Channels Registration?

It's an app service + channel registration.

@Virtual-Josh can you share some more details around the Bot deployment? CORS requires both server and client support so there is some potential for server configuration impacting this.

Closing due to inactivity, feel free to reopen if there are any more details to share.
In effect, what's happening is that the browser is making a CORS preflight request with, among other headers, an Origin header equal to location.origin (you can check the value via the Javascript console in the browser). The server responding to this pre-flight request must include a response header with Access-Control-Allow-Origin: the-same-origin.
If the Javascript running inside the browser is making a complex request, several more headers will be included in the preflight request, which could impact the server's calculation of whether to return Access-Control-Allow-Origin: the-same-origin or to deny the request.
The summary of this is that this is almost certainly on the server handling requests for https://portal.azure.us and not in the SDK.