Support different JavaScript ABI for TypeScript interop

TypeScript’s JavaScript code must in some aspects be treated differently than Ceylon’s JavaScript code, which means that the JS backend will need to support another calling convention, type treatment, etc., which might be summarized as an “ABI” (not actually binary, but unlike an API the concern of compilers).

Classes must be instantiated with new. Currently, this is only done for untyped invocations in dynamic blocks where the name starts with an uppercase letter; Ceylon types are instantiated by just calling Name().

Classes must be extended by calling the superclass constructor with this set to the new object.
SuperClass.call(this, arg1, arg2);

Ceylon class constructors expect the object as the last parameter, and pass it on to the super constructor as last argument (after the other parameters/arguments).

Value access must always be value access, no getter or setter methods. (This is already done to some extent, since ES≥5 supports accessors, but not always. I’m not sure what the rules currently are, but for TypeScript APIs, a value is a value is a value.)

Most types are not reified. This is discussed in #5982.

…

I think it should be sufficient to specify the ABI at the module level, perhaps just as another possible value for $mod-bin (currently "9.0").

I think it should be sufficient to specify the ABI at the module level, perhaps just as another possible value for $mod-bin (currently "9.0").

Currently, the binary version is always int dot int (on both backends), so that’s not really possible right now. @chochos what if we expand the format to: (kind-)?major.minor? E. g. typescript-0.1 or ceylon-9.0 or 9.0 (the last two would be equivalent). That would give us some flexibility for future development (multiple TypeScript versions, perhaps other model loaders too).

@lucaswerkmeister what's the reason exactly for the Ceylon module itself to be marked differently? Isn't it "just" code calling TS that gets generated differently, not Ceylon APIs?

Well I’m exporting the TS code to a “Ceylon” module (-model.js file), so it will mostly look like a regular module to users and the rest of the compiler. But we need to record its source in the module.

Rather than a different binary version, why not either

Add a new boolean attribute $mod-is-typescript (or similar, you get the idea), or
Add several new prescriptive attributes for things like "always use new for this class"

Indeed, I think it would be very wise to try and find some other solution because introducing several module types each with their own binary version numbers is going to make things everywhere much more complex.

responding to @jvasileff’s questions:

That would be possible, I guess, but it raises two new questions:

What’s the meaning of $mod-bin in this case? There’s no Ceylon code in the module, and a break in JS compatibility doesn’t mean that compatibility for TypeScript-loaded modules would have to break as well, so I don’t think it would make much sense to copy over the Ceylon version (9.0).

What happens if TypeScript interop needs multiple versions, for some reason? TypeScript is presumably going to continue development alongside ECMAScript, so it’s possible that new features will be added that need to be reflected in Ceylon.

And putting these two questions together, the simple answer to me is: use $mod-bin to encode the version of the TypeScript ABI. But "$mod-bin": "1.0", "$mod-is-typescript": true is obviously the same thing as "$mod-bin": "typescript-1.0", just encoded differently.

That would just bloat the model in my opinion (both the model file and the typechecker module). Unnecessary, because I don’t think we’ll ever mix TypeScript and Ceylon members in one module.

I would see $mod-bin being at the highest level of abstraction. A particular release of compile-js supports a single $mod-bin, which also happens to define the model.js format. For each $mod-bin version, one or more TypeScript versions may be supported.
New $mod-bin versions will of course require new versions of the TypeScript model.js files even if there are no TypeScript specific changes. (I'm sure it's unreasonable to expect the JS compiler to support what would effectively be multiple historic versions of model.js formats.)
The arguments for 2 would be flexibility and control. Flexibility since the format would then not be TypeScript specific. Control since it would shift the power to tweak code generation rules to the model.js creator rather than teaching the JS compiler about TypeScript specifically (and, various TypeScript versions.) I'm not necessarily arguing for 2 over 1, but rather exploring ideas.

Ad 1.ii.) I'd say that it would be done the same way we do it now for the different JVM versions: you do your best to support all versions up to a point that it becomes impossible because you're depending on features that don't exist in earlier versions and then you just say that you don't support those versions anymore.

Hello All,
Glad to see the interop work going so well. Just wanted to make a few comments. Given there is talk of requiring the Typescript compiler in #3041 and that (at least to my understanding) the Ceylon JS compiler can't switch between ES3, ES5, ES6. I was thinking maybe the best bet was to embrace the TSC completely. Have Ceylon be able convert Typescript definitions as long as node and the TSC are installed and also generate Typescript code. Replacing the Ceylon-JS with one that simply generates an tree to feed directly into the TSC
This way the generated code still contains a lot of type info and the TSC can generate code for specific JS versions. This would also mean all exiting Node tooling for Typescript (like webpack and browserify modules) would work for Ceylon as well.
It might be to much work, but seems like in the long run lets you guys not have to maintain a lot of things as that would be the responsibility of the Typescript team and the surrounding Node community. It also would remove the need to worry about what JS features are currently supported in which browser/Node version.

@corbinu but does Typescript support all the features that Ceylon offers, such as reified generics for example?

I don't know what's the current status of web assembly but if that thing's taking off, then if we have to rewrite the js compiler I'd much rather aim for web assembly than typescript...

@chochos Well it is a super set of Javascript so you can generate Javascript for that feature you can generate it in typescript, just with more type info.
Web Assembly is designed as a C++ build target. As far as I know nobody is suggesting a JS -> Web Assembly compiler for example as it lacks Objects and Arrays and wouldn't really improve execution time Source
Even if there was a benefit there then I would say long term you could probably assume that the TSC would add Web Assembly support

@chochos @lucaswerkmeister Here's a suggestion: how about we just go ahead and conform to the TypeScript mappings to JavaScript. I bet they have put more thought into what is the smoothest mapping to idiomatic JavaScript, and so if we just follow them everywhere, I bet we will make interop easier for ourselves.

I was thinking maybe the best bet was to embrace the TSC completely.

@corbinu what is "TSC" here?

TypeScript Compiler

@gavinking TSC is typescript compiler.

OK, I understand. Not a totally crazy idea, I suppose. But it would an extra layer of transpilation, and an extra moving part in the whole infrastructure, and so there would have to be some pretty convincing benefit...

Here let me open another issue as I feel like I have derailed this one and we all can debate pros/cons there

I’ve updated the issue and added more points.
I think it would make a lot of sense to implement most of these points using the same flag – let’s call it pojs for now, for Plain Old JavaScript (native is already taken).

POJS classes are instantiated with new. (This means the $new flag will be removed.)
POJS classes are extended differently.
POJS does not use getter / setter methods.
POJS uses a different Array type, which must be wrapped / unwrapped.
POJS does not reify types – values must be wrapped in dre$$ calls.
POJS does not escape names. (Reserved words are simply not permitted at all.)

This covers most of the points in the issue, and as I see it, these points all aren’t really specific to TypeScript (reflecting the fact that TypeScript mainly codifies existing JavaScript patterns) – we should also use these rules, for instance, with dynamic interfaces and in dynamic blocks (which already happens to some extent).
@chochos do you think this makes sense? And should the existing dyn flag be expanded to embrace these rules as well?

Is this issue meant for 1.2.3 or should it be moved to 1.3?

I haven’t set any milestone on my TypeScript issues, since I don’t know which one would be appropriate. Probably not 1.2.3, at least. Do they block the 1.3 release?

No no, but we try to give tentative release milestones just to make sure we don't forget about them :) Moving to 1.3 then.


await?

Hi, cool library 👍
I was reading through https://github.com/malcommac/Hydra/blob/master/Sources/Hydra/Await.swift#L109
// Create a semaphore to block the execution of the flow until
		// the promise is fulfilled or rejected
		let semaphore = DispatchSemaphore(value: 0)

so we're turning async code -> sync code by blocking the current thread?
Also, it would be cooler if there's an example project 😉

Hi, thank you. :)
Await is a way to block the execution of current caller thread, resolve passed promise in another queue then resume the execution. In fact you are telling "await the fulfill of a promise". Promise will be resolved in another thread but the queue called by await is blocked.
This allows you to express a sync code in a sync manner.
Howewer if you don't need it you can still use Promise in classic way.
I'll also add async shortcut soon.

An example of await can be found in README. I'll make a project with examples later during development; meanwhile you can take a look at unit tests in order to see how it works.

@malcommac Hi, got it. Thanks 👍

This should be mentioned in the README that await simply blocks the current thread. I first thought await magically worked similar to the C# await keyword and couldn't believe that this was even possible in Swift until I looked into the code to discover that it just blocks.

@tomquist yeah, I had the same thought too

Yeah I think you are right; I've missed to explicitly add it to the documentation.
Here are the commits which fixes it https://github.com/malcommac/Hydra/commit/01ee6c576bfba7279cbba72252cbbcab38743d13 and https://github.com/malcommac/Hydra/commit/09a3afa73712f71bbd68cdc744baac3ee33cfe6d.
I've also added async operator which can works in two ways: to create a Promise or just execute a block of code.
I'm also looking a way to make it easier to write; if you have some suggestions I'll be more than happy to implement them.


No External URL

What if we only want to onboard an onprem app onto AAD so we could manage permissions but don't want to publish an external url like in this document. Like we only want access from people within the network. Essentially using AAD to manage access to internal apps without exposing it to people outside the network.

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

ID: bdbd2045-3f96-0c47-00ed-b514a759a6bd
Version Independent ID: 90e715cd-08c2-09e9-292f-bfc515f0dee3
Content: Add an on-premises app - Application Proxy in Azure Active Directory
Content Source: articles/active-directory/manage-apps/application-proxy-add-on-premises-application.md
Service: active-directory
GitHub Login: @barbkess
Microsoft Alias: barbkess

@Infinitism  Thanks for your feedback! We will investigate and update as appropriate.

@Infinitism
Apologies for the delay.
Publishing a External URL is not a mandatory field while adding the On-Premise Application to Azure AD (Check Step 4 under the section - Add an on-premises app to Azure AD)
Also, you can look into Location Based Conditional Access Policy to block access from outside your network.
We will be closing this thread now. Please re-open/let us know if you need further help.


TensorFLow v1.13.0rc0 tflite_convert fails to convert TensorFlow model with BatchNormalization layers

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Fedora 26
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.13.0rc0
Python version: 3.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10.0
GPU model and memory: Nvidia Tesla P100

You can collect some of this information using our environment capture script
You can also obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the current behavior
tflite_convert fails to convert simple conv2d model with batch norm file to .tflite. The trained model was frozen to a .pb file using freeze_graph. It fails with the following errors:
tflite_convert --graph_def_file=frozen_saved_model.pb --output_file=frozen_saved_model.tflite --input_shapes="1,28,28,1" --input_arrays=conv2d_input --output_arrays=dense_1/Softmax
Traceback (most recent call last):
File "/virtualenv/tf-1.13.0-gpu/lib/python3.6/site-packages/tensorflow/python/framework/importer.py", line 426, in import_graph_def
graph._c_graph, serialized, options)  # pylint: disable=protected-access
tensorflow.python.framework.errors_impl.InvalidArgumentError: Input 0 of node batch_normalization_v1/cond/ReadVariableOp/Switch was passed float from batch_normalization_resource.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/virtualenv/tf-1.13.0-gpu/bin/tflite_convert", line 10, in
sys.exit(main())
File "/virtualenv/tf-1.13.0-gpu/lib/python3.6/site-packages/tensorflow/lite/python/tflite_convert.py", line 442, in main
app.run(main=run_main, argv=sys.argv[:1])
File "/virtualenv/tf-1.13.0-gpu/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 125, in run
_sys.exit(main(argv))
File "/virtualenv/tf-1.13.0-gpu/lib/python3.6/site-packages/tensorflow/lite/python/tflite_convert.py", line 438, in run_main
_convert_model(tflite_flags)
File "/virtualenv/tf-1.13.0-gpu/lib/python3.6/site-packages/tensorflow/lite/python/tflite_convert.py", line 122, in _convert_model
converter = _get_toco_converter(flags)
File "/virtualenv/tf-1.13.0-gpu/lib/python3.6/site-packages/tensorflow/lite/python/tflite_convert.py", line 109, in _get_toco_converter
return converter_fn(**converter_kwargs)
File "/virtualenv/tf-1.13.0-gpu/lib/python3.6/site-packages/tensorflow/lite/python/lite.py", line 274, in from_frozen_graph
_import_graph_def(graph_def, name="")
File "/virtualenv/tf-1.13.0-gpu/lib/python3.6/site-packages/tensorflow/python/util/deprecation.py", line 501, in new_func
return func(*args, **kwargs)
File "/virtualenv/tf-1.13.0-gpu/lib/python3.6/site-packages/tensorflow/python/framework/importer.py", line 430, in import_graph_def
raise ValueError(str(e))
ValueError: Input 0 of node batch_normalization_v1/cond/ReadVariableOp/Switch was passed float from batch_normalization_v1/gamma:0 incompatible with expected resource.
Describe the expected behavior
A TensorFlow Lite version of the frozen model file should have been generated by the tool.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
import tensorflow as tf
import tensorflow.keras.optimizers as optimizers
import tensorflow.keras.losses as losses
from tensorflow.keras.datasets import mnist
(train_x, train_y), _ = mnist.load_data()
model = Sequential()
model.add(Conv2D(32, kernel_size=(3, 3),
activation='relu',
input_shape=(28,28,1),
kernel_regularizer=regularizers.l1(reg_weight)))
model.add(Conv2D(64, (3, 3), activation='relu', kernel_regularizer=regularizers.l1(reg_weight)))
model.add(BatchNormalization())
model.add(MaxPooling2D(pool_size=(2, 2)))
model.add(Flatten())
model.add(Dense(128, activation='relu', kernel_regularizer=regularizers.l1(reg_weight)))
model.add(Dense(10, activation='softmax', kernel_regularizer=regularizers.l1(reg_weight)))
model.compile(optimizer=optimizers.Adam(), loss='categorical_crossentropy', metrics=['accuracy'])
model.fit(train_x, train_y, batch_size=64, epochs=2, verbose=1)
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Any update or workarounds for this?

The workaround I'm using is to replace tf.keras.layers.BatchNormalization layers with tf.layers.batch_normalization(fused=False). tflite_convert converts the model successfully with this.
"fused=False" is needed if converting to a quantized flatbuffer file (in my case, I'm training with fake quantization nodes).
Otherwise, there will be errors regarding missing min/max information in the graph:
2019-03-25 15:04:13.845816: F tensorflow/lite/toco/tooling_util.cc:1702] Array batch_normalization_v1/FusedBatchNorm_mul_0, which is an input to the Add operator producing the output array batch_normalization_v1/FusedBatchNorm, is lacking min/max data, which is necessary for quantization. If accuracy matters, either target a non-quantized output format, or run quantized training with your model from a floating point checkpoint to change the input graph to contain min/max information. If you don't care about accuracy, you can pass --default_ranges_min= and --default_ranges_max= for easy experimentation.

The workaround I'm using is to replace tf.keras.layers.BatchNormalization layers with tf.layers.batch_normalization(fused=False). tflite_convert converts the model successfully with this.

you just add the only property "fused=False" in tf.layers.batch_normalization when you training???

@ruffles1 Thank you for the workaround. I noticed tf.keras.layer.BatchNormalization also has the "fused" parameter and I tried to set "fused=False". But now the tf.contrib.quantize.create_training_graph() only inserts fakequant nodes to partial of my graph... Did you try to use BatchNorm from tf.keras?

@ruffles1 Thank you for the workaround. I noticed tf.keras.layer.BatchNormalization also has the "fused" parameter and I tried to set "fused=False". But now the tf.contrib.quantize.create_training_graph() only inserts fakequant nodes to partial of my graph... Did you try to use BatchNorm from tf.keras?

他不是说用tf.layer.bn代替么？

@ruffles1 @hahadashi  I figured out my problem. Currently (r1.13), BN layer has to be followed by an activation layer (say relu) to be recognized by contrib/quantize. Since my network does have linear Conv-BN layer, it will complain about lacking min/max for FusedBN node when convert to quantized TFLite model.

@NEU-Gou I just tried your suggestion, and unfortunately, fake quant ops are still failing to be added to the batch norm ops. I get the same error when converting to tflite. Could you please share an example of a working model that includes fused batch norm and fake quant ops?

@NEU-Gou I just tried your suggestion, and unfortunately, fake quant ops are still failing to be added to the batch norm ops. I get the same error when converting to tflite. Could you please share an example of a working model that includes fused batch norm and fake quant ops?

you try the 1.13??

@hahadashi Yes, I'm on 1.13.1

@ruffles1 Double checked your error massage. Seems that it's a problem related to tf.keras.backend.learning_phase. Did you try to run "tf.keras.backend.set_learning_phase(1)" in training and when converting, BEFORE loading the .pb model, run "tf.keras.backend.set_learning_phase(0)".

@ruffles1 Double checked your error massage. Seems that it's a problem related to tf.keras.backend.learning_phase. Did you try to run "tf.keras.backend.set_learning_phase(1)" in training and when converting, BEFORE loading the .pb model, run "tf.keras.backend.set_learning_phase(0)".

can you try a training experiment on you env, a net including  tf.nn.conv2d + tf.layers.batch_normalization + tf.nn.relu.

I resolved it by saving the trained weights, clearing the session, setting the learning phase, loading the model back, then restoring the trained weights into the (now evaluation) model. Doing this removes all of the 'switch' ops from the graph which relate to the BatchNorm layers and converts your dropout nodes to identity ops. The model can then be frozen like usual.
trained_model.save_weights(output_weights)

tf.keras.backend.clear_session()
tf.keras.backend.set_learning_phase(0)    # This is the important part
eval_model = model_build_function()
eval_model.load_weights(output_weights, by_name=True)

@LukeBolly This is more or less what I've been dong to quantize aware train a model with batch norm. My problem is when training with fused batch norm layers. In this case, the conversion will fail, even with using the method you've described. Have you tested with fused batch norm layers?

@ruffles1 @hahadashi I tried a few different setting and figured some "requirements" for the contrib/quantize/create_train(eval)_graph to work:

For CONV2D+BN+ReLU, the CONV2D op has to be "use_bias=False"
For CONV2D with bias, it CANNOT followed by BN layer

Hope this is helpful. Good luck

@ruffles1 @hahadashi I tried a few different setting and figured some "requirements" for the contrib/quantize/create_train(eval)_graph to work:

For CONV2D+BN+ReLU, the CONV2D op has to be "use_bias=False"
For CONV2D with bias, it CANNOT followed by BN layer

Hope this is helpful. Good luck

thanks，
which api to call the BN layer??(tf.layers.batch_normalization or??) and which tf version you using??

@hahadashi I'm using r1.13 and tf.keras.layers

thanks
发自我的iPhone
------------------ Original ------------------
From: Mengran Gou<EMAIL_ADDRESS>Date: Mon,May 6,2019 5:46 PM
To: tensorflow/tensorflow<EMAIL_ADDRESS>Cc: hahadashi<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tensorflow/tensorflow] TensorFLow v1.13.0rc0 tflite_convert fails to convert TensorFlow model with FusedBatchNormalization layers (#25301)
@hahadashi I'm using r1.13 and tf.keras.layers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

It seems the original script has some bugs (Keras complaining shape mismatch etc). I fixed the script and tried to call TFLiteConverter.from_keras_model_file directly, and everything works fine (with 1.13.0rc0 and 1.13.1).
The takeaway is you don't need to call freeze_graph by yourself when using a Keras model. It seems TFLiteConverter does a better job for you.
The script that works on my machine:
import tensorflow as tf
import tensorflow.keras.optimizers as optimizers
import tensorflow.keras.losses as losses
from tensorflow.keras.datasets import mnist

(train_x, train_y), _ = mnist.load_data()

from tensorflow.keras.layers import *
from tensorflow.keras.models import *
regularizers = tf.keras.regularizers

reg_weight = 0.00001

model = Sequential()
model.add(Reshape((28, 28, 1), input_shape=(28, 28)))
model.add(Conv2D(32, kernel_size=(3, 3),
                 activation='relu',
                 input_shape=(28,28,1),
                 kernel_regularizer=regularizers.l1(reg_weight)))
model.add(Conv2D(64, (3, 3), activation='relu', kernel_regularizer=regularizers.l1(reg_weight)))
model.add(BatchNormalization())
model.add(MaxPooling2D(pool_size=(2, 2)))
model.add(Flatten())
model.add(Dense(128, activation='relu', kernel_regularizer=regularizers.l1(reg_weight)))
model.add(Dense(10, activation='softmax', kernel_regularizer=regularizers.l1(reg_weight)))

model.compile(optimizer=optimizers.Adam(), loss='categorical_crossentropy', metrics=['accuracy'])

train_x = train_x[:100, :, :]
train_y = train_y[:100]
model.fit(train_x, tf.keras.utils.to_categorical(train_y), batch_size=64, epochs=2, verbose=1)
model.save("/tmp/25301.h5")
converter = tf.lite.TFLiteConverter.from_keras_model_file("/tmp/25301.h5")
tfl = converter.convert()
print(len(tfl))

Closing this. Feel free to reopen if there's unresolved questions.


[WIP] Improve testing & add Symfony 3 support

Todo

[ ] Fix DataCollector templates for the new 2.8/3.0 toolbar/profiler.

This is awesome! Great work so far :+1:

Updated the PR to add tests for the RedisLogger and to enable the redis server on travis, so Travis will no longer skip 50% of the tests. This also revealed a bug in RedisCache.

Added the updated templates/icons for the new toolbar/profiler. Screenshots (generated with dummy data):
Toolbar
No commands executed
Icon is hidden, which is the convention in 2.8+.
Commands executed

Failed commands

Profiler
No commands executed

Commands executed

Failed commands

I haven't tested the templates with the older toolbar, but it should be correct (will test this week).

Nice =). Thanks @WouterJ ! I hope you can finish that soon !

I added one PR to this PR: https://github.com/WouterJ/SncRedisBundle/pull/1

Just tested it in a 2.7 app and everything looked correct. When Travis is green, this PR is ready to merge. (maybe there still are some deprecated usages in the code because tests do not cover anything, but users of this bundle will report them)

Thanks for the great work!

No problem :)

@snc do you have an ETA for a stable release including this work ? The stable versions of the bundle are currently incompatible with Symfony 3 (and throw deprecations in 2.8)

I really would like to solve #184 before releasing a 2.0 version because this might break BC for some specific use cases.


Vim-like navigation

Fixes #96
Changes Introduced

Supported key sequences

G = go to last slide
gg = go to first slide
4j = go forward 4 slides
13G = go to slide 13

Extracted navigateX methods to increase maintainability of
model.Update() given the additional functionality.

This is sweet! Thanks so much for implementing this ❤. I test this out soon!

Looks absolutely fantastic, thanks so much! I see you're participating in Hacktoberfest also is there anything I need to do to make this contribution count towards that before I merge?

I think it just needs the hacktoberfest-accepted label please.
Great to work with you on this, happy to take a look at any other issues you think are suitable.


feat: Add validation by data binding in login layout

Fixes #265

Add data binding to login layout
Rename layouts for better binding name
Add data binding validation library
Validate email, password length and base url

Please resolve merge conflicts.


Analytics - Durations.DisplayName should be Name/Identifier

Analytics includes Durations which is a generic way to have game events with durations. Currently the output format includes DisplayName.
example
As I understand it, this couples the name displayed on the report to the name passed in on the event from the game. This limits localisation, changing the displayed name later, ...
I suggest changing DisplayName to Name/Identifier.

@oliviak any thoughts?

We have completely new message models and they will all be described as part of the recipe they belong to. The duration-events are currently not supported, but when they will they will be documented correctly.


change pin name to Reactor Event

This will be the last one for today :)

Hi @jorgemaia, is this closed?


Make assistive technology learning easier to find from IXD pages

Feedback from interaction design that there is no links to guidance for learning voiceover to test prototypes

Guidance will be added when the HTML templates are added for Voiceover testing. Linking to the general guidance on Assistive Tech testing does not feel adequate at this moment.

Guidance will be added when the HTML templates are added for Voiceover testing. Linking to the general guidance on Assistive Tech testing does not feel adequate at this moment.

Added in v1.0.2


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/36249

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/40986

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/42587


fix some typo in readme.md

Fix some typo.

This is not a typo - since iterall is often used to support older browsers, the example intentionally uses ES5 function syntax.

@leebyron There is a missing bracket at line526. That makes me think maybe there was some typo.

Ahh, I think that was solved by #23


Minimal plugin system

Use case
As a developer, I'd like to have a minimal plugin system so that I can organize common features into individual opt-in plugins that I can re-use across projects, so that my scripts.js is easier to maintain, my code more portable and re-usable, and so that I only load what is really needed for my use case.
In particular, I'd like to:

have a simple instrumentation of my scripts.js for the plugins I use
have the capability to pass down configuration settings to the plugin to tailor it for my project
be able to define various "hooks" in my plugin so it automatically runs in the eager/lazy/delayed phases
be able to reference another plugin from my plugin so I can re-use features

Hre is a first proposal on this idea:
https://github.com/adobe/helix-project-boilerplate/pull/135

Proposal was merged in the forked repo at https://github.com/ramboz/helix-project-boilerplate/


Uglify Plugin Error

Hi, when running a bundling process with uglifyjs plugin I am getting following error:
"string" must be a string, Buffer, or ArrayBuffer

Firt it did not even run and complained that 'uglify-js' does not exists so I installed it manually, yet this is the result ;(

This is my config if it is interesting:
const uglify = {
    compress: {
        warnings: false,
        screw_ie8: true,
        conditionals: true,
        unused: true,
        comparisons: true,
        sequences: true,
        dead_code: true,
        evaluate: true,
        if_return: true,
        join_vars: true
    },
    output: {
        comments: false,
    },
    sourceMap: false,
    verbose: true,
    exclude: /compiler\.js/i,
}

Sparky.task("production", () => {
  const productionFuse = FuseBox.init({
    homeDir: "src",
    output: "public/$name.js",
    hash: true,
    plugins: [
      WebIndexPlugin({ template: 'src/client/index.html' }),
      JsxControlsPugin,
      JSONPlugin(),
      UglifyJSPlugin(uglify),
      CSSPlugin({
        group: "bundle.css",
        outFile: `bundle.css`
      })
    ],
    shim: {
      "crypto": {
        exports: "{ randomBytes: () => crypto.getRandomValues(new global.Uint16Array(1))[0] }"
      }
    },
  });

  productionFuse.bundle("vendor.min")
    .instructions(" ~ client/index.ts") // nothing has changed here

  productionFuse.bundle("client.min")
    .instructions(" !> [client/index.ts]");

  productionFuse.run();
});

Nevermind .. I could not make it work with uglify3, so rolling back to uglify2 solved my problems.


Input Video As CAMERA Type

ON the doc it is missing to specify that on each HDMI IN of the codec pro you have a camera connected to you need to config them as a CAMERA Type. This can be done on the Settings of the device VIDEO INPUT TYPE = CAMERA or could be really great to add this on the Macros regarding what is configured on the Config Macro :-)

Good catch, thanks @remylpat
Will work to automatically set the camera inputs based on the existing config


youtube-nocookie.com isn't recognized as a youtube link

Current Behavior

When I add a youtube video with this url :
https://www.youtube-nocookie.com/embed/8xV95bSyHYk
React player doesn't recognize this url as a valid youtube link.
Expected Behavior
Fix regex  MATCH_URL_YOUTUBE to accept https://www.youtube-nocookie.com/embed/
https://github.com/CookPete/react-player/blob/32bb251a60f1a6449bd20e6dcf4c7dbfaeab02b7/src/patterns.js#L1
Steps to Reproduce

Add this url to player : https://www.youtube-nocookie.com/embed/8xV95bSyHYk

Environment

jsFiddle example:  https://jsfiddle.net/3axqk0ch/1/

Thanks

Thanks @CookPete


change team related permissions for project owners

Overview
Project owners need more control over the teams within their projects.
Design Details
Project owners should be able to...

[ ] create teams within their project
[ ] delete teams within their project
[ ] move teams to other projects they have permissions on (i.e., change the project assignment of the team
[ ] change the name of teams within their project (i.e., update)
[x] should be able to add/remove users from teams in their project

this work has been deprioritized, so closing.


[Request] Slide Component for Menu/Dialog/Drawer

Duplicates

[X] I have searched the existing issues

Summary
I have a Dialog component in my project where I display event information. (Name, Last Name, Event Date, Status, Notes etc.). In this Dialog component, there is a menu that lists the actions that can be applied for the event. One of them is "Event History". This option opens a dialog showing who took what action on this event. (Basicly this is logging system i made).
Today, I decided to solve this complexity of 2 dialog components and combine them into a single dialog component. I could solve this simply with a switch function, but it looks very crude without animation. So I decided to use the Collapse component, but I couldn't get it to work exactly the way I wanted. The Settings popup in YouTube's video player looked pretty nice, I wanted to do something similar but I couldn't. After researching, I learned that this component is called "Slide Menu". Would you consider adding something like this in the future?
Note: I am opening this article because I cannot find anything similar to the component I mentioned in the MUI documentation. If I'm too dumb to find it, please guide me.
Examples
The slide menu of Youtube I mentioned
https://github.com/mui/material-ui/assets/20345854/bd764f48-6f9a-4697-99cb-178cae80b957
Another example
https://github.com/mui/material-ui/assets/20345854/3e7fe509-dc5c-47ea-8920-f7454da4743f
Motivation
No response
Search keywords:

Duplicate of https://github.com/mui/material-ui/issues/11723


Text renames

Text type instances -> TextContent, Nbt data factory -> NbtDataSource, Translatable -> Translation (in class)
Cleanup in mappings for their optimization refactor (good refactors)
We might need to replace more translatable with translation, I guess? It says translation than translatable in class toString, and I wonder about the translatable vs translation usages elsewhere.

Replaces #3102, sorry but that one is making wrong renames on factory methods and doesn't properly describe the new text model IMO

Should we close #3101 and #3102?

Hmm, should they be just TranslationContent or should they be TranslationTextContent?

TranslatedContent?

@Shnupbups @apple502j took both of your suggestions, probably translatable vs translation rename will be moved to another one


xScope 4.1.2

It looks like the RegEx was added for 4.0.0 which had a URL of http://iconfactory.com/assets/software/xscope/xScope-4.0.zip:
However, I don't think they are using semantic versioning or anything, 4.0 was just 4.0, not 4.0.0, and 4.1 was just 4.1, not 4.1.0. So I'd suggest we just pass the version through as is.

Thanks.


Build Azkaban against Hadoop 2.10.

Pull Request Test Coverage Report for Build 5805

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 43.608%

Totals

Change from base Build 5784:
0.06%

Covered Lines:
15095

Relevant Lines:
34615

💛  - Coveralls

Pull Request Test Coverage Report for Build 5805

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 43.608%

Totals

Change from base Build 5784:
0.06%

Covered Lines:
15095

Relevant Lines:
34615

💛  - Coveralls


Fix cpu request limit exmaples use the default namespace

Specify the namespace to cpu-example. So, users can copy the yaml and run the task directly.

This change is

hi @nashasha1  - would you restrict this to the non cn/ content? I would love to have the same content applied to kubernetes/kubernetes-docs-cn. but don't want to mix them within a single PR
/assign

@heckj  Done. Thanks for your reply.

/lgtm


[Bug]: Workflow ContextVar Error w/ Phoenix Tracing

Bug Description
This error clogs up our logs.  It doesn't seem to cause any tracing data loss though.
See #16290 for additional notes and suggestions from others.
Filing this for tracking / visibility at least.
Version
0.11.20
Steps to Reproduce
Make a llama_index workflow, configure llama_index to log traces to Arize Phoenix, run the workflow.  Note that this ContextVar error is reported for each workflow run.
Relevant Logs/Tracebacks
Copied from PR #16290
ERROR [opentelemetry.context] Failed to detach context
Traceback (most recent call last):
  File "/Users/loganmarkewich/Library/Caches/pypoetry/virtualenvs/llama-index-caVs7DDe-py3.11/lib/python3.11/site-packages/opentelemetry/context/__init__.py", line 154, in detach
    _RUNTIME_CONTEXT.detach(token)
  File "/Users/loganmarkewich/Library/Caches/pypoetry/virtualenvs/llama-index-caVs7DDe-py3.11/lib/python3.11/site-packages/opentelemetry/context/contextvars_context.py", line 50, in detach
    self._current_context.reset(token)  # type: ignore
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: <Token var=<ContextVar name='current_context' default={} at 0x2acb049f0> at 0x2cb890500> was created in a different Context

I really don't know how to fix this one, especially because the context var in this case is coming from opentelemtry. There's already endless try/except in the dispatcher code.
In any case, thanks for filling. Will take a look at the linked issue

Oh, you linked the issue I worked on lol

I've resurfaced this to the arize team for ideas. Maybe I need to make a PR in he opentelemtry package?

Thanks for taking a look!
Sorry I don't really have much to add.
Following the referenced issue open-telemetry/opentelemetry-python#2606, I tried searching for locations where a span-decorated function directly calls yield.  However, I only found this to occur in stream-related functions, and I don't think I'm using any of those.

I think the main issue is how workflow.run() returns an asyncio future, and with how async event loops work, something about that ends up generating this error from arize/open telemetry
Sounds like the arize folks might be able to make a PR on their end code (just pinged them 👍🏻 )

@GICodeWarrior This should be fixed in our latest release openinference-instrumentation-llama-index>=3.0.4. Please give that a try and let me know if you see additional issues. Thank you!

I just tested version 3.0.4 of openinference-instrumentation-llama-index, and it has resolved the issue for my current case. Thank you for your work!
For those still using version 3.0.3, I found a temporary workaround. In a normal Python environment, for example:
# https://github.com/run-llama/llama_deploy/blob/main/examples/python_fullstack/workflows/__init__.py
from workflows.agent_workflow import build_agentic_workflow
from workflows.rag_workflow import build_rag_workflow
import llama_index.core
import os

PHOENIX_API_KEY = "xxxx"
os.environ["OTEL_EXPORTER_OTLP_HEADERS"] = f"api_key={PHOENIX_API_KEY}"
llama_index.core.set_global_handler(
    "arize_phoenix", endpoint="https://llamatrace.com/v1/traces"
)

rag_w = build_rag_workflow()
agentic_w = build_agentic_workflow(rag_w)

If I add llamatrace directly like this, it throws an context error. However, if I modify it to:
# https://github.com/run-llama/llama_deploy/blob/main/examples/python_fullstack/workflows/__init__.py
import nest_asyncio
from workflows.agent_workflow import build_agentic_workflow
from workflows.rag_workflow import build_rag_workflow
import llama_index.core
import os

nest_asyncio.apply()
PHOENIX_API_KEY = "XXXXX"
os.environ["OTEL_EXPORTER_OTLP_HEADERS"] = f"api_key={PHOENIX_API_KEY}"
llama_index.core.set_global_handler(
    "arize_phoenix", endpoint="https://llamatrace.com/v1/traces"
)

rag_w = build_rag_workflow()
agentic_w = build_agentic_workflow(rag_w)

This change eliminates the error. It’s important to apply nest_asyncio.apply() as early as possible, ideally right after importing other packages, since the logic in those imports might override the nest_asyncio.apply() behavior.

That fixed it.  Thanks!!

Upgrading the dependency broke tracing via trace.get_current_span() (docs) in our llamaindex workflow, downgrading back to 3.0.3 and applying https://github.com/run-llama/llama_index/issues/16914#issuecomment-2469846690 restored the trace information being populated.
Presumably this is because opentelemetry's get_current_span looks for a context first before returning span information and https://github.com/Arize-ai/openinference/pull/1112 removed the context being linked entirely.
Any suggestions for this scenario are welcome, please let me know if this interpretation is totally off the mark.

@jaleel-neon We have opened an issue on our end to implement a replacement of that functionality.


Immutable Properties

I ran across a bug where I unintentionally defined a property with the same name as another property defined in a target file I included. It would be nice if msbuild had facilities to help detect this type of issue. For instance, if I could have somehow marked my property immutable then msbuild could have issued an error when the property was subsequently overwritten in the imported target file.
One possible way to mark properties as immutable would be via an IsReadOnly attribute applied to property and propertyGroup elements. Also, it would also be nice to be able to change the default to readonly via, possibly, a ReadOnlyProperties attribute applied on the Project element.
There is already a mechanism by which command line properties are treated as immutable. Maybe that same logic could be used to enforce a new readonly property feature. That logic would have to be extended to issue an error instead of silently failing after attempting to update the property as is currently the case for command line properties. I'd like to see an error in that case as well. Possibly enabled via an environment variable ala MSBuildWarnOnUninitializedProperty but MSBuildWarnOnWriteToCommandLineProperty.
Warm regards,
Chris

I accidentally duplicated this as #1673, and it appears to have gotten a bit more commentary so I'm going to unify by moving this comment to that thread. Great idea, though!


Update timeout for .github/workflows/performance_trigger_via_label.yaml

This is an automated pull request to update default timeouts for all jobs in .github/workflows/performance_trigger_via_label.yaml that do not currently contain a default timeout property.
Please review these changes closely to ensure that all jobs have sufficient time to run. If a job needs more time, please add a timeout property to that job to override the organization default.
Additionally, please be attentive to unintended changes to the workflow YAML file, as the parser is not perfect.
Lastly, note that it is up to the reviewer to merge this pull request and delete the PR branch once they are satisfied with the changes.

 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.

 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.

Closing in favour of already merged #5733


Can't open chat after 0.14.0 on Chrome if manually inputting path to executable

Description
After the update to 0.14.0, I can no longer open the chat window for any channel on Google Chrome if I input the path to the Chrome executable location in the options menu. After testing I found out that leaving this input blank fixes the issue. This same chat function was working perfectly fine before this update.
Expected / Actual behavior
Open chat on Chrome if you manually input the path to the executable file.
Reproduction steps

Go to Settings > Chat
Choose Google Chrome as Chat Method  to open Chat.
Manually input the path to the location of the Google Chrome executable.
Apply the settings.
Click on the chat icon when a stream starts = doesn't work
Right click on the thumbnail of a stream, click on open chat = doesn't work
Tick on settings the option to open chat automatically = doesn't work

Environment details (operating system, etc.)
Windows 8.1 64 bit. I am using the 64 bit version of Livestreamer Twitch GUI.
My Chrome path = C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
Comments, logs, screenshots, etc.
I've updated to 0.14.0 by just copying the files on top of the old ones. I have always done this and I had no problems until this update.

Seems like you are not the first one experiencing this behavior (#269)

Sorry, I forgot to search the closed issues. However I don't believe this issue should be closed because the behavior of opening chat when you manually input a path to chrome is still not working on 0.14.0.

Thanks for reporting.
The issue was introduced by 399576f. Setting a custom path when choosing Chromium or Google Chrome does not work in v0.14.0.
The previous issue was closed because there were no informations provided. I will keep this one open until the next release, but I'm not sure if I want to release a hotfix for this soon, since this issue can be prevented by leaving the custom path empty.

The problem is that custom chat application doesn't work either. /usr/bin/chromium --app={url} which worked before is now broken. I prefer chromium in app mode just because it makes a pure little chat window.

/usr/bin/chromium --app={url}

Since the implementation of the new parser/tokenizer, all variables need to be inside quoted strings.
This works: chromium "--app={url}"
As I've already said in the other thread, I should have made this more obvious, the tooltip on the "Available variables" text in the GUI is not enough. I will need to add this to the wiki, too.
The variables parser, tokenizer and substitution logic was changed (see 9eac3b4) because of the --player-args livestreamer parameter. Having custom player window titles with channel data was sometimes being escaped incorrectly, since it was based on a static set of characters. The substitution logic was changed so that it will either escape all characters when not using quotation marks around variables or it will use the same quotation mark used at the beginning of the string (which is the expected behavior).
Players like vlc or mpv will correctly parse their parameters with escaped characters (mpv --title \f\o\o\ \b\a\r file), but chromium does not, that's why you're encounting an error.


DSA 4.2.6 LIFO vs FIFO Approach - Audio/Video

Description
Discuss in detail the LIFO vs FIFO approach - their pros/cons, examples, and implementation.
Two contributors may take up this issue depending on Audio or Video format.
Task

[ ] Audio
[ ] Video

Note

Do not use assign command for this issue.
While requesting the issue to be assigned, please mention your NAME & BATCH NUMBER.
Kindly refer to Issue Guidelines.
Changes should be made inside the DSA/directory & DSA branch.
This issue is only for 'GWOC' contributors of 'DSA' Domain.

@nimisha-yadav plz assign this to me for audio part
Jai Gora
Batch - 7


Just fix one typo

Hi,
C

PS: don't worry for the failing tests... It is a bug with phantomjs... I have to fix it :(

Yes, sorry I always do this mistakes... I close and re-open a PR


Add "I can't find an email" function to ODB plug-in

From @JosephMcArthur on March 3, 2016 17:15
This would allow us to hunt down an email later, but still allow people to request. Could give the status "email needed"
Copied from original issue: OAButton/odb#36

Would you need a checkbox by the email field, or should the field just be optional rather than required? The latter option would likely lead to more requests spuriously missing emails.

I'd suggest checkbox. If we were fancy, we could make it look optional and
if they try to submit without it give them a choice.
On Mon, 11 Jul 2016 at 14:00 Steven Eardley<EMAIL_ADDRESS>wrote:

Would you need a checkbox by the email field, or should the field just be
optional rather than required? The latter option would likely lead to more
requests spuriously missing emails.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OAButton/unified-extension/issues/16#issuecomment-231727027,
or mute the thread
https://github.com/notifications/unsubscribe/AEuWZwMQXE6yxElO9fpZmv3NVM-P4v21ks5qUj5UgaJpZM4JEK7n
.

@markmacgillivray sounds like this is no longer in the plugin. Are we good to close?

Issue moved to OAButton/backend #266 via ZenHub


LatentDImInterpolator change num_images calculation

🐛 Bug

To Reproduce
pass LatentDImInterpolator as callback to the lightning trainer.
use the default args: range_start=-5, range_end=5, steps=11
Train a GAN on some data (eg. a vanilla GAN on MNIST)
Expected behavior
The interpolator creates 11 examples along each dimension (11x11)
Hence, num_images should be 121 and num_rows 11
The calculation though is only based on range_start and range_end:
num_images = (self.range_end - self.range_start)**2
num_rows = int(math.sqrt(num_images))
This results in num_images=10 and num_rows=10
In the Logger (Tensorboard) you obtain an image with 10 columns x 13 rows. In the last row there is a single image.
I suppose that the intention was to generate a rectangular image.
I don't think that simply setting steps=10 is a good solution if the interpolated values should still be integers as in the example (-5, -4, -3, ..., 3, 4, 5).
Probably it would be enough to set num_rows to steps.
Environment

PyTorch Version (e.g., 1.0):
PL Bolts version: 0.3.0
OS (e.g., Linux): Linux
How you installed PyTorch (conda, pip, source): pip
Build command you used (if compiling from source): -
Python version: 3.9

Dear @FlorianMF,
Great catch ! Would you like to submit a PR to fix LatentDImInterpolator ?
Best,
T.C

Would you be OK changing the code as follows:
num_rows = int(math.sqrt(steps))
and drop num_images = (self.range_end - self.range_start)**2 ?


#2164

@singeethamsricharan
Tasks 1 & 2
code not available .check and commit back to github

Ok
On Wed, Jul 17, 2024, 4:58 PM krish952-creator @.***>
wrote:

Merged #2177 https://github.com/vajrang-b/UiPathIn30Days/pull/2177 into
main.
—
Reply to this email directly, view it on GitHub
https://github.com/vajrang-b/UiPathIn30Days/pull/2177#event-13535202371,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJX6CMJN7SJEWIUG2BVKAUDZMZIPFAVCNFSM6AAAAABK6TPOI6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGUZTKMRQGIZTOMI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sir I am not getting the pull request
On Thu, Jul 18, 2024, 12:04 PM sricharan singeetham <
@.***> wrote:

Ok
On Wed, Jul 17, 2024, 4:58 PM krish952-creator @.***>
wrote:

Merged #2177 https://github.com/vajrang-b/UiPathIn30Days/pull/2177
into main.
—
Reply to this email directly, view it on GitHub
https://github.com/vajrang-b/UiPathIn30Days/pull/2177#event-13535202371,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJX6CMJN7SJEWIUG2BVKAUDZMZIPFAVCNFSM6AAAAABK6TPOI6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGUZTKMRQGIZTOMI
.
You are receiving this because you were mentioned.Message ID:
@.***>


does not work on ios mobile devices

Hi
The 3d model display functionality on ios devices is no longer working
https://raw.githubusercontent.com/KakNaZlo/yagodka/master/files/turbofit_mobile_1/img/image_2022-09-19_13-10-56.png

Is this due to a MV version change? Can you link to a working and non-working version so we can debug if so? Generally this error occurs due to an iOS update where Apple has changed their USDZ definition and any existing ios-src has to be updated with a USDZ re-exported with the latest version of Reality Converter. @grorg may be able to comment from Apple.


Check v1beta1RemoteRef.Extract for nil

Unless I'm mistaken the type ExternalSecretDataFromRemoteRef makes the Extract field optional. We're running into an issue in our deployment where we have an externalsecret using a Find and not an Extract which seems to cause some issue referencing memory "runtime error: invalid memory address or nil pointer dereference"
sample externalsecret spec to reproduce (running v0.5.2 as well as v0.5.5)
spec:
  dataFrom:
  - find:
      conversionStrategy: Default
      name:
        regexp: /some/path/in/aws-ssm/prod/.*$
  refreshInterval: 1h
  secretStoreRef:
    kind: ClusterSecretStore
    name: secretstorename
  target:
    creationPolicy: Owner
    deletionPolicy: Retain
    name: alertmanager-values
    template:
      engineVersion: v2
      metadata: {}

hi @chrisdotm ! I'm a bit curious with your use case. Are you setting some externalSecrets in v1beta1 but trying to fetch them using v1alpha1 API?

btw, thank you for your contribution!

hi @chrisdotm ! I'm a bit curious with your use case. Are you setting some externalSecrets in v1beta1 but trying to fetch them using v1alpha1 API?

That actually really confused me to, I'm setting v1beta1 and as far as I know never on my own trying to call it as v1alpha1. It seems the webhook is just constantly trying to convert my v1beta1 objects to v1alpha1

Weird. The webhook convertFrom is only called when someone specifically asks for a v1alpha1 resource view.Just to confirm; the crash is actually on the webhook, correct?  Do you happen to have any logs of it?

It might be worth to investigate it. If there are agents trying to read a v1alpha1, there might be agents trying to write on v1alpha1 as well, which can possibly break your setup once we remove the deprecated APIs.

/approve

I’m waiting for a feature that we released to be stabilized (3 releases in a row so far), and then we’ll include your fix :)


Can this be implemented using a Kanban style (drag and drop across multiple sortable lists)

Does anyone have experience with implementing this using Kanban style? If so, are there any resources/code for reference?

Not yet, but if you or anyone is interested in helping make this happen, @jdbence is working on this on https://github.com/clauderic/react-sortable-hoc/pull/138


Machine translation error in Japanese Document.

The Japanese translated string  of "Backing up virtual machines with more than 16 data disks is not supported." may cause a confuse the reader.
Current Japanese translated string "16 台以上のデータ ディスクを搭載した仮想マシンのバックアップはサポートされません。" will be, more than 16 and over ~~~ instead of more than 16.
Thanks

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

ID: 8d3447ce-fb16-f2ec-126e-91168af07bd1
Version Independent ID: 65048fd2-f450-2046-115d-3846a6f5be1a
Content: Azure Backup: Prepare to back up virtual machines
Content Source: articles/backup/backup-azure-arm-vms-prepare.md
Service: backup
GitHub Login: @markgalioto
Microsoft Alias: markgal;trinadhk;sogup;

@KeiroKamioka0000
Thanks for your feedback! We will investigate and update as appropriate.
@Sara-Nicolini - Please look into the translation concern raised by the user.

@SaurabhSharma-MSFT, can you please assign loc issues to @DavidPrio

Hello, @KeiroKamioka0000
Your feedback is much appreciated.
Since your comments are specific to the localization of the documentation, we have moved this issue to the corresponding localization repository and we will close this one.
Please review the status of this issue here https://github.com/MicrosoftDocs/azure-docs.ja-jp/issues/504
Kind regards,
Microsoft DOCS International Team

@SaurabhSharma-MSFT please close this issue since it's beign worked in the ja-JP repo

@KeiroKamioka0000 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Language Setting

How to change the language to Traditional Chinese?

emm, seems the developer has not developed such feature yet


Allow space before “++” and “--”

This is a feature request.
In Gitter when we mention someone with @, the auto-completion would add a space after user’s name. To trigger this plugin, user need to manually remove the space before typing ++. It would be great if a space before ++/-- is allowed.

I don't have Gitter to test with, but this shouldn't be tough, probably just modifying the -- regex and the ++ regex.
Just add \s? after the first closing parenthesis in each regex. So change these two lines:
robot.hear /@?(\S+[^+\s])\+\+(\s|$)/, (msg) ->
robot.hear /@?(\S+[^-\s])--(\s|$)/, (msg) ->

To be like so:
robot.hear /@?(\S+[^+\s])\s?\+\+(\s|$)/, (msg) ->
robot.hear /@?(\S+[^-\s])\s?--(\s|$)/, (msg) ->

Can you edit your local copy karma-classic.coffee, restart, test and report back?

That regex fix worked for me

That regex worked for me

Can verify the regex worked as well


chore: upgrade gateway-api v0.5.1 to v0.6.0

Type of change:

[ ] Bugfix
[ ] New feature provided
[x] Improve performance
[ ] Backport patches

What this PR does / why we need it:

Pre-submission checklist:

[ ] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?
[ ] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Codecov Report

Merging #1623 (16bc5b8) into master (9476e13) will increase coverage by 0.00%.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##           master    #1623   +/-   ##
=======================================
  Coverage   41.30%   41.30%
=======================================
  Files          87       87
  Lines        7460     7462    +2
=======================================
+ Hits         3081     3082    +1
- Misses       4026     4027    +1
  Partials      353      353

Impacted Files
Coverage Δ

...providers/gateway/translation/gateway_httproute.go
48.38% <0.00%> (-0.23%)
:arrow_down:

pkg/providers/apisix/translation/apisix_route.go
31.30% <100.00%> (+0.09%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

/close  #1601


[Request] Windows release

SQL + GREP together make this tool very interesting and I'd like to ask you for a Windows release.
Thanks

Don't got a working Windows machine sadly.

VM?

VM?

Working well. Thank you!
Although, what's with the need to double escape some regex characters (for example, \d, \t)?

Working well. Thank you!
Although, what's with the need to double escape some regex characters (for example, \d, \t)?


Synch up presentation branch with latest from stable-us branch

start to consolidate work done.
Make graphs and notebooks available in presentation branch to begin work on presentation
Uplift / rebase presentation branch with stable-us

branch synched up with latest work done to date.
pulled latest from branch stable-us into presentation branch.


Fixed NestedMesosContainerizerTest hangs on errors.

Those tests would use a named pipe to synchronize with the task being
started.
The problem is that if the task fails to start, reading from the pipe
would block indefinitely, making the tests just hang.
We could update the code to use a read with a timeout, however it's a
bit fiddly and it's simpler to just use the presence as a regular file
as a barrier.
See https://issues.apache.org/jira/browse/MESOS-10226 for context.
Tested by @martin-g
@qianzhangxa

I have merged this PR, thanks @cf-natali for your contributions!


Add draft of CI and CD Github actions

Hi David
I've recently implemented CI and CD with Github actions for a similar project. Maybe this will help you to have a head start with #122 .
You would need to configure the NPM Token within the project and name it NPM_TOKEN.
Unfortunately I don't know how to make the CD action checking if the CI was successful for the tagged commit.
Have a great weekend 🤓

Hi Robin (@openscript),
Wow, thank you! This draft will definitely help get me started.
Have an awesome weekend,
David

Hello Robin (@openscript),
I just wanted to thank you for your draft. It was helpful! A warm welcome to the contributors club 🙂🎉!
Best,
David


feat: UDP listener + connector

Unfortunately UDP is unstable, but I still think this example is valuable.

Can you please run deno fmt?

Not sure this is mergable given the api is unstable and likely to change. I'll fmt when I get to my hotel.

@kt3k, is this something worth adding?


Forward registration events

Once a user registration event occurs, Norhtstar should be responsible for sending that data over to drupal

I think this could also apply to user PUT/update events.

I think we are going to get into circular requests here, since @DeeZone is posting user registration events from the Message broker, which are sent from Drupal... it will then be sent to northstart then back to drupal :arrows_counterclockwise:

@me : confused?!?
 My understanding is the User API is a central store - if an app wants details it makes a request?
Should MB skip user_registration activities from the Drupal site? There are a few sources of user registrations beyond Drupal that I assume would be of value?

This api needs to send requests into Drupal because not all data is stored in Northstar.
This route specifically needs to post to drupal so that when a user from an external app (the mobile app for instance) needs to authenticate against the app, it should be authenticating to both northstar and drupal

Current plan we started going with, at least from the mobile app POV, was to have all user-related activities, including user registration, directed at Northstar. For cross platform support though, these users would also need to exist in Drupal which prompted the need to forward on registration events to Drupal.
As a possible solution to the :arrows_counterclockwise:, what if... user registrations to Northstar that include a drupal_id don't get forwarded onto Drupal. Seems safe to assume that Drupal already knows about this user then.


Fixed typo

Can one of the admins verify this patch?

This fixes #765 since I don't want to get into describing REST architectures vs. RPC and Richardson's maturity model here or now :)

LGTM, merging.

merged: fba3d7f4bd62e65e4d1d9865e78fc07bc1bc210d


react-cookie useCookies does not refresh in parents

Description
The useCookies hook appears to be stateful rather than updating the global context. Is that intentional?
Example
// This function component never updates
function Parent() {
  const [cookies, setCookie, removeCookie] = useCookies(['test']);

  if (cookie.test === 'test') {
    return <Child2 />;
  } else {
    return <Child />;
  }
}

function Child() {
  const [cookies, setCookie, removeCookie] = useCookies(['test']);

  useEffect(() => {
    setCookie('test', 'test', { path: '/' });
  });

  return null;
}

Hi all, @eXon or anyone in the @reactivestack please do you have a workaround for that ? It's strange that updating cookies only affect the current component instead of the global context...

Hi all, @eXon or anyone in the @reactivestack please do you have a workaround for that ? It's strange that updating cookies only affect the current component instead of the global context...

We'd better fix this by ourselves

@eXon know any thing about the above?

I observe this issue in my app.
I did some testing, and concluded that I only need the underlying universal-cookie package.
I wrote the following alternative provider (simpler and less functionality, but does the job):
import { useEffect, useState } from 'react';

import constate from 'constate';
import Cookies from 'universal-cookie';

const globalCookies = new Cookies();

const setGlobalCookie = globalCookies.set.bind(globalCookies);
const removeGlobalCookie = globalCookies.remove.bind(globalCookies);

function useCookies() {
  const [cookies, setCookies] = useState(globalCookies.getAll());

  useEffect(() => {
    function onChange() {
      const newCookies = globalCookies.getAll();
      setCookies(newCookies);
    }
    globalCookies.addChangeListener(onChange);
    return () => {
      globalCookies.removeChangeListener(onChange);
    };
  }, []);

  return {
    setCookie: setGlobalCookie,
    removeCookie: removeGlobalCookie,
    cookies,
  };
}

export const [CookiesProvider, useCookiesContext] = constate(useCookies);

Actually, now that I look at it more, I believe the issue should be fixed by using useLayoutEffect
This ensures that the hook calls addChangeListener before other code runs.
https://kentcdodds.com/blog/useeffect-vs-uselayouteffect#one-special-case
I changed my solution above to useLayoutEffect which resolved the one issue I had - that cookie listener was not in place on initial page load.

Waiting for PR #288 to be merged.

Please get this merged!

Thanks. But I created my own library cookies management since I didn't know when PR was going to merge.
https://github.com/ngxsmart/react-cookie-service


issue persisting email<EMAIL_ADDRESS>

Related Issues: #IssueNumber if necessary

Introduction
Persisting a email address with certain format gets converted to lower case.
Issue Description
Try to register the following email address and search in the Postgres DB for the result:
email address<EMAIL_ADDRESS>expected inserted value<EMAIL_ADDRESS>curretn result<EMAIL_ADDRESS>Steps to reproduce issue
executing the standard v3 code:
add: async (values) => {
    // Extract values related to relational data.
    const relations = _.pick(values, Userlogin.associations.map(ast => ast.alias));
    const data = _.omit(values, Userlogin.associations.map(ast => ast.alias));

    // Create entry with no-relational data.
    const entry = await Userlogin.forge(data).save();

    // Create relational data and return the entry.
    return Userlogin.updateRelations({ id: entry.id , values: relations });
  },

Expected behaviour
find in DB the exact same value<EMAIL_ADDRESS>Actual behaviour
the big I gets converted to small i<EMAIL_ADDRESS>

Sorry this meant to be for strapi first not for bookshelfjs.
Feel free to close this.


[Server] - Job to search for bookmarks that have no image

[ ] Enable Scheduling:

https://www.baeldung.com/spring-scheduled-tasks
@EnableScheduling on the main class

In BookmarkService.java

[ ] Create a conjob that runs once a day.
[ ] Find all Bookmarks from the bookmark Repository have a screenshot and are scrapable.

Missing screenshotUrls are then attempted to be retrieved by calling the sManager.

[ ] Add method to BookmarkRepository to run the query for where url is "" (i.e. empty)

I've seen that the BookmarkController takes in a 'scrapable' boolean when you create a new Bookmark (which is then checked for before the initial title + screenshot scrape). If we build this daily 'fetch missing screenshots' feature in BookmarkService, we won't be able to check this value again since it isn't saved. This could lead to a retrieval of screenshots for sites that initially weren't scrapable. What do you think?

Hello @R-Sandor, I'm interested in working on this issue. Could I work on it?


v4.4.1

Signed-off-by: Takuro Ashie<EMAIL_ADDRESS>

Note: I'm actually okay if the publishment of RPM/DEB/MSI packages
is done tomorrow.
As far as I'm concerned, it is perfectly fine as long as v4.4.1 is available
early this week.

I've uploaded v4.4.1 packages.

Thank you!


Invalid Mutex directory in argument file:${APACHE_LOCK_DIR}

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 2.2.2
VirtualBox 5.2.22r126460
ansible 2.7.2

Your OS

macOS 14

Full console output
https://gist.github.com/kartagis/6d5d4b5cf3484f152f32e78688b3c77f
Summary
I typed apache2 -t and I got back Invalid Mutex directory in argument file:${APACHE_LOCK_DIR}. The offending line in apache.conf is Mutex file:${APACHE_LOCK_DIR} default```

@kartagis - This is expected. You wouldn't typically run apache2 commands outside of the init scripts or systemd unit files, which also include extra environment variables necessary to start/control Apache correctly.
See: https://serverfault.com/a/558296/15673
If you run instead:
$ source /etc/apache2/envvars
$ apache2 -t
Syntax OK

That should work. See: https://askubuntu.com/a/932059/88829


Publication year missing from new record, but validation still passes

Bug
Publication year missing from new record, but validation still passes: 01GJ5KDPWZ3XY4BKDN06379B5D

I thought I deleted this one, it's not in the project 😅 – but the actual question came from the reviewers, there were imports that came without publication year. Still investigating


Build error with vue-ts template

[x] Read the docs.
[x] Use Vite >=2.0. (1.x is no longer supported)
[x] If the issue is related to 1.x -> 2.0 upgrade, read the Migration Guide first.

Describe the bug
Running yarn build on a project generated with the vue-ts template works fine the first time, but subsequent builds spit a whole bunch of minified JS to the console and then stop with an error. The behavior persists, even if one of the application files is changed. The only way to get the build working again is to delete the dist folder.
I also tested with the vue template, and that seems to work fine.
Reproduction
$ yarn create @vitejs/app vite-build-bug --template vue-ts
$ cd vite-build-bug
$ yarn
$ yarn build
# This build will work fine
$ yarn build
# This build will exhibit the described behavior

This screen recording captures the behavior. Sorry it's a little blurry. Had to make it come in under 10MB:
https://user-images.githubusercontent.com/376073/104078207-1e6b4900-51da-11eb-8505-cf4f23bc703b.mp4
System Info

vite version: vite/2.0.0-beta.12 darwin-x64 node-v14.7.0
Operating System: macOS 10.15
Node version: v14.7.0
Package manager (npm/yarn/pnpm) and version: yarn 1.22.10

Logs (Optional if provided reproduction)

Run vite or vite build with the --debug flag.
Provide the error log here.

Please track here https://github.com/znck/vue-developer-experience/issues/151

Please track here https://github.com/znck/vue-developer-experience/issues/151


2 paginates on same page ... how to get which one was selected on page 1

I have 2 paginations ( Ajax) on the same page , with 2 different param_name ( events_page and tasks_page) , it's fine when pages > 1 are selected  :
i.e.  {"tasks_page"=>"3", "action"=>"show", "controller"=>"backoffice/dashboards"
but when the page 1 is selected, then the params doesn't indicate the param_name :
{"action"=>"show", "controller"=>"backoffice/dashboards"},
so I cannot test in the controller which navigation was hit ....
anyway to solve this issue ?

doesn't work ....  when 2 paginate on the same page, with 2 different param_name , an error is raised on the first display ... in regard to the current_page ...

undefined method `current_page'  for #Array:0x007fc8e8350990)...

which seems obvious as there should be  either 2 current_page or current_page set to 1 for both paginations...
I don't know how to init this current_page to 1


In Yii developers often write
var_dump($var);
die();

while debugging. It is exhausting. In Laravel we can simply write
dd($var);

Why not to do the same thing in Yii?

Use VarDumper::dump().
You can wrap it in whatever you want if you need it.

@klimov-paul I know it, but write
use yii\helpers\VarDumper;
VarDumper::dump($var);

or even
yii\helpers\VarDumper::dump($var);

is also overkill.
Much more characters than in Laravel's dd().
http://laravel.com/docs/4.2/helpers#miscellaneous
Surely I forced to wrap yii\helpers\VarDumper::dump($var). Thought, a lot of developers do the same. May be it will be better to add the alias to Yii's core like it's done in Laravel?

Who does prevent you to declare your own function?
function dd($var)
{
    \yii\helpers\VarDumper::dump($var);
    exit();
}

Name it as you like and use for your good.
Where is "overkill" here?

Create functions.php.
Include it just before run() in index.php.
Define hh() there:

function hh($data)
{
    yii\helpers\VarDumper::dump($data, 10, true);
    Yii::$app->end();
}

@klimov-paul @samdark As I wrote before, I already do it. There is no problem to write a function for me. I want to have it out of the box like in Laravel. Just proposed a feature, that can be useful for many of  Yii's users.

It does not make sense.
It is better for developers to declare this function on thier own, with the name they like and any further adjustments.

It could be reflected in documentation section, like: Adding global helper functions. And just to have couple of examples of such cases.

Yep. I'll handle docs.

@klimov-paul

It does not make sense.

Ofcourse it make sence. From team development point of view. Having official shortcuts much more better than having personal home grown shortcuts per team.

-1
I tend to agree to klimov-paul. At least I don't agree to @samdarks suggestion with an extra functions.php file. Not everyone is using the official base or advanced app templates, so this solution would not work for everyone (leading to confusion). If done right, this should be part of YiiBase.php or at least be included from there.

may be need remove namespace for VarDumper ?

+1 (of course!). I havent seen anything more ugly than the function VarDumper::dump() (well apart from Codeception having  \Codeception\Util\Debug::debug() ;)
The whole thing about dumping is fast debugging (the faster the process, the faster the bug found / solution).  Having to remember + type something like 'VarDumper::dump()' is a bit silly in my option (I even have it in my cheatsheet). So dd() will save an enormous amount of time (over time).

It does not make sense.
how could something like a standard + short cut not make sense. Its standardization and optimization ? @akireikin is showing a use case with is implemented in another big framework.

It is better for developers to declare this function on thier own, with the name they like and any further adjustments.
exactly

Another handy usecase is pre():
function pre($mixed){
echo '<pre>';
print_r($mixed);
echo '</pre>';
}

giving wonderful output like:
Array
(
[0] => foo
[bar] => boo
)

@mikehaertl I'm not for adding it to the framework app templates, I'm for documenting the possibility of doing it yourself.

Adding global methods in core framework is not a best solution because there could be conflicts in names with additional extensions.

If introduce shortcuts officialy it should be done for every helper, for every method... Conflicts can be solved by additinal checking:
if (!function_exists('url_to')) {
    function url_to(...) {
        \yii\helpers\Url::to(...);
    }
}

This is how this works in Laravel. And ofcourse it much more usable than import \yii\helpers\Url at top of view (which is pain) and than write Url::to(). Just simple example for simple helper. We have lots of helpers with lots of methods. Little pain become heavy pain. To be honest very often when i write inside view something like ArrayHelper::something() i cleanly understand that something WRONG with Yii at this point.

documenting the possibility of doing it yourself

Will lead to next generation of "the inventors of the wheel". From companies point of view its better to have official helpers or do not have it at all.

If introduce shortcuts officialy it should be done for every helper, for every method
Its not so bad Idea to shortcut a top X of the most used functions.

@creocoder checks with function_exists is a good idea.
What about naming convention? Helper functions naming should be standartized as other functions (camelCase)?

a naming convention related to Yii software wouldnt be so bad either:
yTag() -> \yii\helpers\Html::tag()
yUrl() -> \yii\helpers\Url::to()
yDump() -> \yii\helpers\Vardumper::dump()
Benefits:

super short
easy to remember
functions as an alias (for future naming changes)
still has some indication to Yii because of its Y
this is very likely not to interfere with other functions because of the y-prefix

What about naming convention? Helper functions naming should be standartized as other functions (camelCase)?

CamelCase for methods. For function there is already naming conventions inside php itself and its underscore. Also such helpers is natural for php. For example: date_create_from_format() as helper for DateTime::createFromFormat(). Laravel go php language way by introducing shortcuts for its helpers.

I'm not for adding it to the framework app templates, I'm for documenting the possibility of doing it yourself.

@samdark I see now. Makes perfect sense to me. The discussion here already shows, that it's hard to find a consensus. So IMO this should really be solved on a per-team basis and not be part of the framework itself.

@creocoder Thats a good point.
I've found this listing of naming conventions of other frameworks & CMS.
Here is a PHP naming convention
Note section: User Functions/Methods Naming Conventions
Function names for user-level functions should be enclosed with in
the PHP_FUNCTION() macro. They should be in lowercase, with words
underscore delimited, with care taken to minimize the letter count.
Abbreviations should not be used when they greatly decrease the
readability of the function name itself::

Good:
'mcrypt_enc_self_test'
'mysql_list_fields'

perfect

http://www.yiiframework.com/wiki/793/debuging-variables-in-yii2/

@akireikin that is already possible with a extension 'Kint', it is the same using on Laravel.
Just add it to your composer. For further informations access: http://raveren.github.io/kint/

@marcoantoniooliveira nice extension. Thank you.

Imho this feature should be include in core framework, it encourage clean code and it will make developers happy :)

For anyone wanting a nice var-dumper, I suggest including the same as in Laravel - symphony var dumper (http://symfony.com/doc/current/components/var_dumper/introduction.html)

Add composer dependency: "symfony/var-dumper": "~2.6"
Define dd() function:

function dd()
    {
        array_map(function ($x){
            $dumper = 'cli' === PHP_SAPI ? new CliDumper : new HtmlDumper;
            $dumper->dump((new VarCloner)->cloneVar($x));
        }, func_get_args());
       die;
    }

...and enjoy ;)
It's just a pitty, that Yii's philosophy isn't about making coding fun, easy, convenient, Laravel is so much better at this.

Can be done w/o using external components:
function dd()
{
    array_map(function ($x){
        echo \yii\helpers\VarDumper::dump($x, 10, true);
    }, func_get_args());
}

Just to think about: maybe we should introduce Yii::dump()? It can cover this pupular demand and will have short enough name.

@samdark Symfony dumper is more pleasant to use - expandable, etc.
If I wan't to browse a model, view attributes, etc, dump a model instance:
Yii dumper: https://ziedlejas.lv/screenshots/img.2015.06.04.OvKENP.jpg (couldn't fit in the screen)
Symfony dumper: https://ziedlejas.lv/screenshots/img.2015.06.04.wMDl8V.png

@briedis There is an error if function only defined, but no libraries used.

Class 'HtmlDumper' not found

Before function dd() add
use Symfony\Component\VarDumper\Dumper\HtmlDumper;
use Symfony\Component\VarDumper\Cloner\VarCloner;
use Symfony\Component\VarDumper\Dumper\CliDumper;

Or write something like this:
function dd()
{
    array_map(function ($x){
        $dumper = 'cli' === PHP_SAPI ? new \Symfony\Component\VarDumper\Dumper\CliDumper :
                                       new \Symfony\Component\VarDumper\Dumper\HtmlDumper;
        $dumper->dump((new \Symfony\Component\VarDumper\Cloner\VarCloner)->cloneVar($x));
    }, func_get_args());
    die;
}

@a-komarev
Sorry, yeah, I had them imported, and forgot to copy that part.
To avoid already defined function, one can use function_exists function.
if(!function_exists('dd')){
   function dd(){
      ...
   }
}

I rethought my mind. I agree this functionality to be covered by an external library like ie. Kint http://raveren.github.io/kint/

I think it's better to leave it to developers. Not going to suggest and "official" way of achieving it.

@briedis thanks.
For anyone who wants the same styles as in Laravel var-dumper, just replace $styles array data in HtmlDumper class with the following:
protected $styles = array(
    'default' => 'background-color:#fff; color:#222; line-height:1.2em; font-weight:normal; font:12px Monaco, Consolas, monospace; word-wrap: break-word; white-space: pre-wrap; position:relative; z-index:100000',
    'num' => 'color:#a71d5d',
    'const' => 'color:#795da3',
    'str' => 'color:#df5000',
    'cchr' => 'color:#222',
    'note' => 'color:#a71d5d',
    'ref' => 'color:#a0a0a0',
    'public' => 'color:#795da3',
    'protected' => 'color:#795da3',
    'private' => 'color:#795da3',
    'meta' => 'color:#b729d9',
    'key' => 'color:#df5000',
    'index' => 'color:#a71d5d',
);

If you're a sublime user and want a more consistent, framework-agnostic experience: https://gist.github.com/gr33k01/4d8ee334f38db027acc3360b2c3e988f

Someone could create an extension for this and developers could place it into require-dev 😺

@schmunk42 there is composer require larapack/dd 1.*


Dev 4.1.0

v4.1 build!

looking good man! some perf issues to look at, but sure we can nail them!


Ninja Gaiden 3 Razor Edge (missing backgrounds in end missions - screen settings)

With graphic pack resolution enabled, some backgrounds are missing (show black screen); disabled pack show game backgrounds ok.
tested graphic default profile, 1600, 1920 profiles

in options main menu > screen settings background is missing (black)
in each mission end results only show records etc, art background is missing (black)

log.txt

Does this issue still happen with the latest graphic pack versions?


Lane Continues Running After Build Failure

I'm getting a build error for my android build that has something to do with a gradle configuration. The troubling thing about this is that the plugin doesn't report it.
begin
   ionic(
     platform: 'android',
      keystore_path: ENV['ANDROID_KEYSTORE_PATH'],
      keystore_alias: ENV['ANDROID_KEYSTORE_ALIAS'],
      keystore_password: ENV['ANDROID_KEYSTORE_PASSWORD'],
    )
rescue
  puts "Ionic build failed"
end

When this runs I get an error
[14:03:45]: ▸ BUILD FAILED in 2s
[14:03:45]: ▸ at ChildProcess.whenDone (/path/platforms/android/cordova/node_modules/cordova-common/src/superspawn.js:169:23)
[14:03:45]: ▸ at emitTwo (events.js:126:13)
[14:03:46]: ▸ at ChildProcess.emit (events.js:214:7)
[14:03:46]: ▸ at maybeClose (internal/child_process.js:925:16)
[14:03:46]: ▸ at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)

When that build error happens, the "Ionic build failed" message is never displayed. So I'm assuming the rescue block is never run.  How do I account for a build failure and stop executing my lane?

You probably can't right now - one would have to investigate where in the chain of tools it doesn't properly throw an error that can stop the execution of the lane. ionic, cordova, this plugin... one of those current doesn't properly handle that error event it seems :/


If no enclosureInputs then hide the Enclosure section header

Small glitch, if you don't have any enclosureInputs then the section header is still shown, this change conditionally hides it.

Didn't test that enough, will work on it more!


Manga cover resizes with title size till it disappears

Device information

Neko version: 2.7.0
Android phone & version: Galaxy Tab 3 10.1 (Android 7)

Steps to reproduce
Open Manga with a long title on a tablet
Expected behavior
Manga Cover should stay the same size on the left side and title should be beside the cover
Actual behavior
Manga Cover on the left disappears when title is too long
Other details
Screenshots show how the cover resizes with the title length or even disappears.
Another one on discord tried it with their tablet and also had that problem.

This should have been fixed in 2.7.2


Forced lightning install

🐛 Bug
pytorch_lightning users who install this package, get lightning installed automatically
To Reproduce
pip install pytorch_lightning
pip install lightning-habana
pip list | grep lightning

Expected behavior
lightning habana should use the existing lightning implementation and not install one
This will mean that it requires that lightning is installed first.
cc @Borda

lightning habana should use the existing lightning implementation and not install one

It does but not PL

This will mean that it requires that lightning is installed first.

Not really, once installed, it is used...

Consider the case where the user has only installed pytorch_lightning and is using pytorch_lightning imports
Since installing lightning-habana will force install lightning: https://github.com/Lightning-AI/lightning-Habana/blob/main/requirements.txt#LL1C1-L1C1
This statement will trigger https://github.com/Lightning-AI/lightning-Habana/blob/a79d6d77e79e626acb14e78dbc5c96ad1712bdd9/src/lightning_habana/pytorch/strategies/parallel.py#L21 and the package will be unusable because it will return a lightning strategy for a pytorch_lightning trainer.
My proposal is that lightning is removed from this package's requirements

ok, the limitation here is that if we remove it from requirements while installing pip won't be checking any compatibility among installed and eventually additional packages... so we can do you to resolve this issue bit still not an ideal way...

Why would we need this compatibility check by pip? At runtime, you can check whether the lightning version is valid.

This shall be also resolved before the final 1.0 release
cc: @jerome-habana


关于uwp触摸的问题|The question about uwp touch

Go to start uwp
Click  any folder or  files on the touch screen
See error.you can't open it

- OS Version: [Windows 10 20H2 19042.630]
- App version: [v<IP_ADDRESS>]

I cannot repro your issue.

@TarsChiu Could you try reinstalling the app from the Store?

This issue just like you use the powerpoint on the touch screen ,it is hard to operat
You clicked but it is hard to poen it

@TarsChiu Does this occur in ListView mode only or GridView also?

Ehh,I‘m not sure that gridview mode was the same,
but i'm sure that listview mode must have this situation,
I will test gridview mode tomorrow.

@duke7553 I have tested it today,this issue only occurs on listview mode,other mode is OK.

doesn't work on Lenovo Yoga C640. Reinstall didn't help.

@BabaJJ I have no Lenovo Yoga C640,and reinstall the issue still.

@TarsChiu Is this because of it's too small? Try pressing hard so we can make sure sizing is a problem.

@Jaiganeshkumaran Yes. I should have been more clear with the reason why it's hard to tap items in the DataGrid. The rows are simply too small of a touch target. We have issues open for introducing custom sizing to the DataGrid.

@duke7553 The default control sizing works well on both mouse and touch input but we could automatically change the sizes when running in tablet mode to make it more touch friendly.

Merging with #2279.


Fix rolling windows params

In rolling_beta() windows1 is half year and windows2 is year, so exchange the order.

Fixed in 0.0.35


BIT(1) type cannot load  as type :boolean #1366

I have not been able to test this code it is to facilitate for the use of BIT(1) in MySQL. I was able to test the previous pull request on branch v1.1 and it works. I have sent this as it was requested by Jose in a previous attempt I made at a pull request was wrong.
I believe https://github.com/liveforeverx thought he had updated this to echo 2.0 but could not find the changes there. Hence this update.

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Enable Track One EventHubs Packages

Add Track 1 EventHubs Packages to the list of Artifacts to operate on during CI runs

Thx for quick fix!

/azp run eventhub - ci

/azp run net - eventhub - ci


Typo in URL fixed

Nice work! Was planning to create a addon in the same way - so I'd like to contriubte. A typo in the URL to vcontrold was fixed.

2

Sorry for the mess - will do some work on my fork before re-opening the PR.

Hi @Schm1tz1 ! You are welcome to contribute :+1:  I fixed the typo in my last version. Thank you!


[Feature Request] CSV转换xml

Checklist

[X] I searched exising issues and no one else requests similar feature.
[X] I think that 25%+ users are positive towards this feature.

Describe the feature
todo
Additional context
No response

https://tableconvert.com/zh-cn/csv-to-xml

数据源 csv,xml,excel,json
转换 markdown, xml, excel, json

添加 Commons CSV 支持

freemarker 生成结果

由于设计复杂，需要重构，因此暂缓上线，在1.0.10开发

https://github.com/tbee/xml2xxx

Hello @unknowIfGuestInDream. This issue is marked as deprecated and closed.


Fix ColumnUnknownException when querying for a sub-col of dynamic system column

Summary of the changes / Why this improves CrateDB
Fixes:
cr> create table t (a int) with ("routing.allocation.exclude.foo" = 'bar');
CREATE OK, 1 row affected (1.603 sec)
cr> select settings['routing']['allocation']['exclude'] from information_schema.tables where table_name = 't';
+----------------------------------------------+
| settings['routing']['allocation']['exclude'] |
+----------------------------------------------+
| {"foo": "bar"}                               |
+----------------------------------------------+
SELECT 1 row in set (0.145 sec)
-- since above works, below must return `bar`
cr> select settings['routing']['allocation']['exclude']['foo'] from information_schema.tables where table_name = 't';
ColumnUnknownException[Column settings['routing']['allocation']['exclude']['foo'] unknown]

Relates: https://github.com/crate/crate/pull/17103.
Checklist

[x] Added an entry in the latest docs/appendices/release-notes/<x.y.0>.rst for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in the latest docs/appendices/release-notes/<x.y.0>.rst
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Is it documented somewhere? If yes, then it's a feature, if not, probably okay to treat it as a bug

Thanks @BaurzhanSakhariev , I did search but could not find anything.

Please do another quick reviews @seut @BaurzhanSakhariev did a little more cleaning up.


Update service-connect-to-microsoft-sustainability-calculator.md

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


upgrade_strategy.timeout on upgraded Rancher clusters sets to 0

Need backend fix for: https://github.com/rancher/rancher/issues/27333#issuecomment-663770808

It also looks like the min value that the backend accepts isn't one that can function so that should likely change as well. https://github.com/rancher/rancher/issues/27333#issuecomment-664691819

@codyrancher Having a little bit of trouble reproducing this issue from the API side. Could you look at the steps I followed and see if I am missing anything?
Here are the steps I took:

Start with Rancher v2.3.8, go to v3/clusters and Create cluster by passing name and {} for rkeConfig.
Upgraded to Rancher v2.4.5, go to v3/clusters/clusterID and confirmed upgradeStrategy doesn't show up. (as expected since this is a new filed in 2.4.x)
Edit cluster in API. Default upgradeStrategy now shows up as:
"upgradeStrategy":{"drain":false,"maxUnavailableControlplane":"1","maxUnavailableWorker":"10%","type":"/v3/schemas/nodeUpgradeStrategy"}}
Tried both use cases mentioned in the original comment, (with 2 different clusters)

upgradeStrategy.drain = false
Edit cluster in API again. Edit goes through successfully and default nodeDrainInput shows up.

upgradeStrategy.nodeDrainInput.timeout = undefined
Edit cluster in API again. Marked drain as True and passed nodeDrainInput without timeout field, like "nodeDrainInput": {"deleteLocalData": "true","force": false, "ignoreDaemonSets": true, "gracePeriod": -1,"type": "/v3/schemas/nodeDrainInput"},.
Edit goes through successfully and nodeDrainInput now has the default timeout.


Add few more os communities

with request 1721

@karanshrm44 Please resolve the conflicts and then we will be good to merge, also I added a minor fix.

how to resolve conflicts  @sara-02 , I don't know,

@karansharma first do a git pull origin <your branch name>. It will add my commit to your branch. Then do a git fetch upstream, and the git rebase upstream/master. Here the rebase will not happen and it will point you to the files where the conflicts are. Pick the part of code that should finally go in master, and remove the rest. Make sure to remove the lines with <<<<< and >>>>. Then push the commit to your branch.

You can read  about resolving conflicts under the additional_materials section of this repo as well :)

Is there any whatsapp group of first contribution where all member contact
easily
On Mon, May 14, 2018, 12:12 AM Sarah Masud<EMAIL_ADDRESS>wrote:

@karansharma https://github.com/karansharma first do a git pull origin
. It will add my commit to your branch. Then do a git
fetch upstream, and the git rebase upstream/master. Here the rebase will
not happen and it will point you to the files where the conflicts are. Pick
the part of code that should finally go in master, and remove the rest.
Make sure to remove the lines with <<<<< and >>>>. Then push the commit
to your branch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Roshanjossey/first-contributions/pull/4386#issuecomment-388647220,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOtW8kToSRS7nB9UOqS1ZRNOifZkauisks5tyH5-gaJpZM4T81WU
.

@karanshrm44 there is slack channel. You can join that.

https://firstcontributors.slack.com/join/shared_invite/enQtMzE1MTYwNzI3ODQ0LTZiMDA2OGI2NTYyNjM1MTFiNTc4YTRhZTg4OWZjMzA0ZWZmY2UxYzVkMzI1ZmVmOWI4ODdkZWQwNTM2NDVmNjY

Why this error generated ?? I don't know this is error or not ?? I am just asking why this thing is happen ??

@karanshrm44 Because meanwhile, you were adding changes to the files, so more content got pushed to the upstream master. And now when you are pushing your changing, it is conflicting with the existing code.  The two heads are out of sync, and Git is unable to resolve those automatically, because for the same line number 2 different versions of the code exist, one at your branch and one at current upstream.

I tried , but failed to understand what to do , Sorry , can you help me little bit more

There is error generated
fatal: 'upstream' does not appear to be a git repository
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

@karanshrm44 Can you give me the output of git remote -v?

This is output @sara-02
origin  https://github.com/karanshrm44/first-contributions.git (fetch)
origin  https://github.com/karanshrm44/first-contributions.git (push)

@karanshrm44 You need to go git remote add upstream https://github.com/Roshanjossey/first-contributions.git
And then checkout your branch(if not already done), then do git fetch upstream, and then git rebase upstream master. It will say cannot merge resolve conflicts, then you open the files which it says has conflict and check for <<<< HEAD or >>> HEAD signs, then you can select which parts to keep and which to delete. After that do git add <file names> and then git rebase --continue. Then just do git commit and then git push origin <branch_name>

I have few question ,

Checkout to the main branch or my-branch
2.How can i decide which part should be deleted

@karanshrm44 Your branch.
Whatever changes you finally want to go in the file keep those. If your changes need to go, then delete the ones from upstream. If you have to keep upstream changes because they are more logical, then keep those and delete your corresponding changes.

When I executed git rebase upstream master
fatal: Needed a single revision
invalid upstream upstream

I am still not confused what to do after that command , what item should be deleted ?? i don't know what is happening ?? what is upstream you are talking about @sara-02  mam

@karanshrm44, Sorry I didn't get on this thread earlier.
I've resolved the conflicts. I'll try this in my local and if everything is okay, I'll merge this.
To understand more about resolving merge conflicts, read this
https://github.com/Roshanjossey/first-contributions/blob/master/additional-material/git_workflow_senarios/keeping-your-fork-synced-with-this-repository.md. If you still have questions, feel free to reach back to us here or on slack

Things looks good here. I'm merging

@Roshanjossey We were hoping to have Karan do it himself, as  learning process :see_no_evil:

@sara-02 , sorry, my bad. I didn't understand that intention.
@karanshrm44, if you're interested in resolving merge conflict yourself, I got an exercise for you. Reply if you're interested.

Yes I am @Roshanjossey

@karanshrm44, I've suggested some changes ☝ for another pull request. Once you do it, I'll make some changes that'd result in a merge conflict. We'll take it from there.