Removing unPrefixed class from root element.

with the css rule ".unPrefixed * { display:none;}" and setting the class unPrefixed on the root element it's now possible to hide everything from the browser as long as the stylesheets are not ready.

What are the use cases for this?

I am working on a website that includes a bunch of css stylesheets that can get rather large both in size (for example they include images as data urls) and number.
While the vendor-specific prefixes aren't present, the page looks rather ugly, and on slower user-agents (both in processing power and conenction speed, think for example older netbooks connected through a not-3g phone) there is enought time for someone to start inserting data into forms and clicking around on the unstyled page.Especially column-count and box-sizing can look rather bad and cause pleny of confusion.. If we add transitions (which clearly get re-evaluated when the new styles are computed, thus confusing the user even more) the site looks unacceptable for a while (not, long, but enough to be noticed by everyone).
If I could hide the affected items as long as the prefixes aren't there it would already be a big improvement.

Are you sure you're including the script right after the stylesheets? Do you have any examples of this behavior uploaded anywhere?

I'm including it right after the last stylesheet..
		<link rel='stylesheet' id='layout_stylesheet_23'  href='layout23.css'/>
		<script src="prefixfree.js"></script>

But even if I add this, it won't work as expected. The unprefixed class gets added BEFORE the stylesheets get processed, just as long as the XHR requests get fired. :(
But I'd really want to look more into this, so if you could create an issue with a linked example that's that slow I might be able to look into it and improve performance.


pd.concat() from list of DataFrames read with pd.read_hdf() without .copy() results in 100X slower concat!

pandas: 0.17.1
numpy: 1.9.2

I have two example DataFrames written to hdf5 files on disk. I use pd.read_hdf() to I/O them into a list. When I attempt to pd.concat([list,of,frames]) it takes 100X longer if I don't explicitly call .copy() right after the pd.read_hdf(). :confused:
Did something change in the pd.read_hdf() implementation to only reference data on disk instead of actually reading it into memory? This was a really subtle performance issue that threw a wrench in my data flow on the latest pandas stack. Profiling shows the bottleneck is in method 'ravel' of 'numpy.ndarray'  (when you don't call copy().) This does not happen in pandas 0.13.1 (I know very old). :anguished:
import os
import pandas as pd
import numpy as np

rows, cols, skip, frames = 777, 10000, 250, 2

def create_df():
    ncols = cols - np.random.randint(0,skip,1,)
    nrows = rows - np.random.randint(0,skip,1,)
    df = pd.DataFrame(np.random.randn(nrows, ncols), columns=[('SUPER_LONG_COLUMN_NAME_NUMBER_THAT_IS_VERY_LONG_'+ str(i))*2 for i in range(ncols)]).astype(np.float32)
    df.iloc[100:25:500,1000:100:2000] = np.nan
    df.iloc[:,-10:] = 'TEST_STRING_COL_VALUE'
    df.iloc[1:,:100] = np.nan
    df.iloc[10:, 100:200] = np.nan
    return df[np.random.choice(df.columns, ncols-skip, replace=False)]

%time dfs = [create_df() for i in range(frames)]
Wall time: 508 ms

%%time
for i, df in enumerate(dfs):
    df.to_hdf(r'./dfs/{}.h5'.format(i), 'df', mode='w', complib='lzo', complevel=9)
Wall time: 508 ms

%time dfs_read_hdf = [pd.read_hdf(r'./dfs/{}'.format(fn), mode='r') for fn in os.listdir(r'./dfs/')]
Wall time: 247 ms

%time dfs_read_hdf_copy = [pd.read_hdf(r'./dfs/{}'.format(fn), mode='r').copy() for fn in os.listdir(r'./dfs/')]
Wall time: 337 ms

%time dfa = pd.concat(dfs_read_hdf, ignore_index=True)
Wall time: 34.6 s

%time dfa = pd.concat(dfs_read_hdf_copy, ignore_index=True)
Wall time: 304 ms

%prun pd.concat(dfs_read_hdf, ignore_index=True)

Here is the %prun of the slow concat. Notice that a whopping 30 seconds is being spent in np.ravel() :exclamation:
298220 function calls (298178 primitive calls) in 34.758 seconds

   Ordered by: internal time

   ncalls  tottime  percall  cumtime  percall filename:lineno(function)
     1431   30.263    0.021   30.263    0.021 {method 'ravel' of 'numpy.ndarray' objects}
      955    3.619    0.004   34.144    0.036 internals.py:4461(get_empty_dtype_and_na)
     1419    0.277    0.000    0.277    0.000 {pandas.algos.take_2d_axis1_float32_float32}
     1423    0.118    0.000    0.144    0.000 common.py:230(_isnull_ndarraylike)
     3337    0.031    0.000    0.031    0.000 {method 'reduce' of 'numpy.ufunc' objects}
     1910    0.027    0.000   30.476    0.016 internals.py:4767(is_null)
    62628    0.024    0.000    0.049    0.000 {isinstance}
     1427    0.024    0.000    0.361    0.000 common.py:697(take_nd)
  ...

INSTALLED VERSIONS
------------------
commit: None
python: 2.7.10.final.0
python-bits: 64
OS: Windows
OS-release: 2008ServerR2
machine: AMD64
processor: Intel64 Family 6 Model 63 Stepping 2, GenuineIntel
byteorder: little
LC_ALL: None
LANG: None

pandas: 0.17.1
nose: 1.3.7
pip: 7.1.2
setuptools: 17.1.1
Cython: 0.23
numpy: 1.9.2
scipy: 0.16.0
statsmodels: 0.6.1
IPython: 4.0.0
sphinx: None
patsy: 0.4.0
dateutil: 2.4.2
pytz: 2015.4
blosc: None
bottleneck: 1.0.0
tables: 3.2.1
numexpr: 2.4.3
matplotlib: 1.5.0
openpyxl: 2.2.5
xlrd: 0.9.4
xlwt: 1.0.0
xlsxwriter: None
lxml: 3.4.4
bs4: 4.4.0
html5lib: None
httplib2: None
apiclient: None
sqlalchemy: 1.0.8
pymysql: None
psycopg2: None
Jinja2: None

I think this was fixed by https://github.com/pydata/pandas/pull/11967 which was merged a few weeks ago, but can you double check?
Here are my timings
In [8]: %time dfs_read_hdf = [pd.read_hdf(r'./dfs/{}'.format(fn), mode='r') for fn in os.listdir(r'./dfs/')]
CPU times: user 355 ms, sys: 28.9 ms, total: 384 ms
Wall time: 391 ms

In [9]: %time dfs_read_hdf_copy = [pd.read_hdf(r'./dfs/{}'.format(fn), mode='r').copy() for fn in os.listdir(r'./dfs/')]
CPU times: user 408 ms, sys: 76.5 ms, total: 484 ms
Wall time: 490 ms

In [10]: %time dfa = pd.concat(dfs_read_hdf, ignore_index=True)
CPU times: user 552 ms, sys: 38.9 ms, total: 591 ms
Wall time: 603 ms

In [11]: %time dfa = pd.concat(dfs_read_hdf_copy, ignore_index=True)
CPU times: user 421 ms, sys: 44.9 ms, total: 466 ms
Wall time: 469 ms

@TomAugspurger Thanks for checking, your results look promising. I guess the copy fixes the ordering for now but wastes memory.
Are there any daily pandas build binaries for windows? Compiling from source on windows is a challenge.

Are you using conda? conda install -c https://conda.anaconda.org/pandas/channel/dev pandas should track our master branch pretty closely.

Windows builds are only once in awhile
but I will put some up soon

Going to close for now since I was able to replicate the slow timings on 0.17.1 and fast timings on master. 0.18 isn't too far off, so let us know if / when you have a chance to test.


css: Fix positioning of text elements in message row without sender.

When I changed the padding for message content in the PR which converted the message row to a grid, I forgot to adjust the position of edited and time elements.

after rebasing on #23696

Merged, thanks @amanagr!


doc: link points to google internal page

the link to the pricing page in step 10 of the docs https://codelabs.developers.google.com/codelabs/firebase-cloud-functions/index.html?index=..%2F..%2Findex#9
points to
https://firebase-dot-devsite.googleplex.com/pricing/
which does not seem to be the official site.

+1, noticed the same issue

Thanks for the heads up @adegreiff and @pbusswood, updated.


Minimal hardware for GPS simulation

Hi there,
I’m a beginner in the world of radio so I want first to ask is possible to do what I want… then I’ll study how to do it.
What I want is to simulate a GPS position of my home,
in order to use my mobile phone to login on my office work (I work remotely).
Like this, I can login at work office when I’m not exactly on my home position.
Please indicate me if you want, the possibility to do this with a minimal hardware cost.
I don’t need to do other radio thinks… I don’t need to receive other RF freq.
I just need to simulate the GPS for about 20 seconds until I login in my work office remotely.
Have you a link that explain me step-by-step how to simulate GPS using minimal hardware please ?
Thanks a lot

This may help: https://drfone.wondershare.com/fake-location/gps-spoofing-with-hackrf-from-windows.html

Thanks a lot for the link... I'll study it.
Now I just search the model of HackRF to buy :-)
Thanks again
Denis


Patterns and records are missing some IDE features like autocomplete, highlight usages under caret, ctrl click go to, and quick docs on hover

Most IDE features work for most interactions with patterns and/or records, but there are a few cases that seem to be missing support.
The following examples were done with the following environment:

Dart 3.3.4 (stable) (Tue Apr 16 19:56:12 2024 +0000) on "windows_x64"
IntelliJ IDEA 2023.1.2
Dart IntelliJ plugin 231.9065

Autocomplete
Place the caret after ba and before the ; and trigger autocomplete (ctrl+space) - the list of suggestions should include bar, but it does not.
// Destructure object in for-loop
for (final Foo(:bar) in [Foo(0)]) {
  ba;
}

// Destructure named record in for-loop
for (final (:bar) in [(bar: 0)]) {
  ba;
}

// Destructure positional record in for-loop
for (final (bar,) in [(0,)]) {
  ba;
}

Show type info in quick doc
Place the caret inside the left-hand-side bar and trigger quick docs (ctrl+q) - the quick docs window should popup and show type information, but instead the popup says No documentation found.
// Destructure object
{
  final Foo(:bar) = Foo(0);
}

// Destructure named record
{
  final (:bar) = (bar: 0);
}

Control+click go to
This one's a bit less clear for records (compared to classes) due to positional fields and a potential lack of explicit type annotation, but at least for cases like below, it seems like control+clicking on .bar and .$1 should be pretty unambiguous. Pretty much anything is better than nothing :P
{
  final x = (bar: 0);
  x.bar;

  void foo(({int bar}) params) {
    params.bar;
  }
}

{
  final x = (0,);
  x.$1;

  void foo((int,) params) {
    params.$1;
  }
}

Highlight usages under caret
Place the caret inside any of the variables below (foo, bar, or $1) - all usages of that symbol within its scope should be highlighted, but they aren't. Note that code like bar; bar; is to show that neither will trigger a highlight. Also note that removing usages triggers an unused variable warning, so at least part of the analysis is working.

// Destructure object
{
  final Foo(:bar) = Foo(0);
  bar; bar;
}

// Destructure named record
{
  final (:bar) = (bar: 0);
  bar; bar;
}

// Destructure positional record
{
  final (bar,) = (0,);
  bar; bar;
}

// Destructure list
{
  final [foo, ...bar] = [0, 0];
  foo; foo;
  bar; bar;
}

// Variable pattern in if-case
{
  if (0 case final bar) {
    bar; bar;
  }
}

// Variable pattern in switch expression
{
  final _ = switch (0) {
    final bar => bar
  };
}

// Variable pattern in switch statement
{
  switch (0) {
    case final bar: bar;
  }
}

// Named record field access
{
  final x = (bar: 0);
  x.bar; x.bar;
}

// Positional record field access
{
  final x = (0,);
  x.$1; x.$1;
}

Happy to provide screenshots and/or screencasts to help clarify any of these cases!

Idk if I did myself a disservice by grouping these into one issue, but if I had to pick one that's the most annoying, it's the missing control+click go-to.
As another concrete example, this was the code I was just typing that prompted me to check back on this issue:
I have a class that just exists as a namespace for some constants:
class JustANamespace {
  static final some = 1;
  static final variables = '2';
  static final and = false;
  static final stuff = 3;
}

final hello = JustANamespace.some; // ctrl+click on `some` brings you to the declaration

I decided I wanted to nest some of these in their own namespace:
class JustANamespace {
  static final nested = (
    some: 1,
    variables: '2',
  );
  static final and = false;
  static final stuff = 3;
}

final hello = JustANamespace.nested.some;  // ctrl+click on `nested` works, but ctrl+click on `some` does nothing


Add documentation for running cadence-web on VSCode Dev Containers

Add instructions to README.md for using VSCode Dev Containers with Docker for running Cadence Web

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Underscore dependency has high security vulnerability in botbuilder-azure

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Versions
What package version of the SDK are you using. 4.13.2
What nodejs version are you using 10.24.1
What os are you using Windows
Describe the bug
High            Arbitrary Code Execution
Package         underscore
Patched in      >=1.12.1
Dependency of   botbuilder-azure
Path            botbuilder-azure > documentdb > underscore
More info       https://npmjs.com/advisories/1674
To Reproduce
Steps to reproduce the behavior:

run NPM Audit

Expected behavior
No high security vulnerabilities
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

We haven't seen anything reported yet. Would you mind opening a separate issue with those details so we can investigate?

Issue with transcript logger resolved via: https://github.com/microsoft/botbuilder-js/issues/3684


Flip score value is not getting copied correctly when a new copy feature is added.

Flip score value is not getting copied correctly when a new copy feature is added.

ah, good catch - those params need to be added to Clone()


dockerize the package

Having the necessary dependencies, like nodejs and dotnet packed into a single docker file would be useful for portability.

The new browser editor superseeds this, as that doesn't need any backend at all. I'm shifting the priority onto making this project work in the browser - at least partially.


Walking gate has not been tested with new gyro

PID added on gyro, oscillation is no longer present


Limited Emoji Reactions Available in Zulip Mobile App

When using the Zulip mobile app, I've observed that there's a limited selection of emoji reactions available when reacting to a message. Currently, only the "OK" emoji with a finger is available for reactions. Having a broader selection of emoji reactions would greatly enhance communication within the app and allow users to express a wider range of emotions.
Steps to Reproduce:
1/Open the Zulip mobile app.
2/Navigate to a message that allows reactions.
3/Attempt to react to the message and observe the available emoji options

Hi @Syrineladeb02, thanks for the report, especially the effort in creating the screenshot and reproduction recipe.
This feature is already tracked in #388, so I'll close this as a duplicate.


interp: don't panic for undefined types

This prevents the runtime panic that otherwise occurs when a symbol that is referenced in a source is not present in the symbol map.
Please take a look.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I was unable to exercise this in the interactive interpreter. However, this will show the effect.
https://play.golang.org/p/teHHMR1Ta4h
package main

import (
	"fmt"
	"log"
	"reflect"

	"github.com/containous/yaegi/interp"
)

func main() {
	type Sub struct {
		S string
	}
	type Config struct {
		N int
		F float64
		X Sub
	}
	i := interp.New(interp.Options{})
	i.Use(interp.Exports{"xcfg": map[string]reflect.Value{
		"Value": reflect.Zero(reflect.TypeOf(&Config{})),
		// "Sub":   reflect.Zero(reflect.TypeOf(&Sub{})), // Bang!
	}})
	_, err := i.Eval(`import "xcfg"`)
	if err != nil {
		log.Fatal(err)
	}
	v, err := i.Eval(`xcfg.Value{
		N: 5, F: 0.1,
		X: xcfg.Sub{S: "set"},
}`)
	if err != nil {
		log.Fatal(err)
	}

	fmt.Printf("%#v", v)
}

If you uncomment the line marked "Bang!" you'll see this otherwise works. Adding the change proposed here results in a returned error rather than a runtime nil-deref panic.

Could add the test file: https://github.com/containous/yaegi/tree/master/_test

I need to look at it more closely: you are defining wrappers using reflect.Zero(reflect.TypeOf(SomeType)) instead of reflect.ValueOf((*SomeType)(nil)) as normally generated by goexports. I need to assess the impact of that.

You can replace the approach I am using with what you have. They are identical in effect.

Could you add the test file: https://github.com/containous/yaegi/tree/master/_test

This is not appropriate. The code at the reproducer is not for the interpreter, it is for host compiled Go code. I can add a test though.


Improvement: Output the login-name alongside of the username for resubscription notices

When handling automated actions for your Users the login name is (way more) useful for that than the username, especially if that one is localized.
Seeing as the Info is there, but just not emitted i think it would be useful to output that, at least for resubs.

While at it, is this somehow (properly / easily) implementable for subscription events?

Please follow the commit message format in the contributing file (<scope>: subject) and you will need to target the v1.2.x branch for the pull request.
I think this would be better if the "userstate" object was supplied as the tags are available similar to a normal chat message rather than just adding more arguments.
Then there's 2 options:
Keep months separate:

channel: String - Channel name
user: Object - Userstate object
months: Number - How many months
message: String - Custom message

Put months into the userstate:

channel: String - Channel name
user: Object - Userstate object
message: String - Custom message

It would be worth it to find anywhere that the userstate is available like in this case for other events.

Having the userstate object would indeed be better for uniformity, but in terms of backwards compatability i think it would be better to add the userstate object as a new argument until 2.0.
I fixed the title but seeing as my whole concept is off i think i am going to create a new PR for emitting the userstate object as a new argument, this time with the correct branch and title :laughing: (If you agree ofc.)

Also, for subscriptions there is no userstate, which is why i asked if there would be some possible workaround

The title of the pull request doesn't affect the commit message which should be similar to "client: add login name to resub event."

I really should more often work with GIT. I hope this works.

merged (up to date) v1.2.x into my local master, since i cant change the HEAD of the PR.


Jesse: Watch fastechy.com

Jesse requests the watch of the watch_keyword fastechy\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

fastechy\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Sup chief

Hey man just reaching out as I noticed you have the name "Aimer" on Minecraft. I found your github by using the minecraft name history to find your previous name. I was just wondering if you'd be interested in selling the account, I quite like the name and would pay a premium for it. Lemme know. Thanks

Sorry, man. I lost access to that account a long time ago. It got pwned somewhere along the way. Otherwise, I would have given it to you, but no luck there.


py> operator doesn't work with continuum/miniconda3 Docker image

While py> operator with Python official docker image works fine, the digdag workload fails with error using anaconda Docker image. Anaconda is a popular Python distribution especially for data science and machine learning. It would be nice if anaconda image works fine for ML workload on digdag server.
+run:
  docker:
    image: 'continuumio/miniconda3'
  py>: simple_task.run

def run():
    print("Hello py> operator!")

if __name__ == '__main__':
    run()

Error logs on digdag server:
$ digdag push pyop-test && digdag start pyop-test py-test --session now
``

```error.log
2018-09-28 14:48:15 +0900 [INFO] (XNIO-1 task-14): Starting a new session project id=1 workflow name=py-test session_time=2018-09-28T05:48:15+00:00
2018-09-28 14:48:20 +0900 [INFO] (0063@[0:pyop-test]+py-test+run): py>: simple_task.run
[FATAL tini (6)] exec python failed: No such file or directory
2018-09-28 14:48:22 +0900 [ERROR] (0063@[0:pyop-test]+py-test+run): Task failed with unexpected error: Python command failed with code 127
java.lang.RuntimeException: Python command failed with code 127
	at io.digdag.standards.operator.PyOperatorFactory$PyOperator.runCode(PyOperatorFactory.java:153)
	at io.digdag.standards.operator.PyOperatorFactory$PyOperator.runTask(PyOperatorFactory.java:91)
	at io.digdag.util.BaseOperator.run(BaseOperator.java:35)
	at io.digdag.core.agent.OperatorManager.callExecutor(OperatorManager.java:312)
	at io.digdag.core.agent.OperatorManager.runWithWorkspace(OperatorManager.java:254)
	at io.digdag.core.agent.OperatorManager.lambda$runWithHeartbeat$2(OperatorManager.java:137)
	at io.digdag.core.agent.ExtractArchiveWorkspaceManager.withExtractedArchive(ExtractArchiveWorkspaceManager.java:77)
	at io.digdag.core.agent.OperatorManager.runWithHeartbeat(OperatorManager.java:135)
	at io.digdag.core.agent.OperatorManager.run(OperatorManager.java:119)
	at io.digdag.core.agent.MultiThreadAgent.lambda$null$0(MultiThreadAgent.java:127)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:514)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1135)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
	at java.base/java.lang.Thread.run(Thread.java:844)
2018-09-28 14:48:27 +0900 [INFO] (0063@[0:pyop-test]+py-test^failure-alert): type: notify

If I use python 3.7.0 official image, digdag workload works fine.
2018-09-28 14:48:27 +0900 [INFO] (0063@[0:pyop-test]+py-test^failure-alert): type: notify
2018-09-28 15:10:31 +0900 [INFO] (XNIO-1 task-22): Starting a new session project id=1 workflow name=py-test session_time=2018-09-28T06:10:31+00:00
2018-09-28 15:10:37 +0900 [INFO] (0065@[0:pyop-test]+py-test+run): py>: simple_task.run
Hello py> operator!
<stdin>:4: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
<stdin>:111: DeprecationWarning: inspect.getargspec() is deprecated, use inspect.signature() or inspect.getfullargspec()

Files are put as follows:
$ tree .
.
├── py-test.dig
└── simple_task.py

0 directories, 2 files

You can see detailed contents at gist:
https://gist.github.com/chezou/bdf1729bcf4254dcffcf7558b53a42e5
Version info:
$ digdag --version
0.9.30
$ java --version
java 10.0.2 2018-07-17
Java(TM) SE Runtime Environment 18.3 (build 10.0.2+13)
Java HotSpot(TM) 64-Bit Server VM 18.3 (build 10.0.2+13, mixed mode)
$ docker --version
Docker version 18.06.1-ce, build e68fc7a

Hello, @chezou
Could you try the following?

Set the following PATH environment variable on digdag server.
Start Digdag server
Run digdag push pyop-test && digdag start pyop-test py-test --session now

Digdag use server PATH envirionment variable on Docker image.
And your environment variable may not set /opt/conda/bin in Digdag server.
(This path the python location in continuumio/miniconda3 image).
export PATH="/opt/conda/bin:$PATH"
digdag server -m

digdag push pyop-test && digdag start pyop-test py-test --session now

Note
This issue also related #609 #611 and #680

@hiroyuki-sato Thanks for your immediate response! I confirmed your work around can avoid the issue.
Just out of curiosity, is it standard way to set PATH environment for specific docker image before starting server? I worries this prerequisite may cause annoying negotiation between administrator of digdag server and users.

@chezou
It's just the workaround. I'm not sure detail but #835 will introduce new CommandExecutor.
It may solve a part of your issue.
The Developer will talk about Digdag in the upcoming event  TDTech in Japan. I'll tell him about this behavior.
@muga What do you think? Does CommandExecutor v2 solve this issue?

@chezou @hiroyuki-sato thank you for reporting. At least the issue could be reproduced on v0_10 branch. I will take a look to find a way to avoid this issue.

Filed https://github.com/treasure-data/digdag/pull/890

I'm closing this ticket because the issue can be fixed by Digdag v0.9.31 released recently. Please feel free to reopen it if need.


[Android] "Unable to dispatch keyboard events in JS as the react instance has not been attached" when add Debug.startMethodTracing in MainActivity

I am using React Native 0.17.0 on Android 5.0.1.
When I add Traceview Code Debug.startMethodTracing("calc"); in MainActivity's onCreate()  method ,and When I launch my app, I got the following logs:
          EGL_emulation  W  eglSurfaceAttrib not implemented
         OpenGLRenderer  W  Failed to set EGL_SWAP_BEHAVIOR on surface 0xa6c87a20, error=EGL_SUCCESS
          unknown:React  W  Unable to dispatch keyboard events in JS as the react instance has not been attached

And My app got stuck without any Rendering(When I pause my app,The UI rendered).
And When I remove the Traceview Code,all works fine.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/android-unable-to-dispatch-keyboard-events-in-js-as-the-react-instance-has-not-been-attached-when-add-debugstartmethodtracing-in-mainactivity
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub. GitHub issues have voting too, nevertheless
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


Implement installation_target webhook event type

Implements the installation_target webhook event type.
Description from the API:

This event occurs when there is activity relating to the user or organization account that a GitHub App is installed on

Thank you, @raynigon !
Merging.

When can we expect a release to be published with this change?

@mnafees - it would be nice to get a couple outstanding PRs merged in before the release since we already have breaking changes. Would you be interested in reviewing these two PRs?

#2836
#2842


[feature] Re-evaluate if we need to record the GH context in setup-token Action

I think we can probably drop it. We should probably white list what we include anyway since we probably don't want to carry any potentially sensitive fields.

if we want to support branches for verifier, we'll need to port the logic from the verifier to here. We could also start without branch support for the first release.

I thought this was for the token - not for the predicate.

+1, I confused the two features. Is recording sensitive fields riskier in the token or the provenance, or same? :)

I think it's the same - they're both posted to Rekor and somewhat public.
So right now we pick and choose the env fields, and record the entire event payload. This gives us branch support, btw.
We can redact more of the event payload, since the only fields we really need for branch support are the base ref, trigger_committish and maybe 1 more of the event payload.
The sensitive info parts are inside larger structs (repository -> owner) of the event payload.

that brings up another question: what do we do with sensitive fields that are necessary inputs in the token https://github.com/slsa-framework/slsa-github-generator/issues/1575?
I forgot that they are recorded in the token too. The masking I proposed just masks them from the final provenance, but not the token.
Does rekor always record the plaintext? Or it depends on the type of record / input (attestation vs blob vs ...)?

Does rekor always record the plaintext? Or it depends on the type of record / input (attestation vs blob vs ...)?

For simple signing a blob, it does not, but the attestation content will be there. So I suppose the token is actually in a better state!

ah, great, Then no need to prioritize it. btw, where can I learn which types have their plaintext recorded?


fix(notify): default notify timeout to false

As of Version 6.0 there is a default timeout set of 10 to ensure that the application closes properly. In order to remove the timeout and have an instantly closing notification (does not support actions), set timeout to false.
- node-notifier readme

Awesome, thanks!


Validate elements before submission

Resolves #94

@jonjamz Does that look good to you? After this is merged I will follow-up with a PR that addresses the default validation on startup.

@jonjamz Have you had a chance to take a look at this yet?

@ramijarrar sorry about that. This looks good. Smart to use Blaze.getView.

@jonjamz Can you release a minor new version (2.1.3?) to include this? Thanks in advance.

Done!

Thanks. Unfortunately, the new code generates an exception on submit:
Here is the stack trace:
_templates_forms.js?hash=67d434e…:758 Uncaught TypeError: Blaze.getView(...).templateInstance is not a function
(anonymous function)	@	templates_forms.js?hash=67d434e…:758
component.submit	@	templates_forms.js?hash=67d434e…:757
evt.submit form	@	templates_forms.js?hash=67d434e…:1628
(anonymous function)	@	blaze.js?hash=38069f4…:3708
Template._withTemplateInstanceFunc	@	blaze.js?hash=38069f4…:3677
(anonymous function)	@	blaze.js?hash=38069f4…:3707
(anonymous function)	@	blaze.js?hash=38069f4…:2550
Blaze._withCurrentView	@	blaze.js?hash=38069f4…:2204
(anonymous function)	@	blaze.js?hash=38069f4…:2549
(anonymous function)	@	blaze.js?hash=38069f4…:825
jQuery.event.dispatch	@	jquery.js?hash=649d9f4…:4707
==================
The code involved is this line:
var instance = Blaze.getView(element).templateInstance();

@jonjamz While you or @ramijarrar are looking into the issue reported, could you please tag version 2.1.2 (the latest working)? That would be of great help as right now, the upgrade broke the code and cannot install the previously version. Thank you so much.

v2.1.2 is tagged. I'm on a train right now and logged in via hotspot. I may revert the changes in a v2.1.4 release, what do you think @davidecantini?

Yes, please.
Also because I'm trying to install 2.1.2 but still get 2.1.3?
> meteor add<EMAIL_ADDRESS>
Changes to your project's package version selections:

templates:forms  added, version 2.1.3

Thanks for fixing this as soon as possible.

Alright 2.1.4 is published.

I've tested with a pretty form heavy app before committing and version 2.1.3 isn't causing any errors for me. Can you elaborate more on the specific form that caused this? Also, what version of Meteor are you running on?

@jonjamz Thank you.

@ramijarrar:
I'm using Meteor 1.3.1. Here is the content of .meteor/versions:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Just using a form with some array fields, pretty normal stuff I guess.

@davidecantini What's the output of Blaze.getView on one of your form elements?

If you want me to test that on my environment, please send the html and js snippets and I'll do it within tonight/tomorrow. Thanks

Just run Blaze.getView(document.getElementById("foo"))

None of the form elements has an id, but after adding , Blaze.getView(document.getElementById("foo")) returns the view, which contains the templateInstance function property.
As expected, the exception reported above may occur only under certain circumstances.

@davidecantini Okay thanks for that. Would it be possible for you to create a repro for this?

Yes, sure. Please manage to have the test version (2.1.3 or whichever you want me to test) installable as follows:
> meteor remove templates:forms
> meteor add<EMAIL_ADDRESS>
or in any other way. The important thing is that I can switch back to the working version at any time, smoothly.

Yes that will work fine, but you should probably do this in a separate repo i.e not your main app (start with one of the example apps and see if you can recreate the error)

@ramijarrar reminder:
Please manage to have the test version (2.1.3 or whichever you want me to test) installable as follows:
> meteor remove templates:forms
> meteor add<EMAIL_ADDRESS>
Until then I cannot do test to reproduce the issue.

@davidecantini To specify an exact constraint, you need to use the = symbol. Otherwise you are specifying the range of compatible versions higher than 2.1.3 which will always include 2.1.4 (semantic versioning).
The correct command is:
meteor add<EMAIL_ADDRESS>
You can remove this constraint by running:
meteor add templates:forms

Thank you. As I'm testing it, I get the error upon submission. With version 2.1.2 (and 2.1.4), everything works well.
        // Issue #94
        formElements = self.findAll(component.elementSelectors.join(', '));
        formElements.forEach(function (element) {
          var instance = Blaze.getView(element).templateInstance();    <== This causes the exception

It would be helpful to instrument the code to have a console.log(element) print on which element the error occur, so it'd be easier to narrow down the issue.

I already know what the error is. I have not been able to reproduce this in my projects. Please create a repro so I can look into this (run meteor create submission-bug and work from there).

Ok, that's great news. If you know what the error is already, why don't you provide a fixed version, which I can quickly test to confirm that it works? That way we can get the new version fixed within today I guess.

@davidecantini Please read the full comment before responding.
Now you make me wonder if I read it too well, actually. Since you wrote that you know what the error is, I thought that you already have a fix. In that case, you could just provide a fixed version for me to quickly test and confirm the fix.
On the contrary, if all you know so far is WHERE the error is in your code but you're not able to reproduce yet, then I'm willing to help you. To do so, I want an instrumented test version where I can see the element that generates the exception, otherwise it could be done but it would take much more time to isolate culprit, and I can't go through that right now.

@jonjamz Have you been able to reproduce the issue described by @davidecantini?

@davidecantini Makes sense, I added some logging to our fork here so you can just add that as a local package and it will take precedence (delete it once you are done and everything is restored back to normal). Thanks again for your help with this.

Hi, is it possible that you are using blaze templates as children of non-blaze Elements?
I should check probably that the Blaze.getView() really returns a view. It surely does so if it is generated from within a template html/jade definition.

@paulbalomiri Can you try with v2.1.3 and see if you can reproduce the error?

@davidecantini Did you get a chance to try your app with the instrumented test version that I linked to above?

@ramijarrar Unfortunately, I didn't. Due to some changes to the project in which this error was occurring, I can't guarantee if/when it'll be possible to test. For sure, if there's a chance I'll let you know (I'm also curious to know what's causing that exception). Meanwhile, if nobody else can reproduce the issue, I don't have any problem if you guys decide to re-include your PR in a future release. In case anyone will happen to report the error, you can just provide that instrumented test version right away. That's it for now. Thanks again, let's keep in touch.


Script will crash on CVaaS if the device EOS version is in between 4.24 and 4.25.5

Currently, this is the behavior of the Custom Bootstrap Script

for EOS versions <4.24, we can provide eosURL and they will get upgraded to the provided URL
for EOS versions >=4.25.5 which is OSLO, everything works fine
for EOS versions in b/w 4.24 and 4.25.5, the TA command will fail as there is no -certsconfig in these versions, and also providing eosURL won't do any image upgrade

Currently, eosURL is only used for devices with EOS <4.24 as there is no SysdbHelperUtils there. Here, we need to handle case 3). So, for this we will redefine the behavior of eosURL parameter.
Proposed Solution:
We already say that CBS is provided for minimum OSLO versions i.e. 4.25.5, so we expect the users to provide an eosURL parameter with EOS image >=4.25.5 whenever the current EOS version is less than OSLO.
Now, instead of checking only for SysdbHelperUtils, we will always check if eosURL parameter is set or not. If it is set, then we always do an image upgrade. If it is not set, things might go wrong as expected.

Found that it is an issue in the bootstrap script and not the custom bootstrap script. Closing this.


feat: implement verifier manager

Quick summary
So the grand idea is that we will have a verificationManager that can take in verificationMethods

so each verificationMethod will have:
- a test to check if that method will run
- the verification mechanism itself

so, for the current one, we will have likely:
- OpenAttestationTamperCheck
- OpenAttestationEthereumDocumentStoreIssued
- OpenAttestationEthereumDocumentStoreRevoked
- OpenAttestationDnsTxtIdentity

it will also be loaded with:
- OpenCertsRegistryIdentity

each of these will be an object
{
  test: (Document) => boolean,
  verify: (Document) => VerificationFragment
}

A VerificationFragement will be like:

{
  type: "OpenAttestationDnsTxtIdentity",
  data: "mpa.gov.sg",
  message: "Error or warning message",
  status: "VALID" | "WARNING" | "ERROR"
}

The verificationManager will summarise all the Fragments and provide a summary as well (for those dev who are too lazy to go further)

The verification manager can also take in conditions to determine the summary status: ie Tamper && Issued && Revoke && (DNS || Registry), etc

Implementation details

verificationManager is a function that takes a list of verifiers and return a function. The function expect a signed document and some options as parameter. Options parameter expect the following:

a mandatory network string (for instance ropsten)
an optional infuraApiKey string
an optional promisesCallback callback function that will provide back the promises resolving to the verifications.

There are 4 verifiers:

openAttestationTamperCheck
openAttestationEthereumDocumentStoreIssued
openAttestationEthereumDocumentStoreRevoked
openAttestationDnsTxtIdentity

There is a defaultVerificationManager pre-built having in parameter all the verifiers available in this repository
There are 2 validations functions: isValid and isInvalid
A verifier is an object that expect 3 functions:

test: who must return true or false. The function must indicate whether condition are fulfilled for the verifier to run on a specific signed document
verify:  who must return the result of the verification as a VerificationFragment
skip: who must return the result of a verification when it's skipped by providing additional data on why the validation didn't run.

TODO

add tests

I've added tests for verifiers (testing the verifier only not internals)
Didn't add test for the managers => there are already integration tests
Removed apikey as discussed

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Content updates from EA

These edits come straight from EA and have been approved for release.
Changes

Updated doing-business-with-us and doing-business-with-us/upcoming-procurement-needs

Testing

navigation to /doing-business-with-us/ and /doing-business-with-us/upcoming-procurement-needs/. Content should match the doc in Jira DFJR-1284

Review

@schaferjh
@anselmbradford
@sebworks
@sarahsimpson09

Screenshots

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

👍

Actually. Going through the live site right now and just realized that the "Future needs" table on the live site has been updated and we need those updates reflected on Upcoming Procurement Needs --
Current site: http://www.consumerfinance.gov/doing-business-with-us/
Beta: Upcoming Procurement needs /about-us/doing-business-with-us/upcoming-procurement-needs/
Is this table in WordPress or is something that could be updated as part of this PR?

@schaferjh that part is coming from WordPress, but other than the function used to grab it, I don't have any info where it is get_document('pages', '36601'). Yesterday @sarahsimpson09 said she tried the ID swap trick in urls and came up empty.

@kurtw now that the WordPress pages was restored in #1948  will that page ID work again and be editable in WP?

Assuming Sarah can find that last thing to edit. :+1:


cvs removed from built-in exclude filter

I have a lot of files with cvs sequence inside file name. For example docvslayer.png, abbreviation dcvs_click.mp3, cvsync.atlas, cvs_log.json, etc. Now these files is ignored and not copying into build folders, I maintain these specific situations by gradle task now, but this is just workaround solution.
So I propose to remove CVS exclude filter from build-in exclude list. I think it would be much better to set cvs exclude filter only by hands in xml project file.

Hmm, I think the real issue is that filter is not supposed to act like a wildcard -- it is supposed to match exactly "cvs" and not "cvs", so I think we'll have other unexpected behavior if that is not resolved :frowning:

Since the wildcard filter was fixed in the Lime tools, I think we should be safe here. Thanks again for your help :smile:


Update to SDK 3.11.0

Blocked by https://github.com/AutonomyLab/parrot_arsdk/pull/4
Blocked by the binary release of the above
Dev branch: sdk-3-11

Fixed in #99


{AKS} Upgrade api-version to 2022-07-01 for ACS module

Related command

Upgrade api-version to 2022-07-01 for ACS module.
Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Queued a pipeline to generate recording files.

AKS

Resolving network/msi SDK/API upgrade conflicts...


Rust avoid rebuffering Stdin

cc @BurntSushi
StdinLock already implements BufRead
> hyperfine 'rust/simple/target/release/countwords < kjvbible_x10.txt'
Benchmark #1: rust/simple/target/release/countwords < kjvbible_x10.txt
  Time (mean ± σ):      1.151 s ±  0.020 s    [User: 1.138 s, System: 0.012 s]
  Range (min … max):    1.123 s …  1.179 s    10 runs

> hyperfine 'rust/simple/target/release/countwords < kjvbible_x10.txt'
Benchmark #1: rust/simple/target/release/countwords < kjvbible_x10.txt
  Time (mean ± σ):      1.150 s ±  0.019 s    [User: 1.137 s, System: 0.012 s]
  Range (min … max):    1.116 s …  1.175 s    10 runs

> hyperfine 'rust/bonus/target/release/countwords < kjvbible_x10.txt'
Benchmark #1: rust/bonus/target/release/countwords < kjvbible_x10.txt
  Time (mean ± σ):      1.308 s ±  0.066 s    [User: 1.299 s, System: 0.008 s]
  Range (min … max):    1.216 s …  1.395 s    10 runs

> hyperfine 'rust/bonus/target/release/countwords < kjvbible_x10.txt'
Benchmark #1: rust/bonus/target/release/countwords < kjvbible_x10.txt
  Time (mean ± σ):      1.252 s ±  0.025 s    [User: 1.241 s, System: 0.010 s]
  Range (min … max):    1.220 s …  1.291 s    10 runs

> hyperfine './simple-c < kjvbible_x10.txt'
Benchmark #1: ./simple-c < kjvbible_x10.txt
  Time (mean ± σ):     855.2 ms ±  31.3 ms    [User: 845.1 ms, System: 8.9 ms]
  Range (min … max):   820.5 ms … 927.1 ms    10 runs

See https://github.com/rust-lang/rust-clippy/issues/6755
https://users.rust-lang.org/t/correct-way-for-taking-large-input-from-keyboard/55304/11

@pickfire Ah nice, whoops. Thank you!

Thanks!


Vite 4 is removing import

Describe the bug
If I have an import in App.vue which is a "double" import. The imported method is removed on build. While developing it works but it is gone on build. It works if I use vite@^3. I also works if I import is imported directly
App.vue
import api from './api/index.js'
import greetings from './api/greetings.js'

methods: {
  hello() {
    console.log('App.hello)
    api.greetings.hello() // <--- this is removed on build. Using yarn dev works
    greetings.hi() // <--- this is not removed on build
  }
}

api/index.js
import * as greetings from './greetings.js'
export default {
  greetings
}

api/greetings.js
export function hello () {
  console.log('hello from greeings.js')
}

export function hi () {
  console.log('hi from greeings.js')
}

Reproduction
https://github.com/johandalabacka/vite4-build-error
Steps to reproduce
Download repo and install
git clone https://github.com/johandalabacka/vite4-build-error
cd vite4-build-error
yarn install
yarn dev

Open http://localhost:5173/ and pressing the button and the output in the log is:
App.hello
hello from greetings
hi from greetings

But if we build it:
yarn build
yarn preview

Open http://localhost:4173/ and press the button and now output in the log is only:
App.hello
hi from greetings

System Info
System:
    OS: macOS 13.0.1
    CPU: (8) arm64 Apple M1 Pro
    Memory: 71.13 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.12.0 - ~/.nvm/versions/node/v18.12.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.12.0/bin/yarn
    npm: 8.19.2 - ~/.nvm/versions/node/v18.12.0/bin/npm
  Browsers:
    Chrome: 108.0.5359.98
    Firefox: 107.0.1
    Safari: 16.1
  npmPackages:
    @vitejs/plugin-vue: ^4.0.0 => 4.0.0
    vite: ^4.0.0 => 4.0.1

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This was an upstream issue. I created an issue to rollup (https://github.com/rollup/rollup/issues/4751).

@johandalabacka , I did run your code, all u need to do is to add the base property to vite.config.js
// pasted here
import { defineConfig } from 'vite'
import vue from '@vitejs/plugin-vue'
// https://vitejs.dev/config/
export default defineConfig({
base: "./",
plugins: [vue()],
})

@Foster0123 Thanks. I try it

did u run the build on a server ??

u need a local server to serve the build else the script will be blocked by CORS

@Foster0123  I'm not sure what you mean. Running yarn build and then yarn preview starts a local server serving the built files. And I watched in the built code and the call is missing. I also tried http-server dist and this didn't work either. The output in the log is
App.hello
hi from greetings

with hello from greetings is missing in both cases

Closing as this is fixed in rollup 3.7.5.


v0.4.0

Describe your changes

Add better support for multi-panel graphs via mpanel_e61() (referring to combining multiple discrete graphs into a single panel, not faceted graphs).
Add dual y-axis support allowing for graphs with secondary y-axes with different scales to be created.
Add experimental automatic graph height calculation in save_e61(), which should reduce the need for users to manually specify height when saving graphs.
Correct whitespace margins around graphs.
Improvements to various documentation.

Do the following before requesting a review

[x] I have written tests covering functions I have added or changed.
[x] I have run tests and they all pass.
[x] I have ensured all new functions will show up in the _pkgdown.yml file.
[x] I have updated DESCRIPTION with any new package dependencies.
[x] I have updated NEWS.md with a brief description of my changes.
[x] I have updated the package documentation with devtools::document().
[x] I have updated the package website with pkgdown::build_site_github_pages().

@JackBuckley I'm done. Please review any time before Friday morning.


Which tool is used for creating the architecture graph?

Thanks!

Hi, it was https://app.diagrams.net :)

Thanks.


Revert "Improving the API (without token usage) and memory store"

Reverts C-Viewer/CViewer#3

The wrong base branch was choosen


add preloaders option

There are loaders built into esbuild that cannot be controlled (unlike loader plugins, no options can be specified), in some cases their default behavior may not work as expected, and requires some preparation of the source code before this code is processed appropriate loader.
it would seem that this is a mere trifle. After all, there is probably an option that would disable the processing of such links at the time of building, putting this part at the discretion of the developer. But the loader did not have such an option. Just as there is no option to manually set the root directory to search for the images.
It would seem like a good idea for such a narrow purpose to write an independent plugin that would deal with the processing of absolute paths in styles. But as it turned out, esbuild does not provide chain passing of execution from plugin to plugin, and thus processed files after processing went to the loader, and not to the svelte-esbuild plugin, and this completely spoiled the whole game.
And it seems to me that since it is impossible to influence loaders and introduce such functionality using plugins, it would be nice to make in such cases in the svelte-esbuild plugin itself the possibility of pre-processing the generated sources before they get processed by the corresponding loaders.
So, I suggest the following usage:
esbuildSvelte({
   preprocess: sveltePreprocess(),
   compilerOptions: {},
   preloaders: {
      css: (content, opts) =>{
         // changes absolute paths to relative
         const code = content.replace(/(\/static\/images)/g, function handleReplace(match) {

            let dirname = path.dirname(process.argv[1])
            let relPath = path.relative(opts.path, dirname + '/scripts') + '/images'
            return relPath;
         });
         return code;
      }
   }
})

For example the above code changes all paths with /static/images to relative path depending on its location position

Interesting! Note that Svelte preprocessors do chain (see https://svelte.dev/docs#compile-time-svelte-preprocess). Any reason you couldn't do this?:
const replaceProcessor = {
  style: ({content, filename}) => {
     // changes absolute paths to relative
     const code = content.replace(/(\/static\/images)/g, function handleReplace(match) {

        let dirname = path.dirname(process.argv[1])
        let relPath = path.relative(filename, dirname + '/scripts') + '/images'
        return relPath;
     });
     return {code};
  }
}

esbuildSvelte({
   preprocess: [replaceProcessor, sveltePreprocess()],
})

I think this could potentially solve your issue, without adding additional complexity to this plugin which I'll admit I'm hesitant to do.
If this doesn't solve your issue, we can talk further. I'm wondering if providing control of the loader used for css could also be a solution to this...

Note that Svelte preprocessors do chain

It is wonderful. Indeed, your solution looks more competent.

I think this could potentially solve your issue, without adding additional complexity to this plugin which I'll admit I'm hesitant to do.

Apparently, there is no reason to make the plugin more difficult since this can be done through a custom preprocessor

Excellent! Glad to hear it. I'll go ahead and close this PR then.


Add Citron XCT

https://explorer.binance.org/asset/XCT-3E6M

{
"asset": "XCT-3E6M",
"assetMap": "XCT",
"officialSiteUrl": "https://citron.cash",
"contactEmail"<EMAIL_ADDRESS>"display": true,
"market": "ALTS",
"contact": {
"Telegram": "https://t.me/citroncash",
"Coinmarketcap": "https://coinmarketcap.com/currencies/citron/",
"Medium": "https://medium.com/@citroncash",
"Twitter": "https://twitter.com/citroncash",
"Facebook": "https://www.facebook.com/citroncash/",
"Github": "https://github.com/citroncash"
}
}

please add market info:
"market": "ALTS",

done


npub strings comparator: profile verification without NIP-05

I need of an empty field on any user profile page where pasting the public key of a contact that I want verifying his authenticity without a NIP-05 username.
In short I want to compare two npub strings: one is already present in the profile page I’m visiting, the other is in my clipboard taken (for example) from the Mastodon page of the person I follow.
In the empty field I paste the one I took from Mastodon, and if the two keys match the inscription "match" appears.
If they don’t match an alert is showed.

Great idea!


Add a testcase for already fixed nested property assignment bug

Nested property assignment bug #427 happens in Duktape 1.3.0 but is already fixed in master. This pull provides a test case for the bug, which was probably fixed in #381.

Now that you have a testcase, it might be interesting to do a reverse bisection to figure out exactly which commit fixed the bug.

It is 99% certainly the one for #381, because there was a bug there in assignment RHS handling.


Controls

Unfortunately still having an issue with the controls in version 3.0 on OS X El Capitan 10.11.5 (running on a Macbook Pro Retina mid 2014)

Media keys still don't respond (they do in other apps), I don't have Chrome installed so that eliminates interference from the extension. Even so I didn't have the extension installed while I had Chrome.

Still unable to reset hot keys, page is completely unresponsive once opened.

Voice controls never worked (but I expect that from this experimental feature so thats not really an issue)

I've uninstalled the app and installed once more and reset my computer. Neither helped.

I really hope you mean 3.3 😆

Media keys still don't respond

The media keys will not work if another Chromium based application has "claimed" them.  You might want to check any other Electron based applications.

Still unable to reset hot keys, page is completely unresponsive once opened.

These issues are starting to get annoying, a quick search of the pre-existing issues explains how to use that page.  But I'll put them here to save time...

Click the box for the shortcut you want to set
Perform the shortcut (Something like "Control + Shift + P")
Tada

Maybe we should expand that disclaimer on the hotkeys settings tab to include those restrictions...


fix: Make base to base sswap with full spread

Fix a potential issue from Zellic audit report.

DIFF: https://www.diffchecker.com/Z7pnbgYC/


uod-web: enable configurable re-deployments

For example:
 ansible-playbook -i inventory uod-web.yml -l www-dev -e dev_site_account=joshmoore -e dev_site_branch=master  -t quickdeploy

Works for me

Thoughts on the tag strategy? Would it be better to extract this to a subplaybook?

(ansible) sbesson@ls30630:ansible ((89d800c...)) $ time ansible-playbook -i inventory/openstack.py -e vm_key_name=seb-dundee -e vm_name=www-dev uod-web.yml -t quickdeploy

PLAY [OME Website Setup] *******************************************************

TASK [setup] *******************************************************************
ok: [9024a592-1b6c-446c-abe6-fbd6551dc693]

TASK [Clone www-dev.openmicroscopy.org source code repository] *****************
ok: [9024a592-1b6c-446c-abe6-fbd6551dc693]

TASK [Build the website using Jekyll] ******************************************
changed: [9024a592-1b6c-446c-abe6-fbd6551dc693]

TASK [Update Nginx configuration for www-dev.openmicroscopy.org] ***************
ok: [9024a592-1b6c-446c-abe6-fbd6551dc693]

PLAY RECAP *********************************************************************
9024a592-1b6c-446c-abe6-fbd6551dc693 : ok=4    changed=1    unreachable=0    failed=0

real	0m13.514s
user	0m2.519s
sys	0m0.825s

so everything works as expected. With regard to tags, it is nice. But my feeling is that trying to maintain it in a complex playbook will not scale. Instead we might work towards extraction the sections of the playbook using include or even role first and then annotate these with tags in the playbook (http://goodcode.io/articles/ansible-tips/)

Currently works as advertised in the training doc. Merging. Primary RFE is to remove the dependency on the jenkins key.


NSNumber custom AnyHashable fix

What's in this pull request?
Resolves rdar://problem/33105735.
When we give NSNumber a custom AnyHashable representation, we want to give it as large a box as possible. When we want to compare it against other AnyHashable boxes such as Int or UInt, it's always possible to upcast the Int/UInt to a larger integer size like Int64 or UInt64 for the comparison. By eliminating the smaller boxes we create, we can maintain the existing behavior that _SwiftTypePreservingNSNumber gave us.

@swift-ci Please test macOS

@swift-ci Please smoke test Linux

cc @airspeedswift

Thanks for fixing!


Add Tallinn, Estonia

[x] Add submodule
[x] Update README.md
[x] Update cities.json
[x] Update package.json
[x] Create automated update
[x] Add MAPBOX_TOKEN secret
[x] Create automated deployment

Close #487

I believe I've added a MAPBOX_TOKEN..

Indeed! 👍

what implication does it have for me? will I have to pay to them, eventually?

The free tier should be enough, don't worry!
AFAIK not a single city reached the free limit.
But yes, in theory, you will have to pay Mapbox if you go over the free limit for Tallinn.

also, why are Global CI builds failing?

Still working on it. 😉


Showing white screen when implementing version 0.6.1.

OS:- Android
version:- 14.0
I think the issue is google map is not initializing.

Hi @hemantrajputcdn
Could you please provide more details about your local setup? Specifically:

React version
React Native version
Android OS version (you mentioned Android 14.0, but any specific device/emulator model?)
Share relevant Logcat logs from Android Studio, especially any errors or warnings that appear when initializing the map.
Confirm if Google Play Services are up-to-date on the device/emulator.
Let us know if you've tried cleaning and rebuilding the project.

This information will help us narrow down what might be causing the white screen. Thank you!

Than you for your response. I've resolved it.

Than you for your response. I've resolved it.
@hemantrajputcdn   May I ask how you solved it? I also encountered this problem.

I just followed the example folder structure and added all the necessary code from there.


Refactor 'ReplaceMap' in CSV handler

Note: the current code might be deleting the csvKey to AddonKey mappings of addons other than the one being reconciled at a given moment.
This patch ensures that only the mappings of the current Addon, which are obsolete are removed
Signed-off-by: Nikhil Thomas<EMAIL_ADDRESS>

/hold
@Ajpantuso I think there is the bug in the existing implementation.
https://github.com/openshift/addon-operator/blob/main/internal/controllers/addon/handler/csv_eventhandler.go#L69 👇
	// Remove all keys from our mapping that we don't need.
	for csvKey := range h.csvKeyToAddon {
		if _, ok := wantedCSVKeys[csvKey]; !ok {
			delete(h.csvKeyToAddon, csvKey)
			changed = true
		}
	}

wouldn't this remove csvKey to AddonKey mappings of all except the one which is being currently reconciled.
If yes, then this patch should fix it.
If no, then please ignore this patch. 🧑💻

/hold cancel

@nikhil-thomas This is a legit bug, i was able to reproduce this bug locally.
Can you please write an integration test for this scenario?

/hold
https://github.com/openshift/addon-operator/pull/282 makes this obsolete


fix _load_data method to handle invalid src argument

The current implementation of _load_data will fail if the src argument is not a valid file path nor URL and simply return the src string as it is.
This then causes the _apply_or_create method to fail at lines 273-275
result = [self._operate(kind, operation, api_version=api_version, body=item,
                                namespace=namespace or item.get('metadata', {}).get('namespace'),
                                **kwargs) for item in items]

With the following error:
ResourceOperationFailed: Apply failed
Reason: 'str' object has no attribute 'get'

The error message does not communicate what the problem was, so let's raise a more clear message in _load_data directly.

@pablofelix we've run a test using this branch and everything seems to work: https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/1694/console
Did you want me to add also a test case specifically for the edge case in the _load_data method?

yes, @lugi0 it helps to ensure that any modifications made to the code do not introduce new bugs or errors. It is worth including specific tests for edge cases. This can help to ensure that the library handles unexpected situations appropriately and avoids errors


Formula mismatch CHEBI:41981

CHEBI:41981 has formula "D2O" in ChEBI1.0, but in ChEBI2.0 it is being calculated as "H2O"
Note: both ChEMBL and PubChem also report it as H2O
Need to investigate if this is a bug or expected behavior.

isotopes are not taken into account for now (we don't do it in ChEMBL) but we may want to do it in ChEBI.
Hydrogen is a bit special since Deuterium and Tritium are sometimes treated by replacing the [2H] and [3H] by D and T.

this has been fixed


Sickbeard_mp4_Automator with Plex, uTorrent, Sonarr, and Radarr not working

I need some help with my SMA setup.  I apologize if this has already been covered elsewhere.
I'm running Plex <IP_ADDRESS>2, uTorrent (Ad-free) 3.5.5, Sonarr <IP_ADDRESS>38, and Radarr <IP_ADDRESS>8.  My downloads are not converting, nor moving to the correct directories.
My autoProcess.ini file:

[SickBeard]
host = localhost
port = 8081
username =
password =
web_root =
ssl = False
api_key =
[Sonarr]
host = localhost
port = 8989
web_root =
ssl = False
apikey = XXXXXXXXXXXXXXXXXXXXXXXXXXXX
[Radarr]
host = localhost
port = 7878
web_root =
ssl = False
apikey = XXXXXXXXXXXXXXXXXXXXXX
[MP4]
ffmpeg = C:\Program Files (x86)\ffmpeg\bin\ffmpeg.exe
ffprobe = C:\Program Files (x86)\ffmpeg\bin\ffprobe.exe
threads = 0
output_directory =
copy_to =
move_to =
output_extension = mp4
output_format = mp4
delete_original = True
relocate_moov = True
video-codec = h264,x264
video-bitrate =
video-crf =
video-max-width =
video-profile =
h264-max-level =
use-qsv-decoder-with-encoder = True
use-hevc-qsv-decoder = False
enable_dxva2_gpu_decode = False
ios-audio = False
ios-first-track-only = False
ios-audio-filter =
ios-move-last = False
max-audio-channels =
audio-codec = ac3
ignore-trudhd = true
audio-language = eng
audio-default-language =
audio-channel-bitrate =
audio-filter =
audio-copy-original = False
audio-first-track-of-language = False
subtitle-codec = mov_text
subtitle-language =
subtitle-default-language =
subtitle-encoding =
fullpathguess = True
convert-mp4 = True
tagfile = True
tag-language = en
download-artwork = Poster
download-subs = False
embed-subs = True
embed-only-internal-subs = False
sub-providers = addic7ed,podnapisi,thesubdb,opensubtitles
permissions = 0777
post-process = False
pix-fmt =
aac_adtstoasc = False
postopts =
preopts =
[CouchPotato]
host = localhost
port = 5050
username =
password =
web_root =
ssl = False
apikey =
delay = 65
method = renamer
delete_failed = False
[uTorrent]
convert = True
couchpotato-label = couchpotato
sickbeard-label = sickbeard
sonarr-label = sonarr
bypass-label = bypass
sickrage-label = sickrage
webui = False
action_before = stop
action_after = removedata
host = http://localhost:8080/
username = rnsimmons
password = XXXXXXXXXXXXXXXX
output_directory =
radarr-label = radarr
[Deluge]
host = localhost
username =
convert = True
password =
sonarr-label = sonarr
bypass-label = bypass
sickbeard-label = sickbeard
port = 58846
sickrage-label = sickrage
couchpotato-label = couchpotato
output_directory =
remove = false
radarr-label = radarr
[SABNZBD]
convert = True
sickrage-category = sickrage
sonarr-category = sonarr
radarr-category = radarr
bypass-category = bypass
couchpotato-category = couchpotato
sickbeard-category = sickbeard
output_directory =
[Sickrage]
host = localhost
port = 8081
username =
password =
web_root =
ssl = False
api_key =
[Plex]
host = localhost
port = 32400
refresh = True
token = XXXXXXXXXXXXXXXXXXXXXXXX

Does my configuration look correct?  If so, is there someplace I can go to see logs in order to figure this out?
I appreciate any help.

I was taking another quick look and was wondering of the uTorrent Run Program option needs to include the path to my uTorrentPostProcess.py?
I'm going to give it a try and will report any success.  (BTW, I'd still like to know the location of any logging files.)

@rnsimmons , did you have any success in getting this to work?  I'm running into the same problem on a Windows 7 machine.

No, I haven’t.  I even started uTorrent with logging and can’t tell whether it even tries to start the process.  I’ve even fully qualified the python executable and the Sick beard uTorrent script to no avail.  I opened a ticket with SMA on GitHub and haven’t heard back from the author.
Let me know if you make any progress.
Russell N. Simmons, Network+
Principal
940.293.5113 Cell |<PHONE_NUMBER> Fax
<EMAIL_ADDRESS><EMAIL_ADDRESS>From: SnarkyCanuck<EMAIL_ADDRESS>Sent: Sunday, October 20, 2019 4:22 PM
To: mdhiggins/sickbeard_mp4_automator<EMAIL_ADDRESS>Cc: Russell N. Simmons<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mdhiggins/sickbeard_mp4_automator] Sickbeard_mp4_Automator with Plex, uTorrent, Sonarr, and Radarr not working (#1063)
@rnsimmons https://github.com/rnsimmons  , did you have any success in getting this to work? I'm running into the same problem on a Windows 7 machine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/mdhiggins/sickbeard_mp4_automator/issues/1063?email_source=notifications&email_token=AGR2HMR7YZNE5NCLBSDMDQDQPTRZ7A5CNFSM4IZN4SIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBYWRXI#issuecomment-544303325 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGR2HMUT2ONA3JHSTOIJC7LQPTRZ7ANCNFSM4IZN4SIA .  https://github.com/notifications/beacon/AGR2HMVDXHPGEYH55GLGYCDQPTRZ7A5CNFSM4IZN4SIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBYWRXI.gif

Sorry I don't use uTorrent but a few things that might help you:

There should be a "index.log" file in the root directory of sickbeard_mp4_automator
Update your ffmpeg paths to escape the backslashes C:\Program Files (x86)\ffmpeg\bin\ffmpeg.exe
Does "Test" work on your Sonarr/Radarr setups?
Have you tried running manual.py?


fix(discord-command): remove prepending slash in names

Small change, removes the / from <discord-command> since context menu commands don't have this

Released as v2.8.1


Add @__ blade directive to "__" helper function.

Adding @__ blade directive makes it much easier to use the "__" helper function without the need for the boilerplate {{ }}
Taking the following lines from Laravel Breeze as an example
{{ __('Remember me') }}
{{ __('Forgot your password?') }}
<x-primary-button class="ml-3">{{ __('Log in') }}</x-primary-button>

Will be transformed to
@__('Remember me')
@__('Forgot your password?')
<x-primary-button class="ml-3">@__('Log in')</x-primary-button>

It also supports placeholders

@__('messages.welcome', ['name' => 'Mohamed'])

It will have more benefits with sites that have a lot of strings that need to be translated.
It makes the code look closer to the Laravel way :)
Credits to Mark Snape Tweet

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Key detection is case sensitive

I'm not sure if this is a K-9 bug or Openkeychain, but I have a contact who, for some weird reason, has a couple of capital letters on his pgp email id. So when I pick him as recipient K-9 fails to detect the correct key.

Email adresses are (partly) case sensitive, so we can't assume<EMAIL_ADDRESS>and<EMAIL_ADDRESS>are the same.
The easiest way to make it work is to make sure the email in the contacts database is the same as the one on the PGP key.

Email addresses are in practice case insensitive, I don't care what any rfc says...

Well, it's up to the crypto provider to decide whether or not to do case-insensitive matching. But I don't want to redo this in K-9 Mail. So the results from the crypto provider need to include the original email address from the query to allow simple equality matching on the K-9 Mail side.

So you are saying that is actually an deficiency on Openkeychain's side?


Moving to yaml format to allow issue templates with form elements

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


OnLayoutChange only fires once

I have a component where I am trying to dynamically change the height and widths of draggable components on the screen. I am setting this with a OnClick on a button that triggers a state change in the layout in a higher component, and then that layout is passed down to ResponsiveReactGrid via props. This issue is, this click event and onLayoutChange is only working on the first instance. All other instances are only enacted after the breakpoint is changed. I dug into the props and state of ResponsiveReactGrid, and it appears that while the "layouts" prop is changing, the "layout" state does not change. Anyone else run into this issue?

Duplicate of #670


If starting_points is not provided in the right format, fail loudly

Even a warning saying the starting_points is wrongly provided and not used in the output would be useful.
Originally suggested in #795

If the warmstart is used it'll be useful to see what parameters are used in the printed log. [Like in this output...]


Fetching subdirectories of LinkResource

Fixes puli/issues#129.
As @tgalopin suggested I'm creating PR for fetching linked resources only.

Thanks @msojda for working on this! :) Please don't forget to add tests that cover the different cases and edge cases of this functionality. Ideally, you should add these tests to the abstract test classes, so that the same tests are used for all repository implementations.

What's the status here?

This aimed to fix https://github.com/puli/issues/issues/129 but became a dupplicate of https://github.com/puli/repository/pull/64 when we chose to not implement add and removal in links children. I think this can be closed, but it depends on the amount of work you did there @msojda : did you work a lot on fetching resources? It could be interesting to see how you did it and integrate some ideas in https://github.com/puli/repository/pull/64.


lk.read() after lc.to_fits() does not preserve all columns

When executing the following snippet:
import lightkurve as lk
sr = lk.search_lightcurve('KIC 1435467', mission='Kepler', cadence='long')
lc = sr[2:5].download_all().stitch()
lc.to_fits('test_llc.fits', overwrite=True)
reread = lk.read('test_llc.fits')

I find that reread does not contain all the columns that appear in lc.
My expectation is that both light curves should have the same number of columns.

Does the problem happen for stitched lightcurve objects only? Or does it affect lightcurves downloaded straight from MAST? How about other generated lightcurves (e.g., fold, flatten, normalize, etc.)?

Those are excellent questions we need to investigate!  I haven't had the time to verify this yet.

I did some digging. It turned out for the basic LightCurve object's roundtrip to_fits() / read() . The columns (and meta keys) don't match.
Basically to_fits() only exports a subset of the columns. I think the codes remained in Lightkurve v1's behavior, of which LightCurve object doesn't represent the whole file.
The current codes failed the following test:
def assert_lc_equal(lc_actual, lc_expected):
    assert set(lc_actual.colnames) == set(lc_expected.colnames)
    # TODO: test meta values
    assert set(lc_actual.meta.keys()) == set(lc_expected.meta.keys())
    # TODO: test meta values

@pytest.mark.remote_data
def test_to_from_fits():
    """Test roundtrip .to_fits() / read, ensuring all are written out and all are  read."""
    filename = TESS_SIM
    lc = read(filename)

    hdu = lc.to_fits()
    # io.read method does not support reading from HDU object
    lc_read = lc.__class__.read(hdu)

    assert_lc_equal(lc_read, lc)

The difference in details:

lc
lc_read

cadenceno',
centroid_col',

centroid_col',
centroid_row',

centroid_row',
flux',

flux',
flux_err',

flux_err',
mom_centr1',

mom_centr1',
mom_centr2',

mom_centr1_err',
quality',

mom_centr2',
time'

mom_centr2_err',

pdcsap_flux',

pdcsap_flux_err',

pos_corr1',

pos_corr2',

psf_centr1',

psf_centr1_err',

psf_centr2',

psf_centr2_err',

quality',

sap_bkg',

sap_bkg_err',

sap_flux',

sap_flux_err',

time',

timecorr',

As it is, to_fits() is not meant to output all columns / headers. Notice the extra_data arguments that let callers specify what additional columns /headers to output, in addition to the predefined minimum set.
https://docs.lightkurve.org/api/lightkurve.lightcurve.LightCurve.html#lightkurve.lightcurve.LightCurve.to_fits

Oops, you're right, this shouldn't have been a surprise!   In Lightkurve v1, every LightCurve object had a pre-defined set of columns.  In Lightkurve v2, we should bring this function in line with the standard AstroPy Table i/o machinery.  I can look into this.

An additional tangential issue but probably touches similar sets of the codes.
The column names, as returned from lc.colnames are all lower-cased.
For meta, we've reverted the lowercased keys back to uppercase. So column names should probably remain the original uppercase as well.

I'm in favor of keeping the column names lower case because it benefits backward compatibility with Lightkurve v1 (where all column i.e. array names were lowercase) and the conversion to lowercase is the default behavior of the existing AstroPy TimeSeries reader functions, cf.:
https://github.com/astropy/astropy/blob/3627a7473e7c73f2e46e74157eaa4f078ccb8574/astropy/timeseries/io/kepler.py#L71-L72
Admittedly, I say these things because I really dislike uppercase names for anything other than constants.
The fact is that the Kepler/Tess FITS files do user uppercase column names, whether I like it or not, so if we convert to lowercase in the reader then we must also add a Kepler/Tess light curve format writer in lightkurve.io which converts the column names back to uppercase.  Fortunately, I think we need such writers anyway to produce files that are exactly consistent with the FITS layout of the Kepler/Tess file format.

Regarding column's case:

when I asked about preserving the case in FITs, I am thinking about doing it in a way similar to meta such that it is natural for typical python and Lightkurve v1 compatible, i.e., lowercased in attribute access (lc.flux, same as v1), but upper (original) cased in column access (lc['FLUX'], new in v2)
However, given Astropy's default for Kepler/TESS format is to lowercase the column names (but not header names). So I suppose we'd better stick with it, as it probably gives the users least amount of surprises.
As an API user, I am surprised that the exception has been made, as the rest of Table, QTable, or base TimeSeries does no conversion.

Fortunately, I think we need such writers anyway to produce files that are exactly consistent with the FITS layout of the Kepler/Tess file format.

That'd be nice as it'd improve interoperability. Given Astropy's default Kepler/TESS format handler supports read but not write, I suppose it's not exactly trivial...


build: Update Vulkan-Headers and tools to 1.3.232

CI Vulkan-ExtensionLayer build queued with queue ID 22796.

CI Vulkan-ExtensionLayer build # 207 running.

CI Vulkan-ExtensionLayer build # 207 failed.

CI Vulkan-ExtensionLayer build queued with queue ID 22806.

CI Vulkan-ExtensionLayer build # 208 running.

CI Vulkan-ExtensionLayer build # 208 failed.

CI Vulkan-ExtensionLayer build queued with queue ID 22818.

CI Vulkan-ExtensionLayer build # 209 running.

CI Vulkan-ExtensionLayer build # 209 failed.

CI Vulkan-ExtensionLayer build queued with queue ID 22840.

CI Vulkan-ExtensionLayer build # 210 running.

CI Vulkan-ExtensionLayer build # 210 failed.

CI Vulkan-ExtensionLayer build queued with queue ID 22867.

CI Vulkan-ExtensionLayer build # 213 running.

CI Vulkan-ExtensionLayer build # 213 failed.

CI Vulkan-ExtensionLayer build queued with queue ID 23196.

CI Vulkan-ExtensionLayer build # 218 running.

CI Vulkan-ExtensionLayer build # 218 passed.


Add doc about classpathResources + multiple versions

Fixes: https://github.com/openrewrite/rewrite-docs/issues/124

Nicely done!


Sidebar won't expand after update

m1 max

to fix that you can use the soft reset button in the config!

I'm not sure where the soft reset button is that you're talking about, I've tried rebooting and refreshing and it doesn't do anything

click on the gear at the bottom

Thanks, I'll try again and see

<img width="227" alt="Screenshot 2024-05-25 21 09 10" src="https://github.com/Nuked88/ComfyUI-N-Sidebar/assets/121092793/147920ea-b080-4ba4- b133-8ddb2655660f">
I've tried over and over, and I think there's a conflict with this node.

I've tried over and over, and I think there's a conflict with this node.

i don't know, i don't have that node
Try to disable it and let me know!

After disabling the node, the settings box popped up, tried a soft reboot, and it still doesn't show the node properly because you said a hard reboot would lose the node, so I didn't try it.

Press Ctrl + Shift + J to open the Developer Tools.
Then, go to the "Application" tab.
Under "Storage", expand "Local Storage" and select "http://<IP_ADDRESS>:8188/" (or the appropriate address if different).
Now, on the right side, you should see a list of variables with their respective values.
What's the value of sidebarWidth?

If you want, i've just pushed a new update, you can try use the soft reset again after the update to see if this time is working

Oh, great, it's working. Easy use, the missing node, is in conflict with this one, and I've deactivated it.

extremely grateful


Change Traffic Ops /update to priv > 10 to facilitate minimal syncds priveleges

This specifically allows creating a role with privilege level between
10 and 20 (e.g. 11), for ORT/syncds, which can only access GET routes
plus POST /update in order to minimize access privileges.

do you want to create a role for this? maybe call it 'ort' with a priv level of 11? this role would be added to the seeds.sql file. i'm guessing everyone could benefit from this new role.

@mitchell852 Done


Email+ - Date formatting helpers

Emails generated with EmailPlus need some control over how dates are rendered. A helper like:
{{local_date SomeDate "yyyy"}}

or:
{{utc_date SomeDate "o"}}

would help to produce more readable date output.

wouldnt it be simple to have a extra props for TimestampUtc and LocalTimestamp that are unambiguous ? that would suffice for our requirements

That's already done, see: https://github.com/datalust/seq-apps/blob/master/src/Seq.App.EmailPlus/EmailReactor.cs#L151
(I can see how the description above is confusing, hopefully your comment and my reply now clarify it :-))

@nblumhardt but that uses the default tostring. which is not unambiguous

Ah, I see what you mean - {{format_date $UtcTimestamp "o"}} etc. ... The challenge is handling dates that are included in other event properties, for example an event might have a CreatedDate property which, irrespective of whether it's expressed as a local or UTC timestamp, the user might prefer to show as one or the other. Splitting formatting from time zone conversions would be another reasonable approach, though - I'm not sure how much support Handlebars.NET has for nested function application, but in principle that does seem cleaner 👍

#74 adds support for this 🎉


[BBC iPlayer] '%(title)s' does not include episode numbers

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2022.02.04. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[x] I've read the guidelines for opening an issue

Description

Link Used: https://www.bbc.co.uk/iplayer/episode/m000kclt/boy-girl-dog-cat-mouse-cheese-series-1-37-putt-putt-pals

The issue I'm having is that the episode number of a series is not included in the %(title)s output. Series/Episodes that dont have a the number in the title (it only has the season number or year) output fine, I noticed this while trying to download a lot of episodes from a series and each one lacked the episode number. It saved as Series Name, Season Number, Episode Name
The webpage title for the specific url is "Boy Girl Dog Cat Mouse Cheese - Series 1: 37. Putt Putt Pals"
Command Used
I also include --proxy socks5://<IP_ADDRESS>:1080/ since I download from a separate server but I ran the below command on the UK machine and got the same output.
yt-dlp -vU \
  --user-agent "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.77 Safari/537.36" \
  --output "%(title)s (%(duration)ss) [%(resolution)s] [%(id)s].%(ext)s" \
  https://www.bbc.co.uk/iplayer/episode/m000kclt/boy-girl-dog-cat-mouse-cheese-series-1-37-putt-putt-pals

What is saved:
Boy Girl Dog Cat Mouse Cheese, Series 1, Putt Putt Pals (660s) [1280x720] [m000kcls].mp4
Expected:
Boy Girl Dog Cat Mouse Cheese, Series 1 37, Putt Putt Pals (660s) [1280x720] [m000kcls].mp4
Boy Girl Dog Cat Mouse Cheese, Series 1 Episode 37, Putt Putt Pals (660s) [1280x720] [m000kcls].mp4
(Or any variation that includes the episode number in the output 'title' argument)

Note:
Downloading from BBC requires a UK connection, I currently use a non-residential IP and it works fine. If you are unable to get a connection to test download I can provide access to the server I use for downloading.
Verbose log
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, err utf-8, pref UTF-8
[debug] yt-dlp version 2022.02.04 [c1653e9ef] (zip)
[debug] Python version 3.8.10 (CPython 64bit) - Linux-5.4.0-29-generic-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4
[debug] Optional libraries: secretstorage, sqlite
[debug] Proxy map: {}
Latest version: 2022.02.04, Current version: 2022.02.04
yt-dlp is up to date (2022.02.04)
[debug] [bbc.co.uk] Extracting URL: https://www.bbc.co.uk/iplayer/episode/m000kclt/boy-girl-dog-cat-mouse-cheese-series-1-37-putt-putt-pals
[bbc.co.uk] m000kclt: Downloading video page
[bbc.co.uk] m000kclt: Downloading playlist JSON
[bbc.co.uk] m000kcls: Downloading media selection JSON
[bbc.co.uk] m000kcls: Downloading m3u8 information
[bbc.co.uk] m000kcls: Downloading m3u8 information
[bbc.co.uk] m000kcls: Downloading m3u8 information
WARNING: [bbc.co.uk] Failed to download m3u8 information: HTTP Error 403: Forbidden
[bbc.co.uk] m000kcls: Downloading m3u8 information
WARNING: [bbc.co.uk] Failed to download m3u8 information: HTTP Error 403: Forbidden
[bbc.co.uk] m000kcls: Downloading m3u8 information
[bbc.co.uk] m000kcls: Downloading m3u8 information
[bbc.co.uk] m000kcls: Downloading MPD manifest
[bbc.co.uk] m000kcls: Downloading MPD manifest
[bbc.co.uk] m000kcls: Downloading MPD manifest
WARNING: [bbc.co.uk] Failed to download MPD manifest: HTTP Error 403: Forbidden
[bbc.co.uk] m000kcls: Downloading MPD manifest
WARNING: [bbc.co.uk] Failed to download MPD manifest: HTTP Error 403: Forbidden
[bbc.co.uk] m000kcls: Downloading MPD manifest
[bbc.co.uk] m000kcls: Downloading MPD manifest
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] m000kcls: Downloading 1 format(s): mf_bidi-5510-1
[debug] Invoking downloader on "https://b4-thdow-bbc.live.bidi.net.uk/vod-hls-uk/usp/auth/vod/piff_abr_full_hd/68b46f-m000kcls/vf_m000kcls_330dea18-56f6-4d44-97e5-8224455c736b.ism.hlsv2.ism/vf_m000kcls_330dea18-56f6-4d44-97e5-8224455c736b.ism.hlsv2-audio_eng_1=128000-video=5070000.m3u8"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 87
[download] Destination: Boy Girl Dog Cat Mouse Cheese, Series 1, Putt Putt Pals (660s) [1280x720] [m000kcls].mp4
[download] 100% of 321.41MiB in 00:11
[debug] ffprobe command line: ffprobe -hide_banner -show_format -show_streams -print_format json 'file:Boy Girl Dog Cat Mouse Cheese, Series 1, Putt Putt Pals (660s) [1280x720] [m000kcls].mp4'
[FixupM3u8] Fixing MPEG-TS in MP4 container of "Boy Girl Dog Cat Mouse Cheese, Series 1, Putt Putt Pals (660s) [1280x720] [m000kcls].mp4"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i 'file:Boy Girl Dog Cat Mouse Cheese, Series 1, Putt Putt Pals (660s) [1280x720] [m000kcls].mp4' -map 0 -dn -ignore_unknown -c copy -f mp4 -bsf:a aac_adtstoasc -movflags +faststart 'file:Boy Girl Dog Cat Mouse Cheese, Series 1, Putt Putt Pals (660s) [1280x720] [m000kcls].temp.mp4'

Work-around (Windows quotes, '' for Unix-like systems):
--parse-metadata "webpage_url_basename:boy-girl-dog-cat-mouse-cheese-series-(?P<season_number>\d+)-(?P<episode_number>\d+)(?:-[\w]+)*" --output "%(title)s S%(season_number).02dE%(episode_number).02d (%(duration)ss) [%(resolution)s] [%(id)s].%(ext)s"

Eg:
$ yt-dlp --parse-metadata "webpage_url_basename:boy-girl-dog-cat-mouse-cheese-series-(?P<season_number>\d+)-(?P<episode_number>\d+)(?:-[\w]+)*" --output "%(title)s S%(season_number).02dE%(episode_number).02d (%(duration)ss) [%(resolution)s] [%(id)s].%(ext)s" --get-filename 'https://www.bbc.co.uk/iplayer/episode/m000kclt/boy-girl-dog-cat-mouse-cheese-series-1-37-putt-putt-pals'
Boy Girl Dog Cat Mouse Cheese, Series 1, Putt Putt Pals S01E37 (660s) [1280x720] [m000kcls].mp4
$

I ran into a similar issue when downloading a playlist, where I needed the episode number and a clean episode title, as they're not available in the metadata. Here's my workaround, in case anyone needs it:
yt-dlp --parse-metadata 'title:(?<=Series )(?P<series_number>\d+)' --parse-metadata 'title:.*Series \d+, (?P<episode_title>.*)' --output "/path/to/%(extractor)s/%(playlist_title)s/S%(series_number)02dE%(playlist_index)02d-%(playlist_count)02d-%(episode_title)s.%(ext)s" --yes-playlist "https://www.bbc.co.uk/iplayer/episodes/m000b1v2/his-dark-materials?seriesId=p0dgc2vk"

This results in the path:
/path/to/bbc.co.uk/His Dark Materials/S03E01-08-The Enchanted Sleeper.mp4

The solution suggested above does not work for playlists that don't have the Episode number in the title. For example, Shetland Series 2, https://www.bbc.co.uk/iplayer/episodes/p01s711r/shetland?seriesId=p01s91yf.
The following works for both scenarios.
I download all episodes of the series to one directory, embed the webpage_url_basename value (minus the first character) into the filenames, and then process the files after the downloads have completed, renaming each file with the episode number based on its alphabetic position in the ordered file list.
yt-dlp --parse-metadata webpage_url_basename:.?(?P<basename_part>.*)$ --parse-metadata title:.*, Series
\d+, Episode (?P<episode_num>\d+)$ --parse-metadata title:.*, Series (?P<series_number>\d+), (?P<episode_title>.*) --output /path/to/%(extract
or)s/%(playlist_title)s/Series_%(series_number)02d/%(playlist_title)s_%(playlist_count)02d_S%(series_number)02dE%(episode_num)02d_%(basename_part)s_%(episode_title)s.%(ext)
s --yes-playlist -S ext,width:720 https://www.bbc.co.uk/iplayer/episodes/p01s711r/shetland?seriesId=p01s91yf

Then run this in the directory:
#!/bin/bash

for ext in $(find . -maxdepth 1 -type f -name '*.*' -exec basename {} \; | awk -F. '{print $NF}' | sort -u); do
  counter=1

  for file in *."$ext"; do
    new_number=$(printf "%02d" $counter)

    new_filename=$(printf "%s" "$file" | sed -E "s/(.+_[0-9]+_S[0-9]+E)NA/\1${new_number}/")

    if [[ "$file" == "$new_filename" ]]; then
      continue
    fi

    mv "$file" "$new_filename"

    counter=$((counter + 1))
  done
done

The resulting files are as follows:
'Shetland_06_S02E01_01sf9kq_Raven Black - Part 1.en.ttml'
'Shetland_06_S02E01_01sf9kq_Raven Black - Part 1.mp4'
'Shetland_06_S02E02_03ynt67_Raven Black - Part 2.en.ttml'
'Shetland_06_S02E02_03ynt67_Raven Black - Part 2.mp4'
'Shetland_06_S02E03_03z94wx_Dead Water - Part 1.en.ttml'
'Shetland_06_S02E03_03z94wx_Dead Water - Part 1.mp4'
'Shetland_06_S02E04_03zqbfn_Dead Water - Part 2.en.ttml'
'Shetland_06_S02E04_03zqbfn_Dead Water - Part 2.mp4'
'Shetland_06_S02E05_040hzvm_Blue Lightning - Part 1.en.ttml'
'Shetland_06_S02E05_040hzvm_Blue Lightning - Part 1.mp4'
'Shetland_06_S02E06_04154p9_Blue Lightning - Part 2.en.ttml'
'Shetland_06_S02E06_04154p9_Blue Lightning - Part 2.mp4'

I have a wrapper script that runs this automatically after yt-dlp, when --yes-playlist is used.

The solution suggested above does not work for playlists that don't have the Episode number in the title.

It's the webpage_url_basename that had to have a suitable format (possibly related to the wepbage <title>.
Otherwise, review the JSON for a typical episode to see if there's another metadata field containing season/series/episode number that could be used.

I had already edited my post by the time you added this. I changed it to display_id.

The point was that if an extractor fails to populate season/series/episode fields but they can be found embedded in some other fields, the solution using --parse-metadata ... can generally be adapted, without any separate scripting.

@dirkf But that's exactly the issue - there's no episode number anywhere in the metadata. If I'm wrong, please let me know where it is in the example, https://www.bbc.co.uk/iplayer/episodes/p01s711r/shetland?seriesId=p01s91yf, which is "Shetland", series 2.

This series has episode URLs like https://www.bbc.co.uk/iplayer/episode/p01sf9kq/shetland-series-2-1-raven-black-part-1. That should set webpage_url_basename to shetland-series-2-1-raven-black-part-1 from which the [\w-]+?-series-(?P<season_number>\d+)-(?P<episode_number>\d+)(?:-[\w]+)* can be parsed as before.
As to why this isn't being extracted by default, the answer is that this part of the module hasn't been updated recently. Obviously the season/series/episode data is available in the trailing part of the URL path as above: that might be considered a lucky hit, although the iPlayer URLs seem to be stable compared with other streaming sites.
The extractor gets the title from the playlist JSON:
{
  ...,
  'defaultAvailableVersion': {
    'pid': 'b03xddfz',
    ...,
    'smpConfig': {
      'title': 'Shetland, Series 2, Raven Black - Part 1',
      ...,

but in the episode's webpage there is hydration JSON:
window.__IPLAYER_REDUX_STATE__ = {
  ...,
  'episode': {
    ...,
    'currentVersion': 'standard',
    'id': 'p01sf9kq',
    'live': false,
    'title': 'Shetland',
    'subtitle': 'Series 2: 1. Raven Black - Part 1',
    ...,
  }

This looks like some sort of inconsistency at the Beeb. Anyway, we should be parsing this JSON before punting to the playlist JSON, saving a request, especially because it also has this data:
  ...,
  'versions': [
    {
      'id': 'b03xddfz',
      'kind': 'original',
      'hd': true,
      ...,
      'firstBroadcast': '9pm 11 Mar 2014',
      'duration': {
        'text': '59 mins',
        'seconds': 3547
      },
      ...,
      },
      'creditsStart': 3519,
      'startTime': null,
      'endTime':<PHONE_NUMBER>,
      'isoStartTime': '2023-03-01T06:00:00Z',
      'isoEndTime': '2025-03-01T06:00:00Z'
    },
    {
      'id': 'p01tpn4j',
      'kind': 'audio-described',
      'hd': false,
      ...,
      'firstBroadcast': '11 Mar 2014',
      'duration': {
        'text': '59 mins',
        'seconds': 3547
      },
      ...,
      'startTime': null,
      'endTime':<PHONE_NUMBER>,
      'isoStartTime': '2023-03-01T06:00:00Z',
      'isoEndTime': '2025-03-01T06:00:00Z'
    }
  ],
  ...,

The series page extraction uses a similar subtitle field to populate the series playlist title, so it would also be possible to get the season_number provided that --parse-metadata ... works at the playlist level: I haven't investigated.

@dirkf Thanks for looking at this. Yeah, it's definitely the Beeb's error - the iplayer seems to be really inconsistent in the backend.
My suggestion wasn't intended as a solution for yt-dlp, but rather just a workaround. From my testing, my yt-dlp command and bash script reliably download playlists and embeds the episode numbers in the file names, regardless of the series url. I'm more than happy with this until yt-dlp is fixed.


Function argument destructuring doesn't work

https://jsmonk.github.io/hegel/try#GYVwdgxgLglg9mABHA1gCgIYCcDmBnASkQG8AoASG3wDoNSBfU00SWBRYDGAGzzWIz0iZSgyA

Yes. Currently, we don't implement a desctruction syntax, but it will be fixed soon.
Thank you for your issue ^_^.


Convert to comprehensive pulumi/devcontainer git submodule

Description
PR to import the github.com/pulumi/devcontainer repository as a git submodule.
This container includes:

hugo
pulumi cli
pulumictl cli
pulumi esc cli
golang
python
dotnet
nodejs
yarn
and a few other nice to haves

Testing
To try it yourself:

use click menu to open PR in github codespaces
execute make command in vscode terminal
open site built 'locally' in a new tab when prompted

Motivation
At this time, basic activities, including writing IaC, submitting blog posts and tutorials, and developing new providers includes many prerequisite steps
Checklist:

[ ] I have reviewed the style guide.
[ ] If blogging, I have reviewed the blogging guide.
[ ] I have manually confirmed that all new links work.
[ ] I added aliases (i.e., redirects) for all filename changes.
[ ] If making css changes, I rebuilt the bundle.

For the testing steps, this is what I did:
go to https://github.com/pulumi/pulumi-hugo/tree/feat/usrbinkat/devcontainer -> "<>" -> "Codespaces +" -> Click the "+" which reads "create a codespace on feat/usrbinkat/devcontainer"
// wait about 10 minutes ...
execute make command in vscode terminal
click http://localhost:1313/  which the forwards it to a working endpoint (https://silver-space-barnacle-4g995495r52q4vw-1313.app.github.dev/) in my case
LGTM!

I think there's just some minor lint things to address so that all the checks pass.

When I try to activate this locally (using the VS Code extension), it fails on this:


Improve screen component, realm & passwordAuth screen.

screen: Add a optional prop keyboardShouldPersistTaps for scrollView. f9d9daa
realmScreen: Fix two taps are required to press Enter button.  8fcf12c
passwordAuthScreen: Fix two taps are required to press Log in button.  ab4f10d
passwordAuthScreen: Clean up code. a91fcc7

Remove extra view wrapper.
Remove unwanted mapStateToProps props.

Looks great, merged -- thanks again @jainkuniya !
I went and revised the first commit message along the lines of my comment about it above. The idea is to summarize the information that matters to us, and how the behavior of that upstream prop as described in the upstream doc relates to things we want to do.

Thanks @gnprice for the clean up :)
Closing this as all commits already landed to the master.

Ah indeed -- thanks for catching that. :)