Where spec?

Hello!
I've read the README.md and I couldn't find a link to the actual spec. Is the specification already visible to the public?
If so, would it be possible to add a link to the spec in the README.md?

Thanks for opening @Luc45!
While we have released the dev container CLI, the more detailed specification is still under active development and has not yet been published. I've opened a PR to make this status more explicit in the CLI blog post, and we also have https://containers.dev/ with information about this effort.
We'll be sure to let everyone know when the spec is available! And we appreciate you checking out the repo and providing this feedback.
If you have any other questions or feedback, please let us know!


Included agrovoc, type matching needs to be checked again with expert

Runs locally for me, we should check the types again

@fillies it looks good so far but there are still some parameter (e.g. short_form) missing in the response.

@fillies could you add the information from the skos:scopeNote as description? To get the data in the response it is important to add the parameter fields=scopeNote&lang=en to the request.
see e.g. results>2>scopeNote>0

source: https://agrovoc.fao.org/browse/rest/v1/search?query=milk*&fields=scopeNote&lang=en

scopeNote is added as a description, please give me more input on what you want matched, especially for short form

scopeNote is added as a description, please give me more input on what you want matched, especially for short form

type issue
there are two more changes for the prototype. The type in the Skosmos api response has to converted to individual for the ols api. This is a workaround and could be solved in the future on a better way.
type with not supported skos:concept

type with supported individual + short_form

short_form issue
For the short_form you could use the Jena library and use the local name to put it to the response. similar to this:
https://github.com/ts4nfdi/api-gateway/blob/50055f78d104b93ba853179425bdc9bbb978a530/src/main/java/org/semantics/apigateway/api/OlsTransformer.java#L21C1-L27C14

@rombaum fixed the type and short_form issues, you can see the screenshot in the description.


Can not Focus TexField

Hello !
I have a probleme, i can't focus a TextField after used Navigator.push()
I have an example :
https://www.youtube.com/watch?v=ppQUZIc8e7s&feature=youtu.be
InnerDrawer(
   scaffold: Stack(
        children: <Widget>[
          TabBarView(
            physics: NeverScrollableScrollPhysics(),
            controller: _pageController,
            children: <Widget>[
              MainKomitScreen(),
              MainSearchScreen(), // TEXTFIELD INSIDE THIS WIDGET
              MainInboxScreen(),
            ]
          ),
    ),
)

if you have an idea why 😇

update:
the Problem is from FocusScope
child: RepaintBoundary(
                          child: FocusScope(
                            node: _focusScopeNode,
                            child: _scaffold()),
                        )),

if i remove FocusScope i dont have the problem !

hi @NicolasCELLA, have you tried to follow this example?

Yes of course ! But if you look the end of the video ! I can't tab inside the textField !
When a don't use your Widget this working and when inside your Widget i remove
FocusScope(
                            node: _focusScopeNode,

This working !  I dont know why but maybe in your widget something is not unfocused !

@NicolasCELLA I removed focusScope, at the moment it seems to work properly even without. try it and let me know.

Yes this work very well ! But why focusScope why used ? It's was important ? Because a don't see difference without

@NicolasCELLA it is used to move the focus between one widget and another. It didn't work because I forgot to enter a parameter node: _focusScopeNode.
Among other things, I had to put it as a wrapper for the child widgets and not for the scaffold.
Try version 0.5.5 + 2 and let me know

This version work fine 👌 ! But i have a recommandation for your widget :
The possibility of unfocus or not !
Actually :
https://www.youtube.com/watch?v=ynb7kHHsdNI
without :
FocusScope(node: _focusScopeNode, child: _animatedChild()),

but with only :
 _animatedChild()

result:
https://www.youtube.com/watch?v=qKtGlzeNw2M

@NicolasCELLA could you show the source code that creates this problem? I have made several tests and it behaves correctly


Migrate to Spring Boot 2.4.1

This is also relevant to Gemma since we will face similar issues when migrating to Spring 5.
The move from Spring 4 to 5 is pretty straightforward. The most considerable change relates to adapting usage of findOne in CrudRepository since it now returns an Optional. There are some deprecated interfaces and annotations that are easy to migrate.
MySQL 8 is required, and it poses some issues since we're still using MySQL 5 on the server side. We're having issues with server timezone string (workaround exists by setting explicit serverTimezone property), and that's pretty much it.
We need to update Flyway since the Spring Boot integration is not compatible anymore with 3.2.1. Doing so results in changes to the database table which I'll have to investigate a bit to address. I think a workaround exists for the table issue.
For now, the best is to remain on Spring Boot 1.5.22 for the upcoming release and then look into migrating for the next minor bump.

Closing this since the work is in https://github.com/PavlidisLab/rgr/pull/94
Some remarks:

MySQl 8 does not really pose a problem as long as we pass useSSL=false
Flyway can be (and has been) configured in a way that works with older version
There were some issues that were caught in the process whose fixes that can be backported


region `flash' overflowed by 3940 bytes

On a clean clone of master branch,  make px4fmu-v2_default fails with error saying:
100%] Linking CXX executable firmware_nuttx
e:/px4/toolchain/bin/../lib/gcc/arm-none-eabi/4.7.4/../../../../arm-none-eabi/bin/ld.exe: firmware_nuttx section .data' will not fit in region flash'
e:/px4/toolchain/bin/../lib/gcc/arm-none-eabi/4.7.4/../../../../arm-none-eabi/bin/ld.exe: region `flash' overflowed by 3940 bytes
collect2.exe: error: ld returned 1 exit status
make[3]: *** [src/firmware/nuttx/firmware_nuttx] Error 1
make[2]: *** [src/firmware/nuttx/CMakeFiles/firmware_nuttx.dir/all] Error 2
make[1]: *** [all] Error 2
make: *** [px4fmu-v2_default] Error 2

It looks like you're using GCC 4.7.4. The build system uses GCC 4.8/4.9 and they're both (barely) under the 1MB max for a pixhawk.
You'll either need to update to a newer version of GCC or disable modules you don't need by commenting them out here - https://github.com/PX4/Firmware/blob/master/cmake/configs/nuttx_px4fmu-v2_default.cmake
http://dev.px4.io/starting-installing.html

That was it, thanks.


Add note to help message in config file explaining the config file path.

Split out from https://github.com/ZcashFoundation/zebra/pull/83#discussion_r345367887
The generated config file should include a note on the default config file location.

Related to #399

this was fixed in #527


Issue with k8s.io/docs/home/contribute/style-guide/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
two extra asterisks in shotcodes section.
{{< note >}}
**Note:**** The prefix you use is the same text you use in the tag.
{{< /note >}}

Proposed Solution:
fix code fencing
Page to Update:
https://kubernetes.io/docs/home/contribute/style-guide/#shortcodes

/close


The textarea on dependency saves twice on the DB

There is a dual entry for the answer to my dependency question. It is a text type and I think it occurred when I entered a text in the textarea and then I quickly press the complete button. My guess the culprit would be the saving when the element is
blurred and then the saving on the complete button. Hehe
But then..
"The ajax call for the textarea isn't triggered until it loses focus,
so that would happen at the same time as the submit button. Could you
submit a github issue, and take a stab at a solution if you'd like to?"  -Mark A. Wimbiscus Yoon

Are you using the latest Rails 3 branch?  I added some JavaScript  recently that should block the UI during the Ajax request that should prevent your issue from occurring.  Here is the commit: 673da85f658309aac3bb84e1486f76d440d54c06.

I'm using Rails 2.3.8 and thanks for the commit. I've already seen before that "BlockUI" plugin, but I don't know how to use it with surveyor. Your work will serve as a jump start for me. I'm gonna study it later. Thank you very much!

Duplicates #218.  Which has been closed.


Model refactoring

Add a refactor to manage cubes and models in a better way and add unit tests

LGTM


Invitation to transfer this package to nuxt-community

Hi @vaso2,
Would you maybe be willing to transfer this package to the https://github.com/nuxt-community organisation? You will become a member of nuxt-community ofc as well, let me know and I will make the proper arrangements.
That way there are more people who could help you maintain this package and it would really become the de facto standard fontawesome integration package for Nuxt. Eg we could also publish it as @nuxtjs/fontawesome

@vaso2 any plans on doing this?

@pimlie @RonWalker22 Greetings! Thanks for the invitation, I really appreciate this. It took me a long time to consider that. But before I can make a choice, can you tell me more about exact terms? What level of actual authorship and control over this package I'll retain? First of all, I'm interested in that link at the bottom of README. Also, what I'll get as a member of nuxt-community, maybe there is a place where I can read about such things? For example, will I be able to update other packages quickly, without a general flow of waiting for someone to accept PR? More precisely I'm gonna write a plugin for Yandex Metrika soon because current @nuxtjs/yandex-metrika is completely useless:)
Thanks in advance!

@vaso2 Thanks for your interest and nice module ❤ There is a package @nuxtjs/font-awesome which can be used/replaced with a major version bump to your code.
Regarding ownership, the module author (you) is always owner so you can freely make changes and doing releases. Nuxt core team checks the quality and status of community modules and making PRs or Issues for improvements and ensuring the module is not left unmaintained.
For doing changes or releases on other exiting community modules that you didn't create, you need to ask permission from original authors. Also for creating/moving new modules, you need to confirm it with core team.
For specific yandex-metrika, that would be great if you want to take maintenance of it because it currently has not a maintainer.

Backing to issue, the procedure is:

We add you to nuxt-community and NPM @nuxtjs org
The repo should be consistent with the official module-template. You can do that yourself or one of us can happily make a PR for it :)
You move the repo to nuxt-community and do an initial release (for this case, a major bump)


What is the purpose of this if it exists in FSharpx.Extras ?

https://github.com/fsprojects/FSharpx.Extras/

This builder is more convenient to use because it supports arbitrary awaitables including Task, Task<'a>, YieldAwaitable, ValueTask, etc. without requiring wrapper functions. This builder's license is also extremely permissive which makes it easy to drop into a project as if you'd written it yourself. I also believe it to be well tested, particularly around exception handling (which can be a little tricky). However, the main reason to use this builder is that it produces state-machine based tasks which are more like what you get from C#. The FSharpx builder is based on Task.ContinueWith which causes problems with long-running loops. See this issue I opened in June, which has not yet been addressed.
I think the FSharpx builder exists as a "nice-to-have" feature in the FSharpx toolbox, and works adequately for writing some glue code here and there for Tasks. I do not think it is suitable for heavy use, where a huge portion of your code is in task blocks because your system is asynchronous from the bottom-up and you have chosen Task<> as your canonical representation of async work (instead of F#'s Async<> type).
See also this issue from last year. To be fair, it was fixed quickly after being reported, but it appears to have existed for  years before being caught. Trivial use binding scenarios exhibited this problem (e.g. open a stream and call ReadAsync on it).
My point here is not to denigrate the FSharpx project. It's simply that the task builder in that project is a tiny part of the whole thing, and doesn't seem to have gotten much real world exercise. By contrast, the entire point of this project is its task builder and so it really tries to get that one little thing right. My TaskBuilder is not very well known, so I can't claim that it has yet been "battle tested" either, although I do use it in my work and in my other open source projects Rezoom and Rezoom.SQL. It is also in use in Giraffe.

Agree, you've done cool thing.
What do you think about integrating into https://github.com/kekyo/FSharp.Control.FusionTasks ?

Don't think I'll pursue that; although that project is of course welcome to borrow any of this public domain code if they wish to.


Index out of range error

I made the function below to generate random strings from the characters in a given string. For example, "ABCDEFGHIJK".randomStringFromString(length: 3) could return something like "GBF".
import SwiftRandom

extension String {
    public func randomStringFromString(length: Int) -> String {
        let characters: [String] = self.map { String($0) }
        var randomString = ""
        for _ in 0 ..< length {
            let index = Int.random(range: 0 ..< characters.count )
            randomString += characters[index]
        }
        return randomString
    }
}

Occasionally I've been getting Fatal error: Index out of range. So I changed line 4 of the function to let index = Int.random(range: 0 ..< characters.count - 1) and that seems to solve the problem. But I thought ..< should've kept me from getting out of range. Was there a mistake in my function or is it an issue in SwiftRandom?

Weird. I'll investigate further. Thanks for seeing this


Fix: Optimized images

Lossless optimizing for SVG and PNG images

Hi @sergejmueller, I think this step is already handled by Webpack. You can check this file - https://github.com/DeFiCh/app/blob/master/webapp/webpack/plugins/pluginImages.js

@thedoublejay I know. I think the sources should be kept clean, then the build process has less to do and is faster.
But if you think that's unnecessary, we can close the PR.

@sergejmueller I think it should be ok. Thanks for your suggestion. You can close the PR.

Unnecessary, closed.


TypeError: Cannot read property 'data' of undefined

Chrome version: 50.0.2652.2 dev-m (64-bit)
extensions option screenshot:

error screenshot:

appropriate code:
https://github.com/nakajmg/ScriptAutoRunner/blob/68bad3111b6838a59a3c89f0b34c4ac4da98a12d/js/content.js#L55

@kubosho
Thanks for reporting.
I tested this on Chrome Stable & Canary that no problem :information_desk_person:
I'm sorry Now I'm not planning to support Dev Channel :bow:
But when Chrome Stable version is bumped 50, I'll test and fix it :+1:

:ok_woman:

Hi, I'm using Chrome Canary and got a same error.
Is threre any progress?
I'll try to fix it.
Its error was may be from changing content_security_policy declaration that allow-scripts needs sandbox settings I think...


Add support for OpenCodeInterpreter

Add support for OpenCodeInterpreter

@AnitaLiu98
I have not try this out yet but I see the prompt seems to be a bit over specialized and longer compared to the prompts for other models. For a consistency of format related promptings, I generalize the prompt at https://github.com/evalplus/evalplus/blob/ebe8ad0e6a5aeb48dfe7beda94f43e2407d29ef8/codegen/model.py#L976
While I did keep the overall requirements such as system prompt, codeblock and format-following, I squeezed such requirements to zero-shot sentences without demonstration. Let me know if I should make the prompt better. :)


Shortcut Guide will not appear

Environment
Microsoft Windows [Version 10.0.19041.329]
Windows build number: [run "ver" at a command prompt]
PowerToys version: v0.19.0
PowerToy module for which you are reporting the bug (if applicable): Shortcut Guide

Steps to reproduce
Press Windows key for some time (tried different durations) tried both with Powertoys in and without Admin mode
Expected behavior
Shorcut guide overlay to appear
Actual behavior
Nothing appears
Screenshots
n/a

Hi @DDdgf
if for you is acceptable to install an unsigned build, I've prepared one that has an alternative way to invoke the guide.
Beside holding the Win key, it also shows the guide when using Win + ? (in case of a US keyboard it's Win + shift + /).
Let me know if you can test it.
Thanks.
PowerToysSetup-0.19.1-x64-UNSIGNED.zip

I can't do an unsigned build, but I have a similar issue.  Windows key does nothing when held, launches start menu on release.  running 0.19 as well, however I have my right alt key mapped to win key, since I don't have a windows key and I suspect this maybe related to the issue.

@workuser1234
in 0.20 we will release the option to use Win + ?.

@workuser1234
in 0.20 we will release the option to use Win + ?.

Only that, or custom hotkey like Run has? Would be nice 🙂

@rasenplanscher
custom hotkeys are causing all sort of problems in PT Run, we don't want to go down the same road with Shortcut Guide. The Win + ? is an application shortcut that is supposed to work in all cases, including when PowerToys runs as a user and an elevated app is on the foreground.

I was not aware of that situation. Of course, until custom keyboard shortcuts become reliable, a fixed keyboard shortcut is the way to go. 👍

@rasenplanscher
is it still not working for you in 0.20?
If not, what about Win + ? (depending on your keyboard it may be Win + Shift + ?)


🐛 BUG: wrangler pages dev failed to start dev server without any errors when using pnpm and hoist=false

What version of Wrangler are you using?
2.6.2
What operating system are you using?
macOS
Describe the Bug
How to reproduce
Initializing project.
pnpm dlx create-remix@latest wrangler_pnpm

Choose these values
The integrity of 2943 files was checked. This might have caused installation to take longer.
? What type of app do you want to create? Just the basics
? Where do you want to deploy? Choose Remix App Server if you're unsure; it's easy to change deployment targets.
Cloudflare Pages
? TypeScript or JavaScript? TypeScript
? Do you want me to run `pnpm install`? Yes

Add .npmrc and installing dependencies.
cd wrangler_pnpm
echo "hoist=false" > .npmrc
cat package.json | jq '.devDependencies.wrangler = "2.6.2"' > tmp.json
mv tmp.json package.json
pnpm install

Run dev server.
pnpm run dev

Problem
pnpm run dev does not fail, however it does not listen the port on localhost.
❯ pnpm run dev

> @ dev /Users/pddg/Factory/wrangler_pnpm
> remix build && run-p "dev:*"

Building Remix app in production mode...
Built in 172ms

> @ dev:remix /Users/pddg/Factory/wrangler_pnpm
> remix watch

> @ dev:wrangler /Users/pddg/Factory/wrangler_pnpm
> cross-env NODE_ENV=development npm run wrangler

> wrangler
> wrangler pages dev ./public

Watching Remix app in development mode...
🚧 'wrangler pages <command>' is a beta command. Please report any issues to https://github.com/cloudflare/wrangler2/issues/new/choose
▲ [WARNING] No compatibility_date was specified. Using today's date: 2022-12-30.

  Pass it in your terminal:
  \```
  --compatibility-date=2022-12-30
  \```
  See https://developers.cloudflare.com/workers/platform/compatibility-dates/ for more information.

Compiling worker to "/var/folders/r3/ys126msj43dfk6gp9tmq9nm40000gn/T/functionsWorker-0.704646440603278.mjs"...
Compiled Worker successfully.
╭─────────────────────────────────────────────────────────────────────────────────────────────────╮
│ [b] open a browser, [d] open Devtools, [c] clear console, [x] to exit                           │
╰─────────────────────────────────────────────────────────────────────────────────────────────────╯
💿 Built in 222ms
Compiled Worker successfully.
Compiled Worker successfully.
╭─────────────────────────────────────────────────────────────────────────────────────────────────╮
│ [b] open a browser, [d] open Devtools, [c] clear console, [x] to exit                           │
╰─────────────────────────────────────────────────────────────────────────────────────────────────╯
[pages:wrn] Parsed 1 valid header rule.
Debugger listening on ws://<IP_ADDRESS>:52020/07d91f2e-97d6-40c1-a8ac-7aab845c3944
For help, see: https://nodejs.org/en/docs/inspector
Debugger attached.
[pages:wrn] Parsed 1 valid header rule.
Debugger listening on ws://<IP_ADDRESS>:52020/568c28d0-1301-40ad-84e1-a22166e51d07
For help, see: https://nodejs.org/en/docs/inspector
Debugger attached.
[pages:wrn] Parsed 1 valid header rule.

I don't know why it shows duplicate output, but I believe this is unrelated to the issue i am referring to here.
Failed to access to the dev server.
❯ curl http://<IP_ADDRESS>:8788/
curl: (7) Failed to connect to <IP_ADDRESS> port 8788 after 2 ms: Connection refused

Expected
This succeeds when hoist=true.
❯ echo hoist=true > .npmrc
❯ pnpm install
❯ pnpm run dev

> @ dev /Users/pddg/Factory/wrangler_pnpm
> remix build && run-p "dev:*"

Building Remix app in production mode...
Built in 173ms

> @ dev:remix /Users/pddg/Factory/wrangler_pnpm
> remix watch

> @ dev:wrangler /Users/pddg/Factory/wrangler_pnpm
> cross-env NODE_ENV=development npm run wrangler

> wrangler
> wrangler pages dev ./public

Watching Remix app in development mode...
🚧 'wrangler pages <command>' is a beta command. Please report any issues to https://github.com/cloudflare/wrangler2/issues/new/choose
▲ [WARNING] No compatibility_date was specified. Using today's date: 2022-12-30.

  Pass it in your terminal:
  \```
  --compatibility-date=2022-12-30
  \```
  See https://developers.cloudflare.com/workers/platform/compatibility-dates/ for more information.

Compiling worker to "/var/folders/r3/ys126msj43dfk6gp9tmq9nm40000gn/T/functionsWorker-0.9697792702022094.mjs"...
Compiled Worker successfully.
╭────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│ [b] open a browser, [d] open Devtools, [c] clear console, [x] to exit                                          │
╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
💿 Built in 213ms
Compiled Worker successfully.
Compiled Worker successfully.
╭────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│ [b] open a browser, [d] open Devtools, [c] clear console, [x] to exit                                          │
╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
[pages:wrn] Parsed 1 valid header rule.
[pages:wrn] Service bindings are experimental. There may be breaking changes in the future.
Debugger listening on ws://<IP_ADDRESS>:52107/0f4de9de-d88a-401f-8ada-a1ffad82f80f
For help, see: https://nodejs.org/en/docs/inspector
Debugger attached.
[pages:wrn] Parsed 1 valid header rule.
[pages:wrn] Service bindings are experimental. There may be breaking changes in the future.
Debugger listening on ws://<IP_ADDRESS>:52107/7e525afe-a5b6-4073-b3b9-64b4289fc7c2
For help, see: https://nodejs.org/en/docs/inspector
Debugger attached.
[pages:wrn] Parsed 1 valid header rule.
[pages:wrn] Service bindings are experimental. There may be breaking changes in the future.

It is desirable for the hoist=false case to succeed as well.
What cause this
This has been occurring since v2.5.0. However, as of v2.5.0, the error is displayed, whereas it is no longer displayed in v2.6.0 and later.
ERR_MODULE_NOT_FOUND is occurring during Miniflare initialization. However, the logger at that time depends on Miniflare, so it appears to be succeeding without any errors being displayed.
Error is occurred here:
https://github.com/cloudflare/wrangler2/blob/32686e42b055c786f9821bbd66dd33960ab8f4d1/packages/wrangler/src/miniflare-cli/index.ts#L127
But no errors are reported:
https://github.com/cloudflare/wrangler2/blob/32686e42b055c786f9821bbd66dd33960ab8f4d1/packages/wrangler/src/miniflare-cli/index.ts#L202-L206

Thanks for flagging this. In an effort to remove stale issues so we can have better signal on current friction points, we’re closing out issues that have not been updated within the last 6 months. If this is still a problem, please feel free to open a new issue referencing this one. Please be sure to include an updated summary and any additional context and we will review. Thank you!


Setting pretrained = true causes the cnn_learner model to not train

But setting the pretrained flag to false causes the same model to train quickly
This is shown in the following notebooks:

Resnet with pretrained = False -- trains ok

Resnet with pretrained = True -- fails to train

I have found that freezing seems to not work???
https://github.com/tyoc213/fastai_xla_extensions/blob/fix_prev_lenet/exploring_freeze_TPU.ipynb
The 2 outputs of the method I found on the forum for print what is freezed and what is not, shows the same output of pretrained is eithere true or false.
In the section of the nb Manual freezing! I extract the part of the code that freezes it like
n = -1
if tpu_learner_manual.opt is None:
    print("create opttttttttttttttttttttttttttttttttttttttttt")
    tpu_learner_manual.create_opt()
    print(f'create_opt {tpu_learner_manual.opt}')
    tpu_learner_manual.opt = MyXLAOptimProxy(tpu_learner_manual.opt)
tpu_learner_manual.opt.freeze_to(n)
tpu_learner_manual.opt.clear_state()

Fixed in 0.0.4
Root cause analysis:
It wasn't the pretrained flag that was causing the problem. It was the fact that the create_opt method was called before the TrainEvalCallback.before_fit method was called (the before_fit method causes the model to be moved to the learner.dls.device) So even if the dls.device was moved to the TPU device, the learner.model hadn't been moved yet, and when the create_opt creates an instance of the optimizer using the learn.model.parameters() in the optimizer constructor, these parameters were still in the CPU. The additional optimizer hyperparameters and optimizer state were then based of this model parameters. When the model (and its parameters) were then moved later by the TrainEvalCallback.before_fit method, the other optimizer state were not moved along with it to the new device (TPU). This kind of behavior seems to be specific to XLA/TPU behavior as this does not cause a problem with moving the parameter to the GPU.
Fix
I patched the Learner.create_opt and the Learner.summary methods (which require both model and dls to interact) so that they first move the model and dls devices to the TPU before doing their usual thing.
This means that loading a pretrained model now will call move the device and model to the TPU since loading a pretrained model from a file will at some point call the create_opt method to recreate the optimizer state after loading it from a file.


New commit makes tests unable to run

Describe the bug
A clear and concise description of what the bug is.
I installed neotest-zig today with commit 3a1c07319e826ad47b8f59824f6a60205a5e11fd. I tried to run tests but got the following output:
test
└─ run test
   └─ zig test Debug native 17 errors
/home/ziontee113/.local/share/nvim/lazy/neotest-zig/zig/neotest_runner.zig:24:14: error: root struct of file 'debug' has no member named 'lockStdErr'
    std.debug.lockStdErr();
    ~~~~~~~~~^~~~~~~~~~~
/usr/lib/zig/std/debug.zig:1:1: note: struct declared here
const std = @import("std.zig");
^~~~~
referenced by:
    log__anon_7581: /usr/lib/zig/std/log.zig:125:22
    err__anon_3288: /usr/lib/zig/std/log.zig:175:16
    remaining reference traces hidden; use '-freference-trace' to see all reference traces
/home/ziontee113/.local/share/nvim/lazy/neotest-zig/zig/neotest_runner.zig:24:14: error: root struct of file 'debug' has no member named 'lockStdErr'
    std.debug.lockStdErr();
    ~~~~~~~~~^~~~~~~~~~~
/usr/lib/zig/std/debug.zig:1:1: note: struct declared here
const std = @import("std.zig");
^~~~~
/home/ziontee113/.local/share/nvim/lazy/neotest-zig/zig/neotest_runner.zig:24:14: error: root struct of file 'debug' has no member named 'lockStdErr'
    std.debug.lockStdErr();
    ~~~~~~~~~^~~~~~~~~~~

Any commit before that works fine for me.
Steps To Reproduce

Go to dir
zig init
Go to main.zig, run neotest.run.run(vim.fn.expand("%"))

Screenshots

Environment

OS: Arch Linux
Zig version: 0.12
Nvim version: 0.10
Neotest version: HEAD
Neotest Zig version: HEAD

Thanks for the report! That was an oversight from me, should be all good now.


resolve issue #212 Boolean field type doesnt work

Fix input name Boolean field type doesnt work

Fixed it another way 57b1384caa


How can I hide the tabbar in some screen

// 指定页面隐藏header
const Stacks = [HomeStack, AntDMobileRNStack, PluginsStack]
Stacks.forEach((item) => {
  item.navigationOptions = ({ navigation }) => {
    let tabBarVisible = true
    if (navigation.state.index > 0) {
      tabBarVisible = false
    }
    return {
      tabBarVisible,
    }
  }
})

It's not work

Were you able to figure out any solution? I am also looking for the same feature, tabBarVisible is not available in material bottom tab navigation. It is only available for bottomTabNavigator.

Same issue here, some info about solution?

Pull request welcome to add a tabBarVisible option. It needs to add a display: none style when tabBarVisible is false.

added in https://github.com/react-navigation/react-navigation-material-bottom-tabs/commit/f9d62a241e668653eccdb21c805050a645e3ef41

how can i use it.Which react-navigation version contain it tabBarVisible

Thanks Bro

I faced this issue, But I want to hide  tabbar when a I reach a screen deep in the stackNavigator.
my routes are something like this:
materialTabNavigator:{
    Homestack:{
         Homescreen,
         Detailscreen
    },
    Librarystack: {
          Libraryscreen,
          Detailscreen
     }
}

and I want to hide the tabbar when I reach the Detailscreen.
when I use tabBarVisible in Homestack it works fine, But using it in Detailscreen, doesn't.
I'm using react navigation.


Basic general pomset theory

[x] Define pomset L structure --- labeled partial order with L being the label set. That is a partial order over carrier Tplus labeling function lab : T -> L

[x] Make subclass of Elem.eventStruct --- elementary event structure, which is a pomset plus axiom of finite cause (move it to prime_eventstruct.v.

[x] Define homomorphism of pomsets.

[ ] Define isomorphism of pomsets.

[ ] Make subclass of finitely supported pomset.
{fsfun lab with dfl -> \bot} where \bot : L is distinguished label (use inhType ?).
Require that forall e1, e2 \in supp lab. e1 < e2 is false (i.e. strict causality is false outside the finite support).

[ ] Define pomset languages.
There are two ways:
(1) using quotient types
(2) as Prop-predicate invariant under isomorphisms: P : lposet E L -> Prop s.t. forall p q, p ~= q -> P p -> P q.

[ ] Define embedding relation of pomset languages. Prove trivial properties (preorder).

[ ] Define ordinary word language corresponding to pomset.
This should work only for finite pomsets?  The type should be lang: fslposet E L -> seq L -> bool ?.
Prove embedding lemma: (p ~> q) -> lang q ≦ lang p

I've partly addressed this in https://github.com/Event-Structures/event-struct/pull/116


Issue with IPublishedElement vs IPublishedContent

On an MNTP picker, with two allowed doctypes, max one item.
Both doctypes are not element types, the 2nd doctype is created as a child of the 1st doctype, inheriting it's properties etc.
When generating models, the model containing the MNTP picker property is incorrectly generating the return value of the property to be IPublishedElement, when I would expect it to be IPublishedContent, or even the most specific common base model, i.e DocType1.
Looking into the source, the issue arises within
https://github.com/callumbwhyte/super-value-converters/blob/e1689b7e50d207d1b332a6640ab5ebeb3635b01f/src/Our.Umbraco.SuperValueConverters/ValueConverters/SuperValueConverterBase.cs#L49-L70
The method is returning the lowest common denominator interface, As an IPublishedContent implements IPublishedElement, it's returning IPublishedElement rather than IPublishedContent for models implementing both interfaces.

Amending the doctype structure so doctype2 is composed of doctype1 the property is correctly returning the IDoctype1 interface. So it only seems to be an issue with doctype inheritance

Thanks to @ja0b, a fix for this has been released in v2.2.1 (for Umbraco 8) and v4.0.1 (for Umbraco 9+)


Optional OomErrors

(Mostly) as discussed in #20.
SwapchainImage::from_raw isn't changed yet.
I fixed the teapot example to work with my changes. The other examples were broken anyway.
I'm not confident about my ability to change stuff inside these huge macros without breaking them, so you should probably double-check them. ;)

Thanks! That's very nice!
About your comments on the fact that Arcs can be omitted, I'm really not sure. It depends on how semaphores/fences are going to be pooled, whose design is currently blocked by some ambiguities in the Vulkan specs.
Did you try running cargo test --lib to see if all the tests compile?

Did you try running cargo test --lib to see if all the tests compile?

Oops! I knew I forgot something. I'll fix those now.

bind_memory doesn't need a raw variant. It's an unsafe and dangerous function which isn't supposed to be used directly, so making it more convenient to use doesn't make sense.

I reverted it in cebabe1f639c45037cb8d40d2a5ca3a88ecba722.

Since the documentation is so long, could you remove the docs from raw and just say See the docs of new()?
This would avoid having to maintain the same docs twice.

I replaced all docs for the "raw" functions in b8269ead29d1b30af09ed8eaddea1095b62f1183.

Did you try running cargo test --lib to see if all the tests compile?

I fixed them in b6b073c888dbb257d2c904572aabfb5d2e69a148:

test result: ok. 64 passed; 0 failed; 6 ignored; 0 measured

Did I forget anything?

Thanks!


Understand why SpringBatch instrumentation is not working using the OTEL agent

I've been using this sample https://github.com/spring-guides/gs-batch-processing
And have been trying to use the Spring Batch instrumentation in the OTEL agent. I added the system properties described here: https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/instrumentation/spring/README.md But was not able to get it to work.
Can you try to troubleshoot?

This app is spring-boot 3.2.0 , spring batch 5.1
otel still doesn't support spring-batch 5, they support only spring-batch 3.
https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/7306
you and Arik were through it already
https://github.com/digma-ai/digma-collector-backend/issues/1199
and even for spring-batch 3 the spring-batch instrumentation is disabled by default and you need to enable it
JAVA_TOOL_OPTIONS=-Dotel.instrumentation.spring-batch.enabled=true


fix neutron test script according to soren comments

this patch updates the neutron test script to make
it a little more reliable.

 If
previous ones get left, just leave them there and clean
them up manually.
Keep a list of objects to delete so that we can ensure
that we also remove them on failures cases (or at least
try to)

please test this

this build failed b/c of a ceph issue that we've already resolved. It verifies that the changes work, but it is worth noting that since it no longer cleans up all networks,  there are already 2 leftovers just after 1 day.

please test this

please test this


dracut/99emergency-timeout: print explainer when Ignition fails

When timeout.sh is run due to a failed boot, check if either the ignition-disks or ignition-files units have failed. If they have, wait until journalctl has stopped receiving messages and then reprint the failed logs. Also print the emergency shell prompt via \r instead of \n so it doesn't slowly push the Ignition logs off the screen.
Waiting until journalctl has stopped receiving messages for 5 seconds (with a 15 second timeout) is a workaround for not being able to prevent systemd from printing to the console once the failure state is reached.
The motivation for this is to make it more obvious to users what went wrong when a machine fails to boot due to a bad Ignition config.
What this looks like in practice:
In QEMU:

In EC2:

Updated, please take a look

Updated, thanks to your suggestions this doesn't need tail or date anymore.

@bgilbert I think this is ready for another pass


Future of fluminurs?

It seems like NUS will completely migrate to Canvas in 22/23 Sem 2, so fluminurs would effectively be defunct. There are a ton of file downloaders for Canvas around (for example https://github.com/stein212/canvas-downloader for a Rust-based version).
However, fluminurs already has a lot of the plumbing done, eg. handling updated files via renames/overwriting.
I'm thinking that it's possible to update fluminurs for downloading files from Canvas, since the trait-based implementation should make it quite straightforward.
In terms of downloading videos from Zoom/Panapto, I think this would be slightly harder: there doesn't seem to be an easy way to get the relevant URL for Panapto based on my quick DevTools experimentation.
Any thoughts?

Sorry, I didn't see this previously. What about contributing to the abovementioned project instead? I'm not sure maintaining another project helps much in this case.


LiveReload, BrowserSync, gulp-connect: Nothing is working in V4

Hi,
I have updated to V4 and now not even one of the live-reload browser preview functionality is working. Could you please share an example of a working V4 gulp setup that automatically updates a the index.html inside the browser upon change.
Thanks.
So this is not working, cause I think gulp-connect is not working with V4.
var gulp = require('gulp');
var connect = require('gulp-connect');

// TASK: connect
gulp.task('connect', function() {
    connect.server({
        livereload: true,
    });
});

// TASK: preview
gulp.task('preview', function () {
    gulp.src('index.html')
        .pipe(connect.reload());
});

// TASK: watch
gulp.task('watch', function () {
    gulp.watch('*.html', gulp.parallel('preview'));
});

// Task Bundles
gulp.task('default',
    gulp.parallel('connect', 'watch'));

This is not working, because the example provided is not compatible with the new V4 workflow.
var gulp = require('gulp');
var browserSync = require('browser-sync').create();

gulp.task('preview', function() {
    return gulp.src('index.html')
        .pipe(gulp.dest('.'))
        .pipe(browserSync.stream());
});

gulp.task('serve', ['preview'], function() {
    browserSync.init({
        server: './'
    });
    gulp.watch('index.html',
        gulp.parallel('preview')
    );
    gulp.watch('./*.html').on('change', browserSync.reload);
});

gulp.task('default',
    gulp.parallel('serve', 'watch'));

What I am supposed to do to get the index.html update in the browser, it has been hours trying to fix this and migrate.

Forget about this. I have just switched to Grunt and guess what, it is working!


Separate Record into PackedStruct and Interface components

Record.connect is very rarely useful, but looks broadly useful. It should be deprecated (and ideally replaced with something better).

Both connect and the direction functionality are used in the test suite (obviously). How would you like to deal with that? Put a with warnings.catch_warnings around them (perhaps with checking we got the right warning), as is done for the AST warnings? Or is there more that needs to be done?

with warnings.catch_warnings is fine; nMigen does have some tests for deprecation warnings but not many of them, so either direction is OK.

After further discussion both on PR #368  and on IRC, the current thinking is that we'll split Record into two components - a PackedStruct component, representing the idea of multiple Values packed together into a larger Value, and an Interface component, representing the idea of multiple signals with defined directionality which can be connected together. Record would be deprecated during the 0.4 development cycle and removed at the time of the 0.5 release.

How should the user interface of PackedStruct and Interface look like, how does one create such a value.
Currently I can think of atleast three different ways:

Using class definitions, similar to how dataclasses look:

@packed_struct
class AxiProtectionType:
    priviledged: unsigned(1)
    secure: unsigned(1)
    is_instruction: unsigned(1)

def stream_type(payload):
    @interface
    class Stream:
        ready: SinkToSource(unsigned(1))
        valid: SourceToSink(unsigned(1))
        last: SourceToSink(unsigned(1))
        payload: SourceToSink(payload)
    return Stream

The main disadvantage I see with this method is the need to dynamically generate the classes in case the shape of some part should be parametrized. Also it might not be obvious in which order the bits are layed out. Finally it is more difficult to generically transform something declared like this (like for example reversing the order of the fields).
2. Using a layout DSL like Record currently does:
AxiProtectionTypeLayout = [("priviledged",  1), ("secure", 1), ("is_instruction", 1)]

def stream_layout(payload_layout):
    return StreamLayout = [
        ("ready", sink_to_source(1)),
        ("valid", source_to_sink(1)),
        ("last", source_to_sink(1)),
        ("payload", source_to_sink(payload_layout)))
    ]

This would work the same way  as the layout specification of Record, but atleast written this way, it lacks the structure the other approaches have. For example one can not see at a glance, if a layout description is for a Interface or a PackedStruct.
3. Adhoc based on fields present on a object:
class AxiProtectionType(PackedStruct):
    def __init__(self):
        self.priviledged = Signal(1)
        self.secure = Signal(1)
        self.is_instruction = Signal(1)

class Stream(Interface):
    def __init__(self, payload: Value | ValueCastable):
        self.ready = SinkToSource(Signal(1))
        self.valid = SourceToSink(Signal(1))
        self.last = SourceToSink(Signal(1))
        self.payload = SourceToSink(payload)

This is very adhoc and would probably need some kind of filtering of the instance variables based on type.
How does a user use a PackedStruct or a Interface
For PackedStruct some time ago @whitequark  suggested only allowing access to the fields via __getitem__ or a additional namespace similar to m.d.<domain>. With the introduction of ValueCastable however I don't think this restriction is necessary any longer.
For Interface I currently envision something like this:
class A(Elaboratable):
    def __init__(self, input_stream):
        self.input_stream = input_stream
        self.output = Interface.like(input_stream).source_side()
        self.packet_counter_stream = Stream(Signal(32)).source_side()

    def elaborate(self, platform):
        m = Module()
        packet_counter = Signal(32)
        m.d.comb += self.output.connect(self.input_stream)
        m.d.comb += self.packet_counter_stream.payload.eq(packet_counter)
        with m.If(self.input_stream.ready & self.input_stream.valid):
            m.d.sync += packet_counter.eq(packet_counter + 1)
            m.d.sync += self.packet_counter_stream.valid.eq(1)
        with m.ElseIf(self.packet_counter_stream.ready & self.packet_counter_stream.valid):
            m.d.sync += self.packet_counter_stream.valid.eq(0)

(Yes this example is stupid and broken)
What do you all think?

To the first question, I highly prefer option 1) or 3) since these result in working autocompletion, while 2) cant be understood by my IDE.

from the yosys issue:

By the way, there is no such thing as a "transparent" asynchronous read port, and it never behaved differently from a plain one. It just so happens that a check function meant to trigger on invalid memory configs was added, and caught this.

It seems like we should reflect this in the nMigen Memory / FIFO?


Auto-run during acceptance tests

The current API for this is very explicit, in that you have to explicitly call a function anytime you wish to test for accessibility. Accessibility testing should be more implicit, as we should expect it to happen. Therefore, we should schedule these tests to run automatically during acceptance tests (maybe even unit tests?) after each rendering update to the DOM.
I imagine, exploration of performance improvements will be needed to make this not super slow.

Issue was relevant to the older version of ember-a11y-testing.


feat!: Remove ZeroMQ MessageBus capability

Signed-off-by: Valina Li<EMAIL_ADDRESS>

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/device-mqtt-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions
run make build and make test,
edgex-compose run make run no-secty mqtt-broker' run export EDGEX_SECURITY_SECRET_STORE=false; ./device-mqtt -cp` under device-mqtt-go/cmd
verified all completed successfully without CGO and ZMQ libs
New Dependency Instructions (If applicable)

Codecov Report

Merging #449 (6d466f0) into main (04dd21c) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 6d466f0 differs from pull request most recent head 0a42294. Consider uploading reports for the commit 0a42294 to get more accurate results

@@           Coverage Diff           @@
##             main     #449   +/-   ##
=======================================
  Coverage   21.34%   21.34%
=======================================
  Files           5        5
  Lines         492      492
=======================================
  Hits          105      105
  Misses        374      374
  Partials       13       13

Impacted Files
Coverage Δ

internal/driver/config.go
15.38% <ø> (ø)

internal/driver/driver.go
12.74% <ø> (ø)

internal/driver/incominglistener.go
0.00% <ø> (ø)

internal/driver/readingchecker.go
100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


JSON Request to Insert record - Data type issue

HI , I have Vertex with 2 fields , 1 as date type and 1 as double.
I am unable to update or rather create a new record to this vertex  using json request.
http://localhost:2480/command/demo/sql/INSERT INTO FundTransfer CONTENT {"FundTrnsfrAmt": "2000.90", "TrnsfrDate": "20-04-2016"}
but it created the FundTrnsfrAmt and TrnsfrDate as string values. I am not able to create them as double and datetime values.
I reffered all the documents couldnt find any solution.
Can you please how can i insert record using JSON request which as datatypes other than String ?

Hi @SatishAMP
A couple of points here:

remember to send the request in POST, otherwise it won't succeed
double values are correctly stored if you just remove the quotes: {"FundTrnsfrAmt": 2000.90,
for the date field, you have to define the property in the schema and then pass the value as a string, in the correct date format (the default is yyyy-MM-dd, but you can change it in the db config), eg. "TrnsfrDate": "2016-04-20"}, OrientDB will do the conversion automatically

I hope it helps
Thanks
Luigi

Hi @SatishAMP
This this help? Can i close this issue?
Thanks
Luigi


Make Behaviors configurable.

You can now enable/disable behaviors you don't want using the bundle config.
By default, all behaviors are enabled. As soon as knp_doctrine_behaviors is
specified, all behaviors are disabled and the developer need to specify which
ones he want to enable.
This fix conflict issues, for example when a setCurrentLocale method is
defined by another third-party library/bundle.
This way does not introduce BC breaks, but only a new feature. The best would
be to add some interfaces and use them in subscribers. But that would imply
a major release... Maybe for the next one :-)

I think now we have a bundle, we should load the orm-services.yml file automatically, and we would thus have complete control over definitions.
The way you implemented it right now is to let the user include orm-services.yml,  and then remove the unwanted definitions (thus the constraint with bundle registration order).
If you go the other way around ( generate definitions on the fly only for wanted behaviors), all the problems are gone.
Thoughts?

EDIT: sorry, the yml file is already included by us.
Anyway, I think we simply could remove the definition in the extension class directly, instead of using a compiler pass.

LGTM

:+1:


Plugins support for Import/Export resources

Is your feature request related to a problem? Please describe.
Currently only GCP Cloud storage is supported for import and export of datapumps:
https://github.com/GoogleCloudPlatform/elcarro-oracle-operator/blob/d2aed2814023cc8b672c727b605165d8fdccfffd/oracle/api/v1alpha1/import_types.go#L41
As a result such features are unusable in Enterprise environments with restricted networks and strict rules for data location.
Describe the solution you'd like
I suggest to change the "generic" Import/Export resources to use something like http/https urls and add ability to "load" plugins for different storages on different cloud providers or protocols (nfs, cifs or any other crazy things)

Hi Igor! This is a great suggestion. Do you have in mind any specific API/plugin/library that abstracts away storage for various cloud and on-prem providers?
Ash G.

@akinfermo Hi!
Last month I've been working with kaniko - a container image build tool  inside a container or Kubernetes cluster. The interesting option is --context where one defines path/url with the appropriate prefix to specify the location of your build context. It supports number of sources out of the box - local dir, targz, gcs, s3, git repo, etc... Inspired by such functionality, I would like to generalize the GcsPath to something like ImportPath and use the code to retrieve the prefix for appropriate backend storage:
https://github.com/GoogleContainerTools/kaniko/blob/dac411f7d61290afcd6f1a226f60ead33acfe82f/pkg/buildcontext/buildcontext.go#L51-L67

Hi @ITD27M01
Your proposed approach sounds reasonable to me. Feel free to file a pull request with your proposed changes. Thank you!

Closing this due to inactivity.


On Android device, when we go to Background to tracking gps, it will appear a notification and then we kill the app but the notification won't be disappeared.

🐛 Bug Report

Expected behavior
Reproduction steps
Configuration
Version: 1.x
Platform:

[ ] :robot: Android

I same

Do you have steps to reproduce this issue?


Project Browser Name column is really small on first open

Version and Platform (required):

Binary Ninja Version: 3.6.4789
OS: macOS
OS Version: 14.0
CPU Architecture: M1

Bug Description:
When opening a project, the initial width of the "name" column in the browser is really small and can only fit like 5 letter maximum. I end up having to resize it every time I open a project, which is rather frequent since my workflow involves restarting Binja a lot of times and I have to open the project after every restart.
Steps To Reproduce:
Please provide all steps required to reproduce the behavior:

Open a project
Observe width of columns

Expected Behavior:
I expected one of the following:

Binja has a sensible default width of the name column such that I don't need to resize it
Binja remembers what width i previously gave to the name column and restores that on open

Screenshots:

Additional Information:
When i resize the window, the only column that resizes is the last one, Status. This is almost never the behavior I want (I want the name column to expand when I resize the window).

This was fixed in 3.6.4786-dev


Add a Merge utility type

Search Terms
merge types
Suggestion
It would be great if you could merge two types. The closest I can get right now is a union:
type Test1 = { id: number, code: string }
type Test2 = { id: string, code: number }
type Test3 = Test1 | Test2

// This is not allowed, because x doesn't match Test1 OR Test2
export const x: Test3 = {
  id: "bob",
  code: "bob"
}

Basically I want to be able to create the following type by merging Test1 and Test2:
type Test3 = { id: number | string, code: number | string }

Perhaps there could be a utility to merge the types:
type Test3 = Merge<Test1, Test2>

Use Cases
React-Native has a few style types (TextStyle, ImageStyle, ViewStyle) and they are very useful for autocomplete and type checking in my code. The TextStyle has a property fontSize: number. I want to be able to redefine the TextStyle type to allow a string value for all of the properties (so that I can use my own variable system). Something like fontSize: "$fontSizeMD"
Examples
type EImageStyle = ImageStyle | Partial<Record<keyof ImageStyle, string>>
type ETextStyle = TextStyle | Partial<Record<keyof TextStyle, string>>
type EViewStyle = ViewStyle | Partial<Record<keyof ViewStyle, string>>

type EComponentStyle = EImageStyle | ETextStyle | EViewStyle
type EComponentStyles = { [P in keyof any]: EComponentStyle }

With the merge it would look like:
type EImageStyle = Merge<ImageStyle, Partial<Record<keyof ImageStyle, string>>>
type ETextStyle = Merge<TextStyle, Partial<Record<keyof TextStyle, string>>>
type EViewStyle = Merge<ViewStyle, Partial<Record<keyof ViewStyle, string>>>

type EComponentStyle = EImageStyle | ETextStyle | EViewStyle
type EComponentStyles = { [P in keyof any]: EComponentStyle }

Checklist
My suggestion meets these guidelines:

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[X] This feature would agree with the rest of TypeScript's Design Goals.

I wouldn't use the word "merge" because it means different things to different people.
I'm not sure if this should be a utility type inside of TS itself. I don't see this as being a common use case.

Naming
Perhaps Merge isn't the right word for it, alternatives could be: Join, Combine or Blend?
I chose Merge, because it feels related to declaration merging.
Different ways of merging
I don't think that it should matter what the left or the right operand is. If you want to do an intersection you can probably get that working by using Merge together with the Omit and/or Exclude utilities.
Use cases
Another use case that I can think of:
Let's say you have a repository of objects Repository<T> and it has a find function to query objects: repository.find({ age: 20 }). The function enforces type-safety by only accepting arguments of Partial<T>.
Next you want to make the function a bit smarter and make it possible to do calls like repository.find({ age: { operator: "lte", value: 20 }}).
You can use a union for that, but then you won't be able to mix the two:
repository.find({
   age: 20,
   name: { operator: "regex", value: "^(Lord|Lady) (.+)" }
})

Your use cases just don't seem common or general enough, imo. But maybe others will feel differently.

Yesterday I was working on our React-Native app and I had another use case for this.
React-Native offers an Image component, but it doesn't have a built-in loading indicator. So I was working on a RemoteImage component that displays a loading indicator until the image is fully loaded.
I want the source property of the RemoteImage to be a bit smarter than the original Image component. You should be able to specify a direct image url (string) or a more complex ImageURISource object. The original Image component only supports the ImageURISource.
Currently, this is the Props type for my RemoteImage component:
type Props = {
  source: string | ImageURISource
  style?: StyleProp<ImageStyle>
}

The Image component has a lot more properties, so it would be great if I could do this:
type RemoteImageProps = {
  source: string | ImageURISource
  ...
}

type Props = Merge<ImageProps, RemoteImageProps>

It would make sense to have a merge as it's already implemented in JS.
const one = {
   message: 'Some text',
   overridedProp: 5 // number.
}

const two = {
  overridedProp: 'test' // string.
}

const three = {
  ...one,
  ...two
}
/*
{
   message: 'Some text',
   overridedProp: 'test' // string.
}
*/

Why not use Omit?
interface ButtonProps {
  color: 'black' | 'white',
  children: React.ReactNode
}

interface ButtonExtendedProps {
  color: string
}

type ButtonCombinedProps = Omit<ButtonProps, 'omit'> & ButtonExtendedProps

@alexandermckay it's not very practical when there're more than two props.
I'm using this helper, though:
export declare type Overwrite<T, U> = Omit<T, keyof U> & U;.

A quick attempt to create the ask... Please feel free to tell me what it's failings are...
type Merge<T0, T1, T2 = keyof (T0 | T1)[]> =   (Pick<T0, Extract<Extract<T2, keyof T0>, Extract<T2, keyof T1>>> &
Pick<T1, Extract<Extract<T2, keyof T0>, Extract<T2, keyof T1>>>) &
Partial<Pick<T0, Exclude<Extract<T2, keyof T0>, Extract<T2, keyof T1>>>> &
Partial<Pick<T1, Exclude<Extract<T2, keyof T1>, Extract<T2, keyof T0>>>>;
Added T2 to allow for specifying the keys you wish to merge from either side..
Hope it helps someone. I'm sure a more advanced TS developer could do better


TwitterClient

TwitterClient will be  a new class that will contain the execution context.

Credentials
Execution Settings
Exception Handler
...

This will allow a correct approach to the async/await complexity.
Twitter client should only expose async methods.

Done :)


Update Tag component

Description
spacing, icon option, image tags style, colors for selected tags
The design can be found on the contributions.

FYI Shantanu is looking a little out of place in the most recent release of this: https://opensource.adobe.com/spectrum-css/tag.html
I'd expect maybe that the Avatar therein expects some version of spectrum-Avatar--size50 to display correctly?


Inhouse docker image builds

This hugely builds on the work by @ajacobson (https://github.com/ajacobson/chiadog-docker) and no disrespect is intended.
Proposing merge to main instead of dev since this does not in any way modify how Chiadog works and will then enable images to be built for any and all branches regardless of when dev is merged to main.
Major benefits of inhousing:

Can easily build any state of the repo regardless of branch or
mid-development into a functional image without mucking about with an
external repo. Doing other chiadog development made me frustrated enough to also fix this. :-)
Faster builds when the source code is already present without an extra clone
Automated Docker image builds into the free GitHub image repository. Official images triggered by official tags & releases, but also dev images any any other branch provide repeatable and traceable pinning of images.

Minor fixes added as well:

entrypoint.sh was missing a shebang
Simplified entrypoint running now that the script has a proper shebang
Simplified logic by switching WORKDIR earlier
Add a .dockerignore file to prune dirty state and useless weight from
the container.

Does not include:

Any documentation. Docker is an advanced option in any case but this
should be added as a followup.

Oops, let me rebase this from dev -> main.

Example image has been automatically built in my fork: https://github.com/Artanicus/chiadog/pkgs/container/chiadog .. this shows the logic for non-special branches with the image name becoming:
ghcr.io/artanicus/chiadog:inhouse_docker-26c7b8

@martomi PTAL, all feedback incorporated and everything simplified quite a bit.


Please provide usb to uart example

I'd like to be able to use this for USB serial and an example would be very helpful.  Thank you

it's buried here: https://github.com/bouffalolab/bouffalo_sdk/tree/master/examples/peripherals/usbdev/usbd_cdc_acm


Fix context lost event leak

https://forum.babylonjs.com/t/unable-to-create-vao-on-context-lost-with-active-frustum-culling/48968/22?u=sebavan

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15165/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15165/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15165/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15165/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15165/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15165/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15165/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15165/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15165/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15165/merge/testResults/webgl2playwright/index.html

nice one ;)


139 on newick strings

Error message Process finished with exit code 139 (interrupted by signal 11: SIGSEGV) after running it directly in python after building with make. Worked fine on the example input files from the repo.
from tree_io import read_newick
from tree_structs import findpath_distance

newick_str = "(((7:0.06868257819,(20:0.02964868806,21:0.02964868806):0.03903389013):0.1042569489,(((((29:0.01270612141,30:0.01270612141):0.01771552468,19:0.0304216461):0.02550493264,11:0.05592657874):0.06617456767,((12:0.05472055727,13:0.05472055727):0.02421792724,(9:0.05724582078,10:0.05724582078):0.02169266373):0.0431626619):0.04780352892,(((24:0.02550858298,(27:0.02441077399,28:0.02441077399):0.001097808993):0.04980295134,4:0.07531153432):0.09038758802,2:0.1656991223):0.004205552989):0.003034851755):0.003578587408,((((((25:0.02482626361,26:0.02482626361):0.02554515754,((15:0.03685974834,16:0.03685974834):0.009891218212,14:0.04675096655):0.0036204546):0.06595014748,(5:0.07084572098,6:0.07084572098):0.04547584766):0.01201396307,(22:0.02694130675,23:0.02694130675):0.101394225):0.03309748355,(3:0.08619590656,((17:0.03682821325,18:0.03682821325):0.02661757797,8:0.06344579122):0.02275011533):0.07523710869):0.012108986,1:0.1735420013):0.002976113241):0;"

newick_tree = read_newick(newick_str)
newick_tree2 = read_newick(newick_str)

print(newick_str)

d = findpath_distance(newick_tree, newick_tree2)
print(d)

The ete3 newick parser does not appear to have an issue with this, maybe we want to switch to that parser instead @lenacoll ? See the package (_converter and time_trees)where this is already implemented so maybe its just copy and paste.

I updated dct_parser, which fixed the problem (it has already been addressed in that repo). See commit 1b2aa0218ec3a6aee906b64a2dbd6945234e6957.
@Lars-B Since we do not use exactly the same tree data structure as ete3, I'm not sure if it's necessary to use that package, it feels more like an unnecessary extra step to me.


Update resources.txt

wait but why

wait but why

Because I was sure that it is a duplication. I will check it few hours later.

fixed that a hour ago
https://github.com/abagayev/stop-russia/commit/18d041f3dc396eb2ce090d0060583001f2cfd4b8


Request to add grouping information to the expression functions in the adaptive-expressions package

When interacting with a field that supports adaptive expression functions in Composer, we would like to be able to show the list of available functions when initially placing the cursor on the field, such as this:

The functions would be grouped by category (eg: String, Collection, etc..), such as specified in the following documentation:
https://docs.microsoft.com/en-us/azure/bot-service/adaptive-expressions/adaptive-expressions-prebuilt-functions?view=azure-bot-service-4.0
The request would be to add this grouping information in the adaptive-expressions package for consumption in Composer.

Will look into how language server protocol can provide multi-level suggestions based category. I'd like to suggest this in the Composer Language server side rather than in the Adaptive-Expressions.

@darrenj please prioritize and add the customer impact.

@mrivera-ms i'm not sure how this could be technically implemented, what we offer is a library containing a list of expressions, and we don't have a mechanism to expression function type as a library.  This should be a Composer issue, which use a table to store the grouping.

@cosmicshuai, Intellisense currently has a completionListOverride functionality to inject any custom component in the Intellisense menu and override the default behavior. This logic is client side. The purpose of this functionality is to handle any non traditional Intellisense interaction (aka any UI that is different from a simple list of completion results). It is currently used to show "Start typing type or write an expression" when you place your cursor on an empty field accepting an expression. I was planning to use this mechanism to show the nested menu displayed in the shared design.
@cosmicshuai, @boydc2014, regardless of how this data is consumed, my goal with this request is to have a single source of truth for this grouping logic. It is technically feasible to hardcode this in Composer but I want to avoid that route. For example, I see that the functions and groupings under: https://docs.microsoft.com/en-us/azure/bot-service/adaptive-expressions/adaptive-expressions-prebuilt-functions?view=azure-bot-service-4.0 change somewhat frequently. Those changes should be in sync with whatever information we surface in Composer. It could be through the adaptive expression package or anything else as long as we maintain single source of truth.

This seems like a prime candidate to be expressed in .uischema.  Why isn't that the right place to have this?

@chrimc62, this would be about grouping the functions of the adaptive expressions language (eg: add, max, min -> Math, length, split -> String, etc..). My understanding is that the uiSchema helps with customizing the ui for the different components already present in the main sdkSchema ("Microsoft.Ask", "Microsoft.AttachmentInput", etc.. ). Could you explain a bit more how you think grouping functions in the uiSchema would work?


⚠ Bofrost.de has degraded performance

In 03f2bef, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1507 ms

Resolved: Bofrost.de performance has improved in d689fc5.


InvalidOperationException: "The type System.Data.Entity.DynamicProxies.x is not known to the DbContext."

While trying to rebuild a graph, my entity provided a relation to a lazy-loaded proxy.
In the EntityManager.cs file, there are multiple references of getting the type from an object. My solution was adding the ObjectContext.GetObjectType() function on the type provided (or the type of the object provided).
For example on line 75:
public IEnumerable<PropertyInfo> GetPrimaryKeyFieldsFor(Type entityType)
{
    var metadata = ObjectContext.MetadataWorkspace
            .GetItems<EntityType>(DataSpace.OSpace)
            .SingleOrDefault(p => p.FullName == ObjectContext.GetObjectType(entityType).FullName);
    if (metadata == null)
    {
        throw new InvalidOperationException(String.Format("The type {0} is not known to the DbContext.", entityType.FullName));
    }

    return metadata.KeyMembers
        .Select(k => entityType.GetProperty(k.Name, BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public))
        .ToList();
}

I had the same error.
In order to make it working, I made the choice to completely disable the use of proxies with EF:
protected override void OnModelCreating(DbModelBuilder modelBuilder)
{
   this.Configuration.ProxyCreationEnabled = false;
}

I work with disconnected entities (ASP.Net MVC) so proxies are not useful for me.

Hello all,
The fix for this issue has been released in v3.1.1,
If you believe we still have the issue, just let us known.
Best Regards,
Jon


these scripts are unmaintained

https://github.com/github/git-lfs/issues/985#issuecomment-181522174

@andyneff Can I get a :+1: ?

:+1:

On Mar 3, 2016, at 12:28 PM, risk danger olson<EMAIL_ADDRESS>wrote:@andyneff Can I get a  ?—Reply to this email directly or view it on GitHub.


Creating Readme

We will create readme files in every directory to make it easier for people to understand the content.

Almost done!
Now try to complete #65


Handle deleted & non-existent workspaces

Coverage increased (+0.02%) to 97.896% when pulling 78b325b9bd382a092c944b5ae55abf2c8bec24b6 on MrCreosote:master into 81749857e0a7aa8d20c0eefc4a93ae35990909d2 on kbaseIncubator:master.


Add Audit Table and Trigger function

Hook business, contact, location to audit on update/delete.
Required adding an updatedBy userstamp on each of the tables/models.  Allows us to see who actually changed the records in the audit record.
Add a wholesale PUT/Update function similar to the IPC POST, only update specific fields in location for now (start date, end date).
Based on Audit Trigger PostgreSQL Wiki.

Description

Types of changes

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

Included in #103


TypeError: Cannot read property 'nOffset' of undefined

The directive is in a modal, the second time I open the modal, is when the error occurs.
The error occurs if: display-utc = "true"
TypeError: Cannot read property 'nOffset' of undefined at link (http://localhost:9000/bower_components/angular-timezone-selector/dist/angular-timezone-selector.js:119:40)
Version: 1.2.2

Has been fixed in version 1.2.4.
See #9

Thank you!


Question: Which is the best approach to update stream when new records are inserted

I am facing a situation in a StreamBuilder that holds a List of Objects from sqlite. That List is a Stream from floor DAO.  What I am trying to achieve is something like is made in firebase. The firebase emit new values to stream when they are inserted, in case of sqlite the values comes from a SELECT. My first idea was create a trigger that notify the bloc about the new record inserted and then just sink.add it to the stream<List>.  However something tell me that I am reinventing the wheel, I'll appreciate if someone could share some experience about it.

Currently, streams are already triggered by dao methods which @insert, delete or update. But those streams contain the whole updated table, not just the changed records, so if I understand you correctly then you only want to see what changed, right?(let's call it a diff).
This should be achievable by making a middleman which recieves the stream update from floor, compares it to the previous version, puts this diff into its output and saves the current stream output for the next update. I don't have code for this ready but it should be doable. Alternatively you could wrap your DAO and take the inserted entities from the source.

Currently, streams are already triggered by dao methods which @insert, delete or update. But those streams contain the whole updated table, not just the changed records, so if I understand you correctly then you only want to see what changed, right?(let's call it a diff).
That sounds very good @mqus (streams are already triggered by dao)! I don't have reached the point where my middleware will sync the sqlite database with the REST API. May the streams from "floor DAO" will  do all the work for me. I still haven't tested if the whole lists of stream will not impact in slow updates. I will need to test it, however I suspect even adding just one item in stream the whole list is rebuild on the screen anyway, in that case getting the whole list from dao may will not be so bad. Just the final tests will capable to answer the performance points of my question. Anyway I have appreciate your expertise.

As the question was answered, I think I can close this question for now, when I got the final results of performance analysis  I will post here, may it take a time cause it's a future task in my sprint. Thks again @mqus, I have using floor since the beginning and both the evolution and stability are making the right choice for persistence layer. Thks!


It seems that the API call to obtain websites is broken

When I went to use Puter Blogger today, it now hangs, and when looking at the Network console, it shows that I got a 401: Unauthorized from the api endpoint.  I checked this repo to see if anything was added for auth in here, and I couldn't find anything with that sort of change. I should open a new issue, but Puter Docs is also down, which I was going to use as a potential resource to see if any API got changed overnight.  The blogger app did work yesterday.
If you need anymore information from me, please reach out and let me know what I should provide, or if an API did change, let me know that and close the issue.  Many thanks.

Hello, I'm investigating this issue now. It seems that the authorization token is no longer being send. I'm going to move this issue to the main puter repo and we'll track it there.


intensity is too strong

The stimulator appears to be overstepping the calibration threshold when administrating stimuli.
The calculations should be reworked to ensure that they only overstep by a fixed amount, for example 10%. Furthermore, considerations should be made about the need to factoring down the amplitude in accordance with how many pads are being stimulated per finger.

Updated to now use the correct amplitude values and added a modifier for adjusting.


Why setActiveTabColor works on at most three tabs?

Hello!
Could you explain this limitation?

Because when using more than three tabs, the entire BottomBar background is non-white, so the all tabs icon and text colors are white.

Why BottonBar background can't be white with four tabs? Can't find this restriction on bottom navigation spec page.

I re-read the spec, seems you're right.
I'll make the limitation optional in the next versions.

Yep, This hasn't been changed as of yet and I looking forward to be able to have white background with four tabs as well. Or be able to choose when to activate the color change.
Thanks! Keep up the good work! :)

Now partially (you lose the shifting mode benefits) possible in version 1.3.2.
// Show all titles all the time.
mBottomBar.useFixedMode();
mBottomBar.setActiveTabColor("#FF0000");

This and much more customization options are now possible in the newest version (2.0 and up)!


masquerade and more async methods

Changes:

Added masquerading mock classes as motor's original, so checks like isinstance(collection, AsyncIOMotorCollection) won't fail. It's possible there will still be issues if you expected to use tornado classes instead of asyncio, but I will wait for actual cases to somehow solve this.
Added some async methods for collection.
Added test that checks basic compatibility with beanie ODM.

Notes:

Inspired by #1.

Hey again! Here is my example code base that prompted the previous PR if you want to test against this. If you look at the init_mongo function you can see I'm passing in an arg and a kwarg. If I remove those, your branch les me pass unit tests 100%. Could we also add something to consume excess *args and **kwargs when initializing the client?
https://github.com/tclasen/fastapiplayground

And also, a HUGE thank you for even making this library. Unit testing async mongo has been such a pain for the longest time. Despite your repo only being a few days old, it is the most complete / working implementation on the market right now.

Pretty much all the credit goes to mongomock, they did most of the job 😄. Nice to see that I'm not the only one who needed a viable mock for async mongo too. I added *args, **kwargs to AsyncMongoMockClient's __init__. This should fix mocking with AsyncMongoMockClient.

I just confirmed your newest commit passes all tests in my repo using Beanie.


Moved SymfonySentinelSession out of the Provider namespace

This PR moves the class SymfonySentinelSession out of the provider namespace as it is not really a service provider. Plus, when switching to Symfony, that class would be the only one left in that namespace.
Related to #618.

Thanks @derrabus


add DisableEmptyText attribute

add DisableEmptyText attribute fix when AllowPaging = true && LoadData.HasDelegate && Count == 0 DataGrid InLine Editing not working
https://github.com/radzenhq/radzen-blazor/blob/master/Radzen.Blazor/RadzenDataGrid.razor#L275
because DataGrid InLine Editing template radzenGrid.InsertRow  must render DrawRows(visibleColumns) fontend,

Hi @pigwing,
Maybe ShowEmptyMessage (with true as default value) will be better name since there is EmptyTemplate and EmptyText properties. Please add the property also to the RadzenDataGrid component only not in PagedDataBoundComponent.


Transfer new logic from Jira feed to converters

Description

Provide description here

Type of change

[ ] Bug fix
[x] New feature
[ ] Breaking change

Related issues

Fix #1

Migration notes

Describe migration notes if any

Extra info

Add any additional information

lgtm
I would like @willmarks to also have a look, since he was the original owner of this code


Fix unstarted StartGG bracket leading to random DQs

Fixes https://github.com/joaorb64/TournamentStreamHelper/issues/301

Approved


Assert for memory allocation failures

If the aging buffers queue is emptied, ensure_has_free_space() may return without ensuring space_to_free is available.
Metal buffers have a max length and may return None on allocation failures. Error immediately if the max is exceeded or None is returned.

Not catching these can cause hard to track downstream issues.

Otherwise thank you for fixing this! I had stable diffusion just be wrong


PEAR tmp fix breaks the builds of Alpine-based images

https://github.com/RobLoach/docker-composer/blob/3ae582213877cb13e576160b42c004ff536da88e/base/alpine/Dockerfile#L32-L35
Example of a failed build: #112

php7-pear is in the "community" repo, not "testing": http://pkgs.alpinelinux.org/package/edge/community/x86_64/php7-pear


Looks like the domain is not working.

was fiddling with the server, demo site should be back up now!


downcase username before saving a bot

Closes #214
Downcase username before saving. This ensures username and slugs are equal.
Line edited in bot's profile: passing @ bot makes Rails look for the correct attribute used on find_bot. This solves the issue on edit page but downcasing usernames seems to be the best idea.

Closes #221


Backed Documentation

Proposal
Write Clear Explanation of all API in server/APIDOCS.md
For reference, You can see here
ps:- If you work on this issue and stuck it at any point you can contact me on Discord :)
Do you want to work on it? - [No]

interested, pls assign


[BEAM-7274] Infer a Beam Schema from a protocol buffer class.

This utility infers a Beam schema from a protocol buffer type. LogicalType is used for types that not natively represented in Beam schemas.

R: @alexvanboxel

Run Python PreCommit

Friendly ping. @alexvanboxel do you have any thoughts on this PR?

After a good night sleep I can understand a schema specific encoding for Row: if you disconnect it from the element implementation you leave room for element specific optimizations (example: process elements in bundles with Arrow). But we need to realize it adds another row encoding that the other languages need to implement.

You are correct, that this requires other language to implement this parsing as well. However I think the visibility advantage of having a fully-represented proto (v.s. just embedding a bytes field in a proto) is worth that tax - and it shouldn't be a huge tax on Beam SDKs (it only took me about 30-40 minutes to write the code here)

@alexvanboxel let me know if you have more thoughts here or if this looks good.
One more comment - once your options work is in, we should switch my use of field metadata over to the structured options approach.

@alexvanboxel let me know if you have more thoughts here or if this looks good.
One more comment - once your options work is in, we should switch my use of field metadata over to the structured options approach.

I've done a second pass, to verify use of logicaltypes and schema. I've added a note in my todo list for the serializeLogicalType boolean for later so we don't forget.
LGTM


Small corrections to episode 1

Hi, thanks for creating the tutorial. I am following along.
A few comments/corrections:

You might like to mention that Docker has to be pre-installed before starting the tutorial. Otherwise you get this:

track:~$ minikube start
😄  minikube v1.15.1 on Darwin 11.0.1
👎  Unable to pick a default driver. Here is what was considered, in preference order:
▪ virtualbox: Not installed: unable to find VBoxManage in $PATH
▪ vmware: Not installed: exec: "docker-machine-driver-vmware": executable file not found in $PATH
▪ vmwarefusion: Not installed: vmrun path check: exec: "vmrun": executable file not found in $PATH
▪ docker: Not installed: exec: "docker": executable file not found in $PATH
▪ hyperkit: Not installed: exec: "hyperkit": executable file not found in $PATH
▪ parallels: Not installed: exec: "prlctl": executable file not found in $PATH
▪ podman: Not installed: exec: "podman": executable file not found in $PATH
❌  Exiting due to DRV_NOT_DETECTED: No possible driver was detected. Try specifying --driver, or see https://minikube.sigs.k8s.io/docs/start/

"minikube halt" -> "minikube stop"

track:~$ minikube halt
Error: unknown command "halt" for "minikube"

Missing final “.” from “docker build -t geerlingguy/kube101:intro”

Thanks,
Frank.

Not stale, just my reaction time is stale...

Gah!

Wait what? I put the planned label on it!


Respect ON_THE_RECORD / OFF_THE RECORD

respects the otr status and shows message when it changes (i.e. recognizes the event)
closes #675

I just tested this out and it seems to properly respect the history being on/off.
One small bug I noticed was that when the history was turned back on, the message I saw was "History has been turned off". It should have said "on".

Having a more permanent notification (icon, changing the background color of the messages like Google does, or both) would be a very welcomed feature in the future.

corrected the off/on :) it was a typo
Do you want to take a crack at styling the "off the record"? :)

waiting for #866 ))


(BOLT-146)(BOLT-153) Update bolt-server schemas

The file-protocol, 'smb-portoptions have been added to winrm transport. Theinterpreters` option has been added to both winrm and ssh. This commit allows those configuration options in the {winrm,ssh}-run_task schemas.

Verification of BOLT-153:
Modified server_client.rb to add the smb config (and print request body) and verified task is uploaded as expected.
cas@cas-ThinkPad-T460p:~/working_dir/bolt$ bundle exec scripts/server_client.rb sample::ps_noop winrm://Administrator@klw97c5l1fgeqap.delivery.puppetlabs.net{"task":{"metadata":{"description":"Powershell task with noop","supports_noop":true},"name":"sample::ps_noop","files":[{"filename":"ps_noop.ps1","sha256":"1bac11e4e612da7e4308d0e94455b6a3e67ea9f000e7a757c4d5b6de5deb765f","size_bytes":58,"uri":{"path":"/puppet/v3/file_content/tasks/sample/ps_noop.ps1","params":{"environment":"production"}}}]},"target":{"hostname":"klw97c5l1fgeqap.delivery.puppetlabs.net","user":"Administrator","ssl":false,"port":5985,"file-protocol":"smb","password":"*****","smb-port":445},"parameters":{}}
{
  "node": "klw97c5l1fgeqap.delivery.puppetlabs.net",
  "status": "success",
  "result": {
    "_output": " with noop \r\n"
  }
}

Then added a bogus smb port to confirm it would not work. Got a net timeout
cas@cas-ThinkPad-T460p:~/working_dir/bolt$ bundle exec scripts/server_client.rb sample::ps_noop<EMAIL_ADDRESS>{"task":{"metadata":{"description":"Powershell task with noop","supports_noop":true},"name":"sample::ps_noop","files":[{"filename":"ps_noop.ps1","sha256":"1bac11e4e612da7e4308d0e94455b6a3e67ea9f000e7a757c4d5b6de5deb765f","size_bytes":58,"uri":{"path":"/puppet/v3/file_content/tasks/sample/ps_noop.ps1","params":{"environment":"production"}}}]},"target":{"hostname":"klw97c5l1fgeqap.delivery.puppetlabs.net","user":"Administrator","ssl":false,"port":5985,"file-protocol":"smb","password":"***","smb-port":666},"parameters":{}}
bundler: failed to load command: scripts/server_client.rb (scripts/server_client.rb)
Net::ReadTimeout: Net::ReadTimeout
  /home/cas/.rbenv/versions/2.5.1/lib/ruby/2.5.0/net/protocol.rb:181:in `rbuf_fill'

Verification for interpreters in winrm-run_task:
Idea is to set a bogus interpreter and show that it is added to extensions whitelist and try to run task with bogus interpreter:
cas@cas-ThinkPad-T460p:~/working_dir/bolt$ bundle exec scripts/server_client.rb sample::echo<EMAIL_ADDRESS>'{"message": "hey"}'
{"task":{"metadata":{},"name":"sample::echo","files":[{"filename":"echo.sh","sha256":"c5abefbdecee006bd65ef6f625e73f0ebdd1ef3f1b8802f22a1b9644a516ce40","size_bytes":64,"uri":{"path":"/puppet/v3/file_content/tasks/sample/echo.sh","params":{"environment":"production"}}}]},"target":{"hostname":"klw97c5l1fgeqap.delivery.puppetlabs.net","user":"Administrator","ssl":false,"port":5985,"password":"****","interpreters":{".sh":"foo"}},"parameters":{"message":"hey"}}
{
  "node": "klw97c5l1fgeqap.delivery.puppetlabs.net",
  "status": "failure",
  "result": {
    "_output": "",
    "_error": {
      "kind": "puppetlabs.tasks/task-error",
      "issue_code": "TASK_ERROR",
      "msg": "The task failed with exit code 1:\nThe term 'foo' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.\nAt line:4 char:8\r\n+ '@ | & foo 'C:\\Users\\Administrator\\AppData\\Local\\Temp\\0mvem1af.y34\\ec ...\r\n+        ~~~\n    + CategoryInfo          : ObjectNotFound: (foo:String) [], CommandNotFoundException\n    + FullyQualifiedErrorId : CommandNotFoundException\n",
      "details": {
        "exit_code": 1
      }
    }
  }
}

Same strategy for ssh-run_task
cas@cas-ThinkPad-T460p:~/working_dir/bolt$ bundle exec scripts/server_client.rb sample::echo ssh://bolt:bolt@<IP_ADDRESS>:20022 '{"message": "hey"}'
{"task":{"metadata":{},"name":"sample::echo","files":[{"filename":"echo.sh","sha256":"c5abefbdecee006bd65ef6f625e73f0ebdd1ef3f1b8802f22a1b9644a516ce40","size_bytes":64,"uri":{"path":"/puppet/v3/file_content/tasks/sample/echo.sh","params":{"environment":"production"}}}]},"target":{"hostname":"<IP_ADDRESS>","user":"bolt","password":"bolt","port":20022,"host-key-check":false,"interpreters":{".sh":"foo"}},"parameters":{"message":"hey"}}
{
  "node": "<IP_ADDRESS>",
  "status": "failure",
  "result": {
    "_output": "",
    "_error": {
      "kind": "puppetlabs.tasks/task-error",
      "issue_code": "TASK_ERROR",
      "msg": "The task failed with exit code 127:\nsh: 1: foo: not found\n",
      "details": {
        "exit_code": 127
      }
    }
  }
}

CLA signed by all contributors.


Refactor FlatList Component

Issue Description
We have components in the code base that use FlatList. Many of these components depend on external data.
Right now we conditionally render a message that tells the user there is no data available or if the app ran into an error while trying to fetch data. (An example of this is the Notifications tab). In some cases we do not even render a message, Yikes!
We can reduce the number of conditionals checks by using FlatList's argument named ListEmptyComponent.
ListEmptyComponent: Rendered when the list is empty. Can be a React Component Class, a render function, or a rendered element.
Here is a list of files that may need to be updated:
campus-mobile/app/views/common/ScrollCard.js
campus-mobile/app/views/dining/DiningListView.js (not in prod yet)
campus-mobile/app/views/dining/DiningListViewAll.js (not in prod yet)
campus-mobile/app/views/messaging/Messaging.js
campus-mobile/app/views/schedule/FinalsCard.js
campus-mobile/app/views/shuttle/ShuttleOverviewList.js
campus-mobile/app/views/shuttle/ShuttleSmallList.js
campus-mobile/app/views/weather/SurfReport.js

This issue is being closed due to platform migration and may be revisited at a later date.


HHH-12670 - Allows native SQL queries that take a given result Class to map the result set to the required type

https://hibernate.atlassian.net/browse/HHH-12670

@sebersole If you have time, please take a look at this issue too. I think it's worth implementing this mechanism for native SQL queries.

@vladmihalcea maybe I misunderstood the change but aren't you changing how the results of a query are organized? That would break compatibility for existing applications, wouldn't it?

@gsmet Without my change, you get an Exception. With my code, it works. It would break compatibility only if previously passing a non-entity class to a query actually worked.

@vladmihalcea ah ok, that makes sense :).


update libs highfive comment to mention new ACP process

This PR removes the libs-api review rotation set and edits the highfive comment for libs PRs to remove the reference to r? rust-lang/libs-api and to add links to the new API Change Proposal process.

Holding off on merging this until we make sure the new proposal process is working (i.e., integration with triagebot)

this should be good to go since we're no longer using triagebot, I updated our docs to no longer mention it


docs: linkTo

The linkTo docs are incomplete, references https://github.com/DefinitelyTyped/DefinitelyTyped/pull/19499

It's the exact same as above - it was introduced in the PR mentioned in the
README, but never formally written down. Basically a callback-based link.
On 5 Sep. 2017 06:28, "Norbert de Langen"<EMAIL_ADDRESS>wrote:

@ndelangen commented on this pull request.
In addons/links/README.md
https://github.com/storybooks/storybook/pull/1793#discussion_r136868096:

@@ -36,12 +36,14 @@ Have a look at the linkTo function:
import { linkTo } from '@storybook/addon-links'

linkTo('Toggle', 'off')
+linkTo(() => 'Toggle', () => 'off')
I do not understand this example, what does this do?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/storybooks/storybook/pull/1793#pullrequestreview-60465654,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALeho9Id6CYQy-kaVoEVBdN-9pivgWiks5sfF1VgaJpZM4PLxpr
.

Interesting, TIL we have callback based linkTo 👍


added information about contribution and npm commands to readme file

PR Title
[DOC-145 Update readme file]
This Pull Request was created to update information about getting started to contribute and some show some npm-commands
The purpose of this Pull Request is to fix https://github.com/jsvigneshkanna/tailwind_ui_components/issues/145
Description
With this PR the contributors could find all information directly in README.mb how to install project locally.
How you solved
Added requested information from ISSUE into readme file.

Great @Yordaniss
These changes are important this time as we are getting more contributors than expected
Thank you!


[Spark-4848] Stand-alone cluster: Allow differences between workers with multiple instances

This refixes #3699 with the latest code.
This fixes SPARK-4848
I've changed the stand-alone cluster scripts to allow different workers to have different numbers of instances, with both port and web-ui port following allong appropriately.
I did this by moving the loop over instances from start-slaves and stop-slaves (on the master) to start-slave and stop-slave (on the worker).
Wile I was at it, I changed SPARK_WORKER_PORT to work the same way as SPARK_WORKER_WEBUI_PORT, since the new methods work fine for both.

I'm not sure how mesos and yarn clusters are started/stopped (nor do I have such clusters on which to test), so I'm not sure how this will affect them. I think the way I did this should be safe - it's mostly just moving code around - but I could use a knowledgeable set of eyes to be sure.

Test build #29005 has started for   PR 5140 at commit d739640.

This patch merges cleanly.

Test build #29005 has finished for   PR 5140 at commit d739640.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29005/
Test PASSed.

@nkronenfeld thanks for working on this. The changes look pretty reasonable and most of my comments are minor. Would you mind updating the title of this PR to say something like "[SPARK-4848] Allow different Worker configurations in standalone cluster" instead? I think it's slightly clearer.

Sure, I'll try to take care of all of these tonight.

Test build #30004 has started for   PR 5140 at commit 044ca6f.

This patch does not merge cleanly.

Test build #30004 has finished for   PR 5140 at commit 044ca6f.

This patch passes all tests.

This patch does not merge cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30004/
Test PASSed.

Ok, I think this is all set.
Out of curiosity, are you Andre Wor, or Andrew Or?

Test build #30035 has started for   PR 5140 at commit cf5f47e.

Test build #30035 has finished for   PR 5140 at commit cf5f47e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30035/
Test PASSed.

@andrewor14 - is there anything else that needs to get done here?

Ok LGTM I'm merging this into master thanks.

and to answer your question I am Andrew, not Andre :)


Rename NodeFactory variables from context to factory

Half NodeFactory variables were named "context" and the other half were
named "factory". Since "factory" makes more sense, rename all existing
variable names to reduce confusion with TypeSystemContext variables
which also use "context".

LGTM

LGTM

@dotnet-bot Please test Windows_NT Debug


Implement IF EXISTS clause for DROP [BLOB] TABLE

Coverage increased (+0.03%) to 81.49% when pulling 399fc2d24279f050e7b156799ab5d1ec7259d878 on wallner:drop_table_if_exists into 7acf5671e931422834919180b9d38726fd7b70fc on crate:master.

ack


[FEATURE] Item Repairer with essence and energy

How will this work, exactly? Will each work cycle restore 1 durability, and you’ll just have extremely quick cycles? Or will it restore multiple points at once?