LibreOffice <IP_ADDRESS> & <IP_ADDRESS> *.desktop files have incorrect binary names in Exec command

Describe the bug
The current unstable versions of libreoffice-still and libreoffice-fresh are unable to be launched via their *.desktop files as found in the derivation's share/applications/ directory.
This means that for me, Rofi can no longer launch LibreOffice Writer etc.
Steps To Reproduce
Steps to reproduce the behavior:

Update unstable channel
Add libreoffice, libreoffice-still or libreoffice-fresh as a package to be installed
nixos-rebuild switch
Try and run Writer or any other LibreOffice app from a desktop launcher

Expected behavior
Should be able launch LibreOffice applications from a desktop launcher, e.g. Rofi.
Screenshots

Additional context
The desktop files are expecting libreoffice7.3 as the binary name, but it's libreoffice:

It looks like in https://github.com/NixOS/nixpkgs/pull/196535 the renaming of binaries in *.desktop was removed.
The following was taken out for some reason...
    for f in $out/share/applications/*.desktop; do
      substituteInPlace "$f" \
        --replace "Exec=libreofficedev${major}.${minor}" "Exec=libreoffice" \
        --replace "Exec=libreoffice${major}.${minor}"    "Exec=libreoffice"
    done

Notify maintainers

@gador
Metadata

system: "x86_64-linux"
host os: Linux 6.0.2, NixOS, 22.11 (Raccoon), 22.11pre418904.db25c4da285
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.11.0
channels(root): "nixos, nixos2205-22.05"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Workaround until the fix hits a channel:
{
  environment.systemPackages = [
    (pkgs.resholve.writeScriptBin "libreoffice7.4"
    {
      interpreter = lib.getExe pkgs.bash;
      inputs = [ pkgs.libreoffice-qt ];
      execer = [ "cannot:${pkgs.libreoffice-qt}/bin/libreoffice" ];
    }
    ''
      set -eEuo pipefail

      exec libreoffice "$@"
    '')
  ];
}

You can of course also use runCommand or one of the many other script helpers to do this.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/libreoffice-not-found/22781/2


Updated facebook.js to german

Added German support.

You're welcome to have a German fork, but I'm not going to replace this English version with German :)

Oh dear, sorry! I'll fork it!

Haha, no problem. I assumed it was a mistake :)


[ui] design review changes  - default/primary btn colors

subtle rounded corner for buttons? try 2px.
For the top naviation bar, use a 4 pixel underline to highlight which tab is active
use airpal blue for link color
use blue from airpal as primary btn color

plz review @mistercrunch @bkyryliuk @elibrumbaugh

Can we keep the default button dark on light background? (black on light-gray?)
On the explore view we should use btn-primary only on the query button I think. Side note, the timer should become a label left of the button group.
Getting subjective here: what about a darker blue? Close to Github's blue maybe?

i think i prefer the airpal blue over the github blue. github blue looks too much like FB blue, and like that we are tying the airpal primary color over to caravel. it's nice and bright, and works well as a highlight color.

here is the explore page with dark default buttons and brighter primary for the query button.

https://github.com/airbnb/caravel/pull/1010

can we finish discussing the changes in this pr and make the timer changes another PR?

i'm not sure what you mean by blank btn-default @mistercrunch

feedback from @elibrumbaugh

let's use the brighter airpal blue
let's nudge the stroke on hover on the nav bar up into the white bg part of the nav bar

blank btn-default :

I don't like the black bg buttons

made changes based on feedback. thanks for the review @mistercrunch @elibrumbaugh!


Custom date formats

[x] Read new formats
[x] Write new formats
[x] Read linked dates
[x] Write linked dates
[ ] Add more formats requested in #296 (and possibly #500)
[ ] Tests
[ ] Reduce code duplication
[ ] Documentation
~Pull in date from dailyNotes plugin (see comment #296)~

Like the addition of the links for dates!

Will do, thank you for the idea! There’s lots of duplication of many things at the moment 🙈

Shout if you want any assistance.

Thank you for the offer! Regarding the "signifiers", I just realized they will move to the settings soon, anyway. (See https://forum.obsidian.md/t/task-management-devs-define-a-shared-date-and-action-format/)
I am always happy about contributions. If you want to help, check the list of open issues and let me know if anything is interesting to you 😊

Have done so, determined the issue with the description filter and the change to strip dates out of description in the task object.

@schemar is this still in the works?  For my use case, I use tag dates (see https://forum.obsidian.md/t/using-dates-as-tags-does-it-make-sense/26024) and it would be great if I could specify a date format of #YYYY/MM/DD so it would insert the tag when I complete a task

@schemar Push for this too, in my location we have a different format and it confuses the hell out of me.

Closing as too outdated ...

=/ very bad for me, the most waited feature. Why? Because i wan't to make date format to match my date-tags format like: #date/10-10-2022.

I don’t have time anymore to work on this. You could try implementing it.

Closing as too outdated ...

Understood @schemar.
I know the code isn't mergeable due to the passage of time, but the ideas in it will still be useful whenever this topic is picked up. Thank you for your work on it.

updates? this is the #1 most requested feature of many people 😕

updates? this is the #1 most requested feature of many people 😕

That #1 link confused me there, as it's not relevant to this...

updates? this is the #1 most requested feature of many people 😕

Hi @Creative-Difficulty Not sure where you got that from...
Here are the top-voted features that I have not already implemented and released:
is:open category:"Ideas: Any New Feature Requests go in Issues please" sort:top -label:"status: released"
Currently, it looks like this:

And indeed, the top one is being worked on:

https://github.com/obsidian-tasks-group/obsidian-tasks/discussions/60

It's a lot of work, so unknown when it will be released...


The data is too much, a timeout will be displayed.

When I test with 10,000 data entries, a timeout occurs; however, testing with just a few entries works perfectly fine.

https://swiftpackageindex.com/apple/live-caller-id-lookup-example/main/documentation/pirservice/testinginstructions#Processing-the-datasets has some information on processing the datasets. In particular, the Parameter Tuning documentation will be helpful to achieve best performance.
You can try using more shards, e.g. via decreasing entryCountPerShard or, equivalently, increasing shardCount.


msvcrt: add _EH_prolog

used info from here: https://github.com/MiroKaku/ucxxrt/blob/master/src/i386/ehprolog.asm#L36-L45
closes #167

relies on read/write ptr from #162


Don't fail to build the agent locally when system probe is disabled

What does this PR do?
Disable system probe build throughout various locations to allow building the agent when --system-probe-bin isn't provided
Motivation
Simplify building the agent locally
https://datadoghq.atlassian.net/browse/BARX-622
Describe how to test/QA your changes
I'm not sure this is worth doing QA, but it can be verified manually by attempting to build the agent on main of before that commit is merged and confirm that if fails, and then with this commit and confirm that it doesn't
Possible Drawbacks / Trade-offs
Additional Notes

/gitlab estimate-cost

/merge


Error caused by setUntil() --> Only certain dates work, others do not

When using setUntil(), only a few dates work, others return this error:
lxml.etree.ParserError: Document is empty
This is my code:
max_tweets = 20
tweetCriteria = got3.manager.TweetCriteria().setUntil("2017-02-02").setQuerySearch("bitcoin").setMaxTweets(max_tweets).setLang(Lang="en")
for i in range(max_tweets):
    tweet = got3.manager.TweetManager.getTweets(tweetCriteria)[i]
    print(tweet.text)
    print(tweet.id)
    print(tweet.username)
    print(tweet.date)

Here is a little list of dates which work or not:
2017-02-05 error
2017-02-04 error
2017-02-03 ok
2017-02-02 error
2017-02-01 ok
2017-01-31 error
2017-01-30 ok
2017-01-29 ok
2017-01-28 error
2017-01-26 error
2017-01-25 ok
Also when I change the max_tweets to > 20 I get the same error!
Anybody an idea what happens here? Many thanks in advance.

Hi, can you check on default got's folder (phython 2.7)? I don't give a expressive support for python 3+ and I a few improvements were done in the default implementation.


Fix incorrect line numbers in jest tests

Description
Use ganache-core fork which bypasses the import of source-map-support/register during tests.
See:

https://github.com/trufflesuite/ganache-core/issues/522
https://github.com/kulshekhar/ts-jest/issues/727
https://github.com/celo-org/ganache-core/commit/3dbc32b214b3bb8f0dda2bfd909850a752a73a55

Tested
Failed jest tests now reports correct line numbers for failed expectations.
Other changes
N/A
Related issues

Fixes #2961

Backwards compatibility
Yes.

Some additional info:
I noticed the the failing expectations showing incorrect line numbers in the mobile project started to show correct line numbers since we deleted walletkit.
I tried on the blockchain-api project and got incorrect line numbers without this fix and removing the mock on contractkit (so ganache-core ends up being indirectly imported).
To sum up, this fix is needed whenever ganache-core ends up being imported during a jest test.

I believe that the 0x/subproviders in the protocol/package.json is not used anymore.
But other 0x packages that we use, end up retrieving the 0x/subproviders. We should probably make an issue to clean old unused dependencies, and see which ones could be avoided if we just extract some code.
But this PR looks ok, nice catch!
LGTM


Add configuration for NCSA Teststand environment

You need to add either gafaelfawr or authnz for authentication. Currently the configuration you have assumes authnz, so you'll want to include that chart and a new values file for the authnz service directory. (I can then move it to gafaelfawr later.)

Think this is the necessary fields.


Add Conan support

I'm packaging all my libraries as Conan packages so I avoid compiler flag hell. Can you add this project as "os" for Conan so I can just recompile all my code for standalone without going back to compiler flag hell?

Huh... interesting idea. Let me play with this and I will get back.


stop logging to /config volume?

Hi, thanks for the useful container. Right now, it's set up to log to both stdout and to /config/domoticz.log. I'd like to just use docker logging infrastructure, so I've modified my entrypoint so that it no longer logs to /config:
ENTRYPOINT ["/src/domoticz/domoticz", "-dbase", "/config/domoticz.db"]

Is that something you're interested in merging in?

Hello, thanks for the input. When I did this part, it was one of my first docker image so it's far from perfect :)
Ok, send me a pull request and I'll merge it !
thank you!

Cool, done in https://github.com/sdesbure/docker_domoticz/pull/5


Make footer link color a11y compliant (more contrast)

Purpose
Make footer link color a11y compliant (more contrast)

Coverage remained the same at 56.936% when pulling 67c675b81cac54f122f269052f4da90707d4cb1f on hmoco:dev into e597c73588a415b475d8b89931794c5ec43c7bb3 on CenterForOpenScience:develop.


indexer-cli includes ANSI color codes in YAML output

I wanted to backup my rules and cost models, so I did the following:
$ graph indexer rules get all --output yaml > rules.yaml
$ graph indexer cost get all --output yaml > costmodels.yaml

Looking at the contents of these files, they contain ANSI color codes (which confusingly don't display in the terminal when heading the file since it just interprets them).
rules.yaml:
- deployment: global
  allocationAmount: "50000.0"
  parallelAllocations: "4"
  maxAllocationPercentage: null
  minSignal: "130.0"
  maxSignal: null
  minStake: null
  minAverageQueryFees: null
  custom: null
  decisionBasis: rules

...

costmodels.yaml:
- deployment: QmR2uzq3Tt3htYTmefFwJ6eX2aYySY6ZRZkW9LZG39Byrh
  model: default => $CONST + $DAI * $MULT / $DEV;
  variables:
    DAI: "32.9955383622343452"
    DEV: "100"
    MULT: 1
    CONST: "0.5"

...

Hi Chris, ANSI color codes seemed to be added in by the CLI toolkit (gluegun). Error messages should be in red if you try out a wrong command, but otherwise the default is white.
I will go ahead and add some more ~colors~ for printing, and can add a print command that strips ansi if the command is desirable?


feat: updated unsupported browser page to be a static HTML page

Summary | Résumé
closes https://github.com/cds-snc/platform-forms-client/issues/2081

Updated existing unsupported browser page so that it now is a static HTML page
Middleware will always show that new page to any user on Internet Explorer. IE version or accessed page will not have any influence on whether the page should or should not be presented
Added GoC logo as PNG to be accepted by old Internet Explorer (SVG was not supported before IE 10)
Simplified (avoid including any Javascript) unsupported browser page to make sure we get the same result on all Internet Explorer versions

Here is the unsupported browser page in Internet Explorer 8 in a Windows 7 virtual machine:

Note: here is what we had before:

Everything you see that is not in the first screenshot was removed because it either required Javascript or would redirect to pages requiring Javascript
Test instructions | Instructions pour tester la modification

Open Internet Explorer
(If you really cannot use Internet Explorer you can change line 7 of middleware.ts by if(true))
Load any page of our application. You should be presented with the unsupported browser page

Pull Request Checklist
Please complete the following items in the checklist before you request a review:

[ ] Have you completely tested the functionality of change introduced in this PR? Is the PR solving the problem it's meant to solve within the scope of the related issue?
[ ] The PR does not introduce any new issues such as failed tests, console warnings or new bugs.
[ ] If this PR adds a package have you ensured its licensed correctly and does not add additional security issues?
[ ] Is the code clean, readable and maintainable? Is it easy to understand and comprehend.
[ ] Does your code have adequate comprehensible comments? Do new functions have docstrings?
[ ] Have you modified the change log and updated any relevant documentation?
[ ] Is there adequate test coverage? Both unit tests and end-to-end tests where applicable?
[ ] If your PR is touching any UI is it accessible? Have you tested it with a screen reader? Have you tested it with automated testing tools such as axe?

@samsadasivan Let me know if the refactoring still works for you in terms of design (see PR description with comments). I decided to avoid using any Javascript to make sure we could have the same result on all Internet Explorer versions.


copy path and copy value buttons not working

Current Behavior
the copy path and copy value buttons in the debug tab are not working anymore.
Going back to version 2.1.4 makes the buttons work again.
Expected Behavior
copy the path or value from the debug tab to clipboard
Steps To Reproduce
Click the copy path or copy value button on any message in the debug tab
Example flow
No response
Environment

Node-RED version: 2.1.5
Node.js version: 14.18.3
npm version: 6.14.15
Platform/OS: Debian Bullseye
Browser: Firefox 95.0.2 and Microsoft Edge 97.0.1072.55

#3331 introduced this regression when using http to access the editor (and not using localhost)

Same issue in Chrome for Mac Version 97.0.4692.71 (Official Build) (x86_64) and Safari Version 15.1 (176<IP_ADDRESS>.20)

Same issue for me. I just updated to v2.1.5 and I cannot copy anymore the path and values. Buttons are present but are not copy. Any solution?

We don't need any more "me too" comments in this issue. We know exactly what is wrong and will release a fix next week.
If you read the pr I linked to, you'll see it happens if you are not using https or localhost to access the editor.

Fixed in 2.1.6 via #3363

@knolleary Thank you for fixing it. I checked that v2.1.6 works correctly on other PC (not localhost) via HTTP.


AndroidX migration

The support library artifacts are being deprecated and all future development is going into AndroidX, so there’s no point to avoiding this migration.

It seems like current AndroidPdfViewer library doesn't support for an app migrated to androidX. Is it right?

@katie0809 yeah, you are right. Here is a fork with AndroidX https://github.com/TalbotGooday/AndroidPdfViewer
 implementation 'com.github.TalbotGooday:AndroidPdfViewer:3.1.0-beta.2'

@TalbotGooday Thank you, actually I solved problems by adding maven { url 'https://maven.google.com' } to the build.gradle. And I don't know why...='(

@katie0809 so your problem was not the "migration problem")

👍 look good to me :) @barteksc any chance to get this merged and forge a release?

@TalbotGooday as there seems to be no update in this place, have you consider to continue your fork?

@madmas it all depends on what desires the community will have) Feel free to create issues in my fork)

@TalbotGooday thanks for the quick response :)
actually, you don't have issues enabled:

@madmas oh, sorry. I will enable this option asap)


why using test data for retrieval

Hi authors, thanks for your work!
I have a question regarding the use of test data during retrieval.
Could you kindly clarify why test data is also included for retrieval? I thought test data should primarily be reserved for evaluation. Shouldn't we be retrieving only from training videos?
https://github.com/YuanJianhao508/RAG-Driver/blob/24a0bd01f56c6fa4f2563a4c94925856e2ba707e/retrieval/BDDX_RAG_tuned_vpmatch_t13.json#L8
Thanks for your time and assistance!

Hi @javyduck  thanks for raising the question! When training the retriever, we combined both the training and test datasets to train the projector. However, during the retrieval and training VLM, we do not we the test data. Thank you very much!


Can't remove a binary installed to the relative path

I installed a binary specifying only the binary name like this:
bin install https://github.com/mglaman/drupalorg-cli dcli

It put the binary to the current directory, instead of the default ~/.local/bin - that's probably okay.
But then I can't remove this binary using the bin remove command - it produces no errors, but the binary remains in the list of installed, here is the output:
$ bin

 Path                                                   Version         URL                                                     Status
 /home/alexey_korepov/.local/bin/bin                    v0.17.2         github.com/marcosnils/bin                               OK
 /home/alexey_korepov/.local/bin/drupalorg.phar         0.5.1           https://github.com/mglaman/drupalorg-cli                OK
 dcli                                                   0.5.1           https://github.com/mglaman/drupalorg-cli                OK

$ bin remove dcli
$ bin remove /home/alexey_korepov/dcli

$ bin

 Path                                                   Version         URL                                                     Status
 /home/alexey_korepov/.local/bin/bin                    v0.17.2         github.com/marcosnils/bin                               OK
 /home/alexey_korepov/.local/bin/drupalorg.phar         0.5.1           https://github.com/mglaman/drupalorg-cli                OK
 dcli                                                   0.5.1           https://github.com/mglaman/drupalorg-cli                OK

By the way, maybe treat a simple string (just a string without slashes) as a binary name, not as a path, so put it into the default directory, instead of the current one? And for the current directory recommend passing ./dcli. What do you think about this?

I'm not so familiar with Golang, but will try when will I get over the fear of trying :)

I think the detection of the path is it relative or just a name - should be done somewhere here:
https://github.com/marcosnils/bin/blob/f7265c40a7b245a698a7eee603234ece8e1c7c86/cmd/install.go#L44
The regexp check via ^[^/]+$ and if it's match - concat the config.Get().DefaultPath + / to the string start. Right?

:wave:  apologies the delay. just released a new version with the fix.


Data Service Editor does not support the same set for "SQL type" as Management Console when input parameter mapping is edited.

Data Service Editor does not support the same set for "SQL type" as Management Console when input parameter mapping is edited.
https://wso2.org/jira/browse/DEVTOOLEI-812

Fixed with the new DSS editor.


Catalan Translation. Revised and corrected some errors detected.

Revised and corrected some errors detected.

Thanks Santiago :+1:


Problem with serializing empty list of objects to JSON String.

Hi!
First of all I just want to say how much i like this library :) Reminds me of Google Gson.
But I have a problem with serialization of empty list of objects. Serializing empty list does not create it in json. Here's my snippet code.
var searchConditions: SearchConditionsDTO = SearchConditionsDTO()
searchConditions.documentClassKeyDTO = keyDTO
searchConditions.searchLimit = 20
searchConditions.conditions = SearchConditionDTO
let jsonString = Mapper().toJSONString(searchConditions, prettyPrint: true)!
println(jsonString)
This is what i get:
{
"searchLimit" : 20,
"documentClassKeyDTO" : {
"id" :<PHONE_NUMBER>23123,
"kind" : "DocumentClass"
}
}
This is what want to achieve:
{
"searchLimit" : 20,
"documentClassKeyDTO" : {
"id" :<PHONE_NUMBER>23123,
"kind" : "DocumentClass"
},
"conditions" : [ ]
}
Is there a way to do it?
Rest api i connect to accepts empty list, but not null value :/

Hey there,
Until today this wasn't possible. I have just added updated the library to support this. If you pull the latest from master you can access the changes.
Cheers!


Leaving transition does not work properly when using JavaScript hooks.

Version
3.0.5
Reproduction link
https://codepen.io/ustbhuangyi-the-bold/pen/MWboJav
Steps to reproduce

Click the Toggle button, wait for the enter transition end.
Then click the Toggle button again.

What is expected?
The leaving transition works well and after the transition end, the  red square disappear.
What is actually happening?
The square disappear immediately, the leaving transition does not work properly.

This will be fixed in https://github.com/vuejs/vue-next/pull/3230, as a work, you can use v-if instead of v-show.

Another workaround, you can remove the changes to the state, see https://codepen.io/hcysunyang/pen/wvoeOaa

Another workaround, you can remove the changes to the state, see https://codepen.io/hcysunyang/pen/wvoeOaa

Haha, that's the reason why it looks strange. But I do need to do this


About causal mask and decoder

The part of self-attention in the TargetQueryDecoderLayer is removed and the causal mask is modified (not been used).
It is to remove the connection between the four coordinates, and does it have a significant impact on the results?

Yes. The experiments told that it is better to make four coordinate trajectories independent with each other. The idea of auto-regression is not adopted in this work.

Thanks.
I am curious about how many performance differences there may be.

Not much for accuracy. Even smaller than one point. But more efficient without auto-regression.

ok, thanks


Moves DEFAULT_PREFENCES into JSON format.

Makes it easier to address #7192 , e.g. PDFJSDev.json('file.json') will be inlining loaded object.

/botio-windows preview

r=me, with the comment addressed.

Object.assign is not present in IE -- I'll just add object cloning function for that.


Posix x86_64 calling conventions fix

Depends on https://github.com/dlang/dmd/pull/9013
Without this, phobos unittests fail on https://github.com/dlang/dmd/pull/9013

Since dlang/dmd#9013 was closed, I am also going to close this.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


storm:"index" storm:"unique" is ignoring the unique constraint

If I have a struct field defined as:
SubURL    string storm:"unique"
Then I can safely try to create the same entry in the DB multiple times, knowing that only the first one will success.
But if I define it as:
SubURL          string storm:"index" storm:"unique"
Then "unique" is ignored and I get duplicate entries in the DB for the same contents.
Ah, just as I was writing this I tried
SubURL    string storm:"unique" storm:"index"
And that worked. So the order of the tags seems to be important. Is this a feature or a bug?

You are not supposed to use the same tag multiple times in go i believe, one of them will get ignored.
So in general you would do something like this:
`json:"blabla,omitempty"`

instead of
`json:"blabla" json:"omitempty"`

As of Storm, only one value for the storm tag is currently supported so it is either storm:"unique" or storm:"index".
The difference between unique and index is only that the latter allows multiple ids for the same value. unique IS an index but has an additional unicity constraint, so you don't need to write both.

Ah got it, thanks!


Cumbersome to pass curly brackets to field converters

At the time of this writing, it is cumbersome to instantiate converters with any parameter containing curly brackets in its expression.
For instance, the following attempt to validate color to be one of "red", "green" or "blue"
import falcon
import falcon.routing

class In(falcon.routing.BaseConverter):
    def __init__(self, values):
        self._values = frozenset(values)

    def convert(self, value):
        if value not in self._values:
            return None
        return value

class ColorResource:
    def on_get(self, req, resp, color):
        resp.media = {'color': color, 'path': req.path}

app = falcon.App()
app.router_options.converters['in'] = In

app.add_route('/colors/{color:in({"red", "green", "blue"})}', ColorResource())

... fails with the following ValueError: Field names must be valid identifiers (""red", "green", "blue"" is not valid).
One could easily work around the problem in this contrived example by using a list or tuple instead of the set literal, but what if we need curly brackets in a regex, or need to pass a dict?

Perhaps we could support escaping curly brackets by doubling them ({{ & }}), just as the Python format string syntax does?

Perhaps we could support escaping curly brackets by doubling them ({{ & }}), just as the Python format string syntax does?

I like that idea, since it would be somewhat natural to python users to escape them that way. Especially since f-string has become a thing

(It's the same {{ & }} even in str.format(), which dates back to early 3.x versions and 2.7.)

Double Braces {{ feel like bloating the route string/parameter definition.
Looking into where this parseing happens i think it would be best, keeping multiple parametercases in mind, to split parsing into
two steps:

Extract every parameter definition from the route segment
→ Every grouping of single { like {object_id:int} in the segment obj_{object_id:int}_img
Extract the parameters specification.

This should cover at least all python native, non-recursive (not Object of Object of ...), structures.

@chgad thanks for the input, I remember our discussion at the EuroPython2024 Sprint :)
Just checking you had any code prototypes left in this direction, if you wrote any back then?


Missing dependencies

For whatever reason, redux-offline isn't installed when I added this to my project...
(Truncated) yarn list output:
...
├─<EMAIL_ADDRESS>│  ├─<EMAIL_ADDRESS>│  ├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>│  ├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>...

Hey @justinmakaila,
please take a look at the README's "Install" section.
The following packages are peer dependencies:

apollo-client (or any distribution including it, e.g. react-apollo)
redux
redux-offline

They will not be installed automatically; adding them manually is required. This is a design decision that aims to give you more control over which specific versions of these you want to use in your project.
I hope this clears things up for you.
If you want to discuss this decision or the way it is communicated in the README, feel free to share your thoughts!

Got it, thanks!


text-decoration: underline missing in export

Steps to reproduce:

Created all of my text styles, the links are underlined.

Used plugin to export the styles to text-styles.json file

Opened a new Sketch file, imported text-styles.json file and this is what is imported:

Hey Jenny, this should not be the case. I'll be working on a fix soon!

@nilshoenson Thanks! 🙌

This is now fixed! You can grab the update.

Thank you!! ⚡


Missing color on method call after version 0.4.4

after version 0.4.4:

before version 0.4.4:

The color on dark mode is alright. This is a bug or a intended change?

It is intended (See #49).


check: make (Service)Address() return an IP address

change the Address method to return an IP address when available (i.e. when ClusterIP is not empty, which happens in case of headless services) instead of just returning the service name.
towards: https://github.com/cilium/cilium/issues/24151

Always returning an IP (beside the clusterIP == "" case) causes the PodToService() and PodToHostPort() scenarios to fail, specifically the flow validation:
2023-05-23T13:17:40.1564008Z   ℹ  SYN and(ip(src=<IP_ADDRESS>,dst=<IP_ADDRESS>),tcp(dstPort=8080),tcpflags(syn)) not found
2023-05-23T13:17:40.1569614Z   ❌ Aborting flow matching: context deadline exceeded
2023-05-23T13:17:40.1570235Z   ❌ Flow validation failed for peer test-namespace/client-64c655df64-mb6pd: 2 failures (first: -1, last: -1, matched: 0)

as it's not clear yet what's causing that, and I'm not able to reproduce this locally, for the meantime I switched the first condition to:
	if s.Service.Spec.ClusterIP == "" || family == IPFamilyAny {
		return s.Service.Name
	}

so that we keep the current behavior of returning the service name when the IP family is set to any


Prevents program from exiting

Noticed a peculiar issue in that any library that I included winston-papertrail in would never actually exit. Here is how to reproduce:
Put the following in test.js:
  var winston = require('winston');

  //
  // Requiring `winston-papertrail` will expose
  // `winston.transports.Papertrail`
  //
  require('winston-papertrail').Papertrail;

  var logger = new winston.Logger({
    transports: [
        new winston.transports.Papertrail({
            host: 'logs.papertrailapp.com',
            port: 12345
        })
    ]
  });

  logger.info('this is my message2');

Execute it with node test.js and the program will never exit. It will correctly log your message to Papertrail.
This behavior doesn't exist with, say, just the default Console logger.
This bug is reproduceable on 0.1.1 and 0.1.2, using winston 0.6.2. Also reproducable using 0.1.2 and the latest winston 0.7.1. I'm using node 0.8.14.

This works for me, but I get an ugly 'socket hang up' exception in the console. Any suggestions on how to resolve this?
Thanks for the library by the way, very handy :).


Keep track of number of remounts in details.txt

Display the number of remounts as a value in details.txt. This allows test scripts to determine if a remount actually occurred.

@mbed-bot: TEST

@mbed-bot: TEST

@mbed-bot: TEST


Why no FromRawFd?

Why there is no std::os::unix::io::FromRawFd impl?
How do I make tokio_file_unix::File from a fd number without workarounds like going though temporary socket/file/whatever struct?

Currently you can do this via:
// either
tokio_file_unix::File::new_nb(std::fs::File::from_raw_fd(fd))
// or
tokio_file_unix::File::raw_new(std::fs::File::from_raw_fd(fd))

To implement FromRawFd for  tokio_file_unix::File, a choice would have to be made between these two, and I'm not sure which one makes a good default.

Now I understand.
This (like "To implement FromRawFd (from_raw_fd), a choice would have to be made between new_nb and raw_new behaviour, and as there is no reasonable default this impl is not provided. Use std::fs::File as a proxy.") should probably be in the documentation, so searching for words "FromRawFd" and "from_raw_fd" finds something.

Hm, actually…  raw_new is the only sensible choice for impl FromRawFd for tokio_file_unix::File, because new_nb returns Result.  Still, it's a risky one and I'd say it's better to encourage explicit use of raw_new.
Another complication comes from the choice of F: the ownership semantics of tokio_file_unix::File<F> are determined entirely by the underlying F object.  If you choose for example F = std::fs::File, you are implicitly asking for the file descriptor to be closed during destruction, which may or may not be intended behavior.
I'll add some explanation to the docs.


v0.9.11 Combobox Selecteditem binding not worked

XAML -

VIEWMODEL -
[Reactive]
public Sport SelectedSport { get; set; }
    [Reactive]
    public List<Sport> Sports { get; set; }

public class Sport
{
    public int Id { get; set; }

    public string Name { get; set; }

}

Can you provide MCVE? Asking because I think code that you posted is not full enough to check the issue.


Namespace issue when using kemalyst-validators & kemalyst-model

Both have a variable errors. If you include kemalyst-validators,
errors << validator[:message]

fails, because errors is defined as an Array(String)? in kemalyst-model.

thanks for reporting this.  I am still working on the integration with kemalyst-model.

This should now be fixed with the latest release v0.2.2 of kemalyst-model and kemalyst.


fix a few issues with the new linux stuff, get windows running again

Ran into a couple of issues on Windows, previously reported here:
https://github.com/Jigsaw-Code/outline-client/issues/435
This seems to work around it, and a few random tidy-ups.

@AurangZ FYI


Date Range for Components Mode

Ionic version:  (check one with "x")
[ ] 2.x
[x] 3.x
[ ] 4.x
Ion2-calendar mode:  (check one with "x")
[x ] components mode
[ ] modal mode
I'm submitting a ...  (check one with "x")
[ ] bug report
[x ] feature request
[ ] help me
Current behavior:

Default Date Range available only in modal mode
Expected behavior:

How to set Default Date Range  in components mode. I have from and to date. I need to show these dates ranges as selected while page load
Steps to reproduce:

Related code:
insert short code snippets here

Other information:
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
insert the output from ionic info here

This is what I have:
`
dateRange: {
from: Date;
to: Date
};
openCalendar() {
const options: CalendarModalOptions = {
pickMode: 'range',
defaultDateRange: this.dateRange,
title: 'Start/End'
};
    let myCalendar = this.modalCtrl.create(CalendarModal, {
      options: options
    });

    myCalendar.present();

    myCalendar.onDidDismiss((date: { from: CalendarResult; to: CalendarResult }, type: string) => {
        if (type === 'done') {
            this.dateRange = Object.assign({}, {
              from: date.from.dateObj,
              to: date.to.dateObj,
            })
          }
    });
}`

you can easily set the from and to dates in the dateRange object

Thanks for your response. But your example is a modal mode.  I want to set date range in components mode. There is no defaultDateRange parameter in CalendarComponentOptions.

same issue, any solution?

I have solved this by,
<ion-calendar [(ngModel)]="dateRange"  [options]="optionsRange"
(onSelect)="onSelect($event)"   [type]="type"  [format]="'YYYY-MM-DD'">
dateRange: { from: string; to: string; }; //--> Here set your start and end date
optionsRange: CalendarComponentOptions = {
pickMode: 'range',
showMonthPicker:false,
from : new Date("2018/01/01"),
color:'dark',
};

@sabi-Equadriga thank you, solved!


codecov readme updates

[Current coverage][cc-pull] is 92.75%

Merging [#25][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #25   diff @@
==========================================
  Files            12         12
  Lines           276        276
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            256        256
  Misses           16         16
  Partials          4          4

[][cc-pull]

Powered by Codecov. Last updated by 59452ce
[cc-base-branch]: https://codecov.io/gh/codingsince1985/geo-golang/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/codingsince1985/geo-golang/pull/25?src=pr


delegate debug_traceTransaction in forking mode

Component
Anvil
Describe the feature you would like
Anvil currently returns "Requested data unavailable" when trying to call `debug_traceTransaction' on an forked anvil instance. It should however be delegated to the fork, same as in https://github.com/foundry-rs/foundry/pull/2832
Additional context
No response

the debug_traceTransaction  is already delegated to the fork if we're unable to find it
doe your provider support this call?

my provider is alchemy, so yes

testing debugTraceTransaction using web3py against the same RPC worked, but when done with an anvil fork it's data unavailable

Fork it's data unavailable

this should be fixed via https://github.com/foundry-rs/foundry/pull/7229


[Request] compatibility with the v60 feature to adjust weight based on interior

v60 added a feature that allows entities to be flagged as spawning more often on a certain interior, it would be nice if we could customize this

The issue with this would be that the enemy spawn weights would have to be changed after the interior is selected, which afaik is after certain mods(like event based ones), so this would probably overwrite those somewhat. It is however a planned feature, I am just not sure yet what is the best way to implement it.


Read only mem tests

Coverage decreased (-0.006%) to 98.452% when pulling a9f01d9f1cb381cc8c46e256b3860189008fb0e4 on read-only-mem-tests into deee1d3b4e9d95e518abe9e4ee170e38e522e552 on master.


KBinsDiscretizer Aggregate Encoder (Helper + Mode) #19081

Reference Issues/PRs
References #19081.
What does this implement/fix? Explain your changes.
This extends the functionality of KBinsDiscretizer to encode matrices with aggregate functions such as min, max, mean, and mode.
I created a higher order helper function that utilizes ordinal encoding to bin data into discrete intervals, and simply requires a lambda function like min or max to be passed in.
Example

output

@RylandWang Can you also attach the screenshot of the test result, because the pipeline doesn't run for our repo.

There are so many lines changed. I couldn't tell the difference. Did you change the format or something?

Sorry, it was a line break issue when I switched from Linux to Windows.


Modifiers

First of all: thank you for this library. It seems to be a fresh air among trending overcomplicated query builders/orm libraries out there. So far, I really like the namings, the powerful typescript bindings, and the overall code quality (including so many tests) looks promising to me.
Just started to use, but what I noticed: I found no way to add specific modifiers to the builtin queries. For eg.: I would need to add a STRAIGHT_JOIN to a select statement. So far I've seen only the 'bottom' modifiers related to locking (forUpdate, etc).
Is there any option to add 'top' modifiers (like STRAIGHT_JOIN, SQL_NO_CACHE, etc), maybe even dialect-specific ones?
Doesn't matter if could add only as raw SQL and not though dedicated methods, just the option is missing to add custom ones (and all dialects has quite a few specific ones which could be required in some corner cases).

🤔 but then we couldn't have custom modifiers. Kysely wouldn't know where to put them. Maybe the method could accept a second argument when raw sql is given that specifies top or bottom.
Another problem with the selectModifier or modifySelect method is that it's not easy to find.

An extremely hacky way to add these modifiers now is to use raw sql:
db.selectFrom('person')
  .select([sql`SQL_NO_CACHE first_name`, 'last_name', 'age'])

Thanks for your quick response.
Typescript complains about the hacky way, I could only solve it by casting to 'any':
.select([sql`SQL_NO_CACHE` as any, 'first_name', ....]
Of course not the best way, but at least works!
The best would be to have better control over these modifiers. My first thoughts were about a modifier(type: T) method, where:

T: 'for update' | 'for share' | ... as you mentioned too

And could have an overload method with modifier(type: 'custom', name: string, position: P), where:

name (or maybe sql as we're talking about raw string?): name of the custom, specific modifier
P: 'top' | 'bottom' => where to place it

Btw: removing all modifier-related methods maybe not necessary, the frequently used ones (distinct, forUpdate) could deserve a comfortable access with 'shortcut' methods, imho.
This way, the most common modifiers (that most of the SQL servers support) could be 'built in', but with the 'custom' option we could add any dialect specific ones too. And of course select/insert/update/delete commands have their own modifiers, but those should not be a problem as those are different query builders, could have different T options.
Any thoughts about it, could work?

Typescript complains about the hacky way, I could only solve it by casting to 'any':

Oh, that's right. Kysely needs to know the name of the selection. You need to use something like this
// Notice the first selection is inside the raw expression
.select([sql`SQL_NO_CACHE first_name`.as('first_name'), 'last_name', ....]

Whenever you pass raw SQL somewhere in kysely, it has to be done using the sql template tag. That way we don't concatenate input strings to the SQL without an explicit permission from the user. The method could look something like this
modifier(modifier: 'for update' | 'for share' | 'sql_no_cache')
modifier(modifier: RawBuilder<any>, position: 'top' | 'bottom')

You'd use it like this:
.modifier('for update')

Or if you pass in a raw snippet, it will require the second argument
.modifier(sql`my_custom_mod`, 'top')

Sorry for not working on this. I've been trying to prioritise things that benefit the most people. We need some way to do this, so I'll reopen this.

https://github.com/koskimas/kysely/releases/tag/0.19.9


Can you add login and saving bookmarks stuff, I will buy domain and server for u to host it so that u can add those features

Instead of login server, we'll probably use firebase and FB/TW login.
More at #60


How to get multiselect refresh displayed options when changed externally

I must be missing something trivial. I'm changing the array assigned to the options. The array itself gets changed.
When I try search, the labels are updated afterwards.However, I can't figure out how to make the multiselect show the new labels as soon as I make changes to the options list variable.

How are you changing the labels? Please remember there are several caveats regarding mutating arrays. https://vuejs.org/v2/guide/list.html#Caveats

The setup is: a parent component with a vuetable and a child with a modal window consisting of a couple of multiselects.
The modal is unhidden by clicking a cell in a vuetable. The click should update the content of the modal and show the modal itself
It seems that the multiselects in the modal window render before the props from the parent are updated and show with default values.
However, it happens only the very first time I reload the whole page.... and I'm debugging with a remote database / in a docker / ... i.e., everything is pretty slow.
Maybe, the modal window needs some kind of pre-render?

writing things down sometimes help :) calling hide() method on the modal window when the parent is created seems to have solved the issue!
Thanks for the link above, it was quite useful on its own as well.


response 'error' events aren't handled by .end()

We noticed this cryptic error coming from zlib:
events.js:141
      throw er; // Unhandled 'error' event
      ^

Error: incorrect data check
    at Zlib._handle.onerror (zlib.js:363:17)

If zlib encounters an error while decompressing a stream, an error event is emitted on an internal stream object:
https://github.com/visionmedia/superagent/blob/master/lib/node/utils.js#L86
The response stream's .on method is then hooked up to the internal stream object:
https://github.com/visionmedia/superagent/blob/master/lib/node/utils.js#L113
So in order to handle errors on the internal stream object, we have to listen for 'error' on the response stream.
It seems like the only access to the response stream prior to stream end is inside of Request.prototype.end - the response will have ended by the time we reach our callback.
All this is to say, I think if zlib encounters an error decompressing a data chunk, there's no way for us to handle it, because it happens before the response 'end' event. Do you think this analysis is correct?

@wonnage:
We're seeing something similar:
events.js:141
      throw er; // Unhandled 'error' event
      ^

Error: unexpected end of file
    at Zlib._handle.onerror (zlib.js:363:17)

Any luck in resolving this?
This crashes our node-process entirely without an apparent way to catch??

Possibly related to #410

They are now, if you use promises


Add other sensors to REST api

Is your feature request related to a problem? Please describe.
I don't have a MQTT server currently but Home Assistant provides the rest sensors which could be used to obtain the sensor data.
However when calling the REST api on http://<IP_ADDRESS>:2971/api/state only the currentUrl, screenOn and brightness values are returned, however when sensors are enabled additional data such as Battery level, Temperature is provided to MQTT.
Describe the solution you'd like
If sensors are enabled the REST API would also return the sensor data.
Option I see would be to add the additional sensors to the following in WallPanelService.kt
    private val state: JSONObject
        get() {
            Timber.d("getState")
            val state = JSONObject()
            try {
                state.put(MqttUtils.STATE_CURRENT_URL, appLaunchUrl)
                state.put(MqttUtils.STATE_SCREEN_ON, isScreenOn)
                state.put(MqttUtils.STATE_BRIGHTNESS, screenBrightness)
            } catch (e: JSONException) {
                e.printStackTrace()
            }
            return state
        }

Would also be nice to have motion and face detection in here (maybe clearing the flag on next call or after a timeout period) as while this wouldn't be as fast it would save on needing to implement an outbound rest/service call.
PS. Basic example for Home Assistant using the currently available values in the REST API
sensor:
  - platform: rest
    name: Front Door tablet screen
    resource: http://<IP_ADDRESS>:2971/api/state
    value_template: '{{ value_json.screenOn }}'
    json_attributes:
      - brightness
      - currentUrl

camera:
  - platform: mjpeg
    name: Front Door tablet
    mjpeg_url: http://<IP_ADDRESS>:2971/camera/stream

We have a PR out now for getting sensors with discovery in HA.  I am closing this feature.

We have a PR out now for getting sensors with discovery in HA.  I am closing this feature.


Draft of new API for discussion

Here is a draft API for discussion. I think it is fairly idiomatic Rust, and would be easy to use in Nanobot.
The code does not compile, but short of that I tried to include as much information as possible. Although I created a separate api.rs file, I would expect this code to be part of lib.rs.
Some points for discussion:

In most languages I avoid "Object Oriented" APIs, but I think what I'm proposing is idiomatic for Rust. The alternative is passing a config struct as the first argument for the functions I'm defining.
VALVE is fundamentally about mutating the database, but I think that the Valve struct that I'm proposing could usually be created as immutable, because the configuration data does not change. The main exception is the connect() method. The Rust Book talks about the interior mutability pattern which could be used in this case, I guess. Or build() could take two arguments: table table path and database connection string.
If the struct is immutable, then I would make the configuration fields public, for reading.
The error types I sketched are probably not complete.

I have added the stubs from this PR in this commit which has been pushed to the branch new-api-forked-from-guess which is being tracked by PR #70.
This PR (#68) can now be closed unless further discussion is required.


[BEAM-2188]  Cherry-pick #3024

R: @kennknowles
cc: @davorbonaci

Confirmed that diffs are identical. LGTM upon green.

Changes Unknown when pulling 9a3f9ea023eab19cab26ce20fc505baef0fc574a on aaltay:rc8 into ** on apache:release-2.0.0**.

Tests passed except for an HBase flake. merging.

Merged, close at your leisure :)

Thank you.


Sourcetracker2 runs for 1m30s then all processes switch to 'Sleep' and hangs

Hello,
Issue
I just tried running Sourcetracker2, as installed via instructions on the README (anaconda then pip install).
When I run the program with the --jobs parameter, and monitor via htop, the program runs for 1m30s and then stops, essentially hanging. A bunch of sourcetracker processes remain sitting with the 'sleep' status, and sourcetracker2 runs indefinitely (i.e. never completes - I left it for 24 hours in one case).
Description
To describe what I did:

Get the github repo to use the tiny-test data git clone https://github.com/biota/sourcetracker2.git
Installed Anaconda3.-5.1.0.Linux-x86_64.sh
Made the st2 env conda create -n st2 -c biocore python=3.5 numpy scipy scikit-bio biom-format h5py hdf5 seaborn
Activated the env source activate st2
Installed sourcetracker pip install sourcetracker
Changed to the tiny-test directory
Run command with no --jobs (this completes successfully) sourcetracker2 gibbs -i otu_table.biom -m map.txt -o example1/
Run command with --jobs 5, which hangs sourcetracker2 gibbs -i otu_table.biom -m map.txt -o example1/ --jobs 5

I attach two screenshots htop during the --jobs 5 run, both pre- and post 1 minute 30 seconds. Note the pre-1:30 has many more st2 related processes but I've cropped for space reasons.

I have noted that individual files are generated for each sample in the results directory (s0.txt, s1.txt, s2.txt etc.). I also tried with and without modifying the --cluster-delay-start parameter with no differences.
As a sidenote, it would be nice of some stdout and stderr messages could be added, as it would help me investigate further and be more precise in my error reporting.
Environment Information
The sourcetracker2 environment information:
Anaconda
conda environments:
base                  *  /projects1/users/fellows/bin.backup/anaconda3
st2                      /projects1/users/fellows/bin.backup/anaconda3/envs/st2
Sourcetracker2
st2_env_info.txt

Hi @jfy133 - thanks for the detailed post. I will take a look at this later tonight.

I can confirm this issue using OSX. It is related to ipyparallel not shutting down the cluster properly on this line: https://github.com/biota/sourcetracker2/blob/b81220f26ca566cd1f131cc7f17c826320a96076/sourcetracker/_cli/gibbs.py#L238
As discussed in #92 removing ipyparallel as a dependency will fix this issue

:tada: glad to see this hopefully fixed. Do you have an approximate ETA when the bioconda version will be updated?
Also, I see that the conda install instructions refers to -c biocore but as far as I can see it is under the bioconda repository. Which is the correct method to use?


phone permission being added on Android

Phone permission being added on Android as READ_PHONE_STATE.  The same behavior described below is happening with the 'storage' permission.
Your Environment

Plugin version: 2.2.5
Platform: Android
OS version: 7.1.1
Device manufacturer and model: nexus 6p
Cordova version (cordova -v): 6.4.0
Cordova platform version (cordova platform ls): 6.0.0

Context
The plugin is requesting 'phone' permission on android.  It shows up in AndroidManifest.xml as read_phone_state and is added each time during building no matter my edits to plugin.xml or other areas that I can see referencing permissions.  The problem goes away upon removing this plugin, so it must be somehow caused by it.
I'm on windows and my coworkers do not experience the same issue on their Mac with the same plugin/repo/etc.  This makes literally no sense to me so other explanations are welcome!
Expected Behavior
This should not request phone permissions, or at least should be configurable to not request them.  This does not happen after running cordova plugin rm cordova-plugin-background-geolocation
Actual Behavior
Requested phone permission.
Reproduce

Install plugin, build android.  Phone permission requested
remove plugin, build android.  phone permission not requested.

Notice: this issue has been closed because it has been reported for branch 2.x or later. It could be already fixed in v3 or is not relevant anymore. You may reopen this issue if it has been closed in error.


messageStatus should be called inboxMessageStatus

https://github.com/OpenSTFoundation/mosaic-contracts/blob/fa436b06cacf3ea84d72550b7398f94220476648/contracts/lib/MessageBus.sol#L256

This change was already done in another ticket. _messageStatus is updated to _inboxMessageStatus and _outboxMessageStatus.
https://github.com/OpenST/mosaic-contracts/blob/6714a938dba15a7eac59fa0108757f2671f77f11/contracts/lib/MessageBus.sol#L245
https://github.com/OpenST/mosaic-contracts/blob/6714a938dba15a7eac59fa0108757f2671f77f11/contracts/lib/MessageBus.sol#L359
https://github.com/OpenST/mosaic-contracts/blob/6714a938dba15a7eac59fa0108757f2671f77f11/contracts/lib/MessageBus.sol#L510

OK. Next time you can also close the issue 😀


Update MLIR-AIR

Update MLIR-AIR to main (74d1b738aabbb132cc26f9eba560dae5876e7e7c), before we start landing packet-switched routing work in MLIR-AIR and MLIR-AIE.

I don't know what's going on with the peano wheel - i've been messing around there today but it shouldn't have broken the existing wheel. i'll investigate tomorrow

I don't know what's going on with the peano wheel - i've been messing around there today but it shouldn't have broken the existing wheel. i'll investigate tomorrow

Ok. I'll keep trying today, too, to see if there's any changes.


Fix regression when using teuthology-suite with --kernel

http://tracker.ceph.com/issues/16838
First I created a reproducer, then I discovered two ways to fix the bug.
They both seem like good ideas, and appear to not cause further regressions, so I think we may want both commits.

In addition to the unit test I used:
teuthology-suite -vv -c master -m smithi -s krbd -k testing --dry-run -l 1 -n 7

-n 7 is present because master HEAD was not built at the time.


Bug found in EmitListWhisper()

Example Loop path : loop-development-kit/ldk/csharp/OliveHelpsLDK/Example/Program.cs
When attempting to run the csharp Example Loop I got this error when calling the EmitListWhisper() function provided by OliveHelps
System.ArgumentNullException: Value cannot be null. (Parameter 'value')\r\n   at Google.Protobuf.ProtoPreconditions.CheckNotNull[T](T value, String name)\r\n   at Proto.WhisperListRequest.set_Markdown(String value)\r\n   at OliveHelpsLDK.Whispers.WhisperFormBuilder.BuildRequest(WhisperList list, Session session)\r\n   at OliveHelpsLDK.Whispers.WhisperClient.ListAsync(WhisperList message, CancellationToken cancellationToken)\r\n   at AdvocatiaLoop.Loop.EmitListWhisper() in D:\\Dev\\Saiph\\AdvocatiaLoop\\AdvocatiaLoop\\Program.cs:line 512\r\n   at AdvocatiaLoop.Loop.<SearchbarStream>b__8_0() in D:\\Dev\\Saiph\\AdvocatiaLoop\\AdvocatiaLoop\\Program.cs:line 98

This is caused by the WhisperList initialization not containing a "Markdown" attribute:
private void EmitListWhisper()
  {
      var ccs = new CancellationTokenSource();
      ccs.CancelAfter(5000);
      var whisperList = new WhisperList
      {
          Config = new WhisperConfig {
              Label = "C# Whisper"
          },
          Elements =  new Dictionary<string, ListBase>
          {
              ["Nickname"] = new ListPair {Label="Nickname", Order=1, Value="Old Greg"},
              ["Link"] = new ListLink {Align=ListAlign.Center, Href="https://isitchristmas.com/", Order=2, Text="Is it Christmas?" }
          },
      };
      _services.Whisper.ListAsync(whisperList, ccs.Token);
      Logger.Info($"Sent List Whisper");
  }

It can be fixed by setting the Markdown attribute upon initialization of new WhisperList to a default value.
private void EmitListWhisper()
{
    var ccs = new CancellationTokenSource();
    ccs.CancelAfter(5000);
    var whisperList = new WhisperList
    {
        Markdown = "DEFAULT VALUE HERE",
        Config = new WhisperConfig {
            Label = "C# Whisper"
        },
        Elements =  new Dictionary<string, ListBase>
        {
            ["Nickname"] = new ListPair {Label="Nickname", Order=1, Value="Old Greg"},
            ["Link"] = new ListLink {Align=ListAlign.Center, Href="https://isitchristmas.com/", Order=2, Text="Is it Christmas?" }
        },
    };
    _services.Whisper.ListAsync(whisperList, ccs.Token);
    Logger.Info($"Sent List Whisper");
}

This does seem to be an issue with the contract for the ListWhisper in C#, as you shouldn't have to include that Markdown value. This should be addressed though when we release the next major version of the LDK, as the concept of a "ListWhisper" will disappear and be replaced by just a "Whisper", where you can mix an match components from the different types of whispers we have currently.

@BrettMugglinOlive can we confirm that this was resolved with our LDK update?

@advocatia Given all of the recent work to transition the LDK over to a JavaScript-only model, as @BrettMugglinOlive mentioned, the way that the Whisper Aptitude works has changed.
Please check out the new WhisperAptitude interface. You will be required to be on an LDK version > 3.0.0-beta.1 to take advantage of this.
We believe that this will resolve your issue, but give this a try and let us know if it's still not behaving the way that you expect.


SassError: $color: currentColor is not a color.

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
running an angular serve it comes to an error:
./src/styles.scss - Error: Module build failed (from ./node_modules/sass-loader/dist/cjs.js):
SassError: $color: currentColor is not a color.
   ╷
37 │   @return red($value), green($value), blue($value);
   │           ^^^^^^^^^^^
   ╵
  bootstrap/scss/_functions.scss 37:11          to-rgb()
  node_modules/bootstrap/scss/_root.scss 87:39  @import
  bootstrap/scss/bootstrap-reboot.scss 9:9      @import
  src/styles.scss 7:9                           root stylesheet

./src/styles.scss?ngGlobalStyle - Error: Module build failed (from ./node_modules/mini-css-extract-plugin/dist/loader.js):
HookWebpackError: Module build failed (from ./node_modules/sass-loader/dist/cjs.js):
SassError: $color: currentColor is not a color.
   ╷
37 │   @return red($value), green($value), blue($value);
   │           ^^^^^^^^^^^
   ╵
  bootstrap/scss/_functions.scss 37:11          to-rgb()
  node_modules/bootstrap/scss/_root.scss 87:39  @import
  bootstrap/scss/bootstrap-reboot.scss 9:9      @import
  src/styles.scss 7:9                           root stylesheet

Reduced test cases
Create a style.scss
and import the following files
@import "bootstrap/scss/functions";
@import "bootstrap/scss/variables";
@import "bootstrap/scss/mixins";
@import "bootstrap/scss/bootstrap-reboot";
@import "bootstrap/scss/bootstrap-grid";
@import "bootstrap/scss/bootstrap-utilities";

What operating system(s) are you seeing the problem on?
macOS
What browser(s) are you seeing the problem on?
No response
What version of Bootstrap are you using?
v5.3.0

Compiles fine for me in a quick Stackblitz, albeit without Angular. I think your issue lies elsewhere, though I'd suggest improving your import stack here.
@import "bootstrap/scss/functions";
@import "bootstrap/scss/variables";
@import "bootstrap/scss/mixins";
@import "bootstrap/scss/reboot"; // @import "bootstrap/scss/bootstrap-reboot";
@import "bootstrap/scss/grid"; // @import "bootstrap/scss/bootstrap-grid";
@import "bootstrap/scss/utilities"; // @import "bootstrap/scss/bootstrap-utilities";
@import "bootstrap/scss/utilities/api"; // might also need this


Pose decoder weights not included in checkpoint

Hi,
I tried to load depthnet and posenet, and find that, the weight pose decoder is not included in the ckpt file.
I print the keys for each ckpt(ppgeo_depth.ckpt ppgeo_pose.ckpt)

could I know how should I load the pretrained weight?
Thanks

Hi, we only share the encoder of the PoseNet so that it can be flexibly used in pose estimation networks as the pre-trained backbone.


Fix account dropdown

Fixes the account change dropdown. It never opened the account screen so I couldn't disconnect an existing wallet.

Forgot about this PR. Have done something similar today, since I've experienced a similar frustration with not being able to disconnect from the incorrect network here: https://github.com/OriginProtocol/origin-dollar/pull/844
Mind if I close this one since the other PR has some other fixes?


feat(ci): disable cache-to option to run health-check from forked branch

Description
There is currently an issue where health-check cannot be executed from a PR created in a branch of a forked repository. https://github.com/autowarefoundation/autoware/pull/4988#issuecomment-2240631197
Therefore, this PR removes the cache-to option from the health-check, allowing users without write permissions to execute the health-check.
Tests performed
https://github.com/autowarefoundation/autoware/actions/runs/10051385473?pr=5021
Effects on system behavior

Not applicable.
Interface changes

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@xmfcx This PR resolved the issue https://github.com/autowarefoundation/autoware/pull/4988#issuecomment-2240631197.
I've already checked the health-check success with a forked branch.
https://github.com/autowarefoundation/autoware/actions/runs/10051050835

And this PR's health-check was also succeeded.


Confusing mix of references and absolutes in various function calls

I see a few types (not comprehensive though) of functions defined in the library depending on their input and output
-- those with input: absolute, references, Arc
-- those with output: absolute, references, Arc
Given that, I am facing issues when using reference/Arcs outputted by some functions as arguments of others that take absolutes as inputs. Here is an example
struct Some{
_session: Arc<Session>,
...
}

fn some_new() -> Some{
  ...
  let session:Arc<session> =
          Arc::new(
              Session::connect(user_at_server, KnownHosts::Accept)
              .await.expect("Could not connect via SSH!!")
          );
  ..
  Self{
  _session: session,
...
  }

fn some_other(){
   ...
   session.clone().close();
...
}
}
....

From the above, the compiler does not likesession.clone().close() because clone() returns another Arc and Session does not implement Copy/Clone and close() takes an absolute self as argument.
Is this an already discussed case, or am I missing something really obvious :) ?

Session::close would need to take ownership, to ensure that nothing can use it afterwards.
I think you could use Arc::into_inner to convert Arc<Session> to an owned Session, then you could use Session::close
BTW, Session implements Drop, so even if you don't call Session::close it'd close it on drop, the difference is that Session::close would return an error on failure.

Firstly, thanks for the super-fast response.
I had tried into_inner, try_unwrap, etc. in my own drop i.e. Drop the Some struct which I suspect would be dropped before the inner _session is dropped -- somehow none of these work, i.e. they end up with a none/error output when unwrapping the inner session. Would really appreciate it if you can give it a shot at your end/playground. :)
line 93: Arc::<Session>::into_inner(self._session.clone()).unwrap();

thread 'cmd::test::test' panicked at src/cmd.rs:93:59:
called `Option::unwrap()` on a `None` value

Secondly, I am connecting to an openssh server on a windows box and regular drop does not shut the session. So I suspected some form of error, and hence went on this route.

So I suspected some form of error, and hence went on this route

About that, I think it's time for me to add optional tracing support to log these errors in drop.

Drop the Some struct which I suspect would be dropped before the inner _session is dropped -- somehow none of these work, i.e. they end up with a none/error output when unwrapping the inner session.

Maybe there're have some Command/Child/ChildStd* that reference the Arc<Session>?

Drop the Some struct which I suspect would be dropped before the inner _session is dropped -- somehow none of these work, i.e. they end up with a none/error output when unwrapping the inner session.

Maybe there're have some Command/Child/ChildStd* that reference the Arc<Session>?

Indeed -- that's just a command that builds of a session -- typical case
let mut cmd = session
        .clone()
        .arc_raw_command(
        program
        .display().to_string()
        );

But I tired by hashing it out -- issue remains
Anyhow, the way I see it -- Rust puts this incredible burden of being able to remember data flows statically which becomes painful at times -- as such this issue probably occurs due to the interplay between ownership and referencing.
Are the functions such as close, disconnnect, etc. purposefully built to take ownership of self? If not, can we take references instead?

Ok so -- just noticed the sessions are going into TIME/CLOSE/FIN WAIT -- so maybe the automatic drop->close is working -- but again I will make a double check later.

BTW -- does the child automatically disconnect when it goes out of scope? i.e. say I execute commands in a loop and each call execution is performed in a block of its own -- will each child be disconnected after the block ends?

Are the functions such as close, disconnnect, etc. purposefully built to take ownership of self? If not, can we take references instead?

@CowBoy4mH3LL  yes, that's definitely possible!
TBH, openssh initially supports only reference, and then we added support of Arc/Rc for certain workflow.

BTW -- does the child automatically disconnect when it goes out of scope? i.e. say I execute commands in a loop and each call execution is performed in a block of its own -- will each child be disconnected after the block ends?

Yes, the remote will kill it.


RDSv2: fix flag

Fixes --rdsv2 flag that got dropped in the last shuffle of discovery code.
Signed-off-by: Kuat Yessenov<EMAIL_ADDRESS>

@kyessenov: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
7e1342dcb5e026dfed1bf1f45331a7c41ebd6628
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/assign @ldemailly

@kyessenov: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
a13fb47e63db5305bc4043c5ee65e0d7cc33966b
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kyessenov: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
a13fb47e63db5305bc4043c5ee65e0d7cc33966b
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
a13fb47e63db5305bc4043c5ee65e0d7cc33966b
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kyessenov: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
a13fb47e63db5305bc4043c5ee65e0d7cc33966b
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
a13fb47e63db5305bc4043c5ee65e0d7cc33966b
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
a13fb47e63db5305bc4043c5ee65e0d7cc33966b
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm


USI Exploration Pack v<IP_ADDRESS>  has been released but the update is not in CKAN

We are aware of this issue but this has to be solved by RoverDude as he is now indexing his own mods.
There is a PR waiting on merge to fix this issue: https://github.com/BobPalmer/CKAN/pull/21


Bluetooth: Fix storing name in plain text

Settings consider the character space the end of the value, so instead
encode the name using settings_str_from_bytes and restore it with
settings_bytes_from_str.
Signed-off-by: Luiz Augusto von Dentz<EMAIL_ADDRESS>

Codecov Report

Merging #9208 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9208   +/-   ##
=======================================
  Coverage   52.75%   52.75%
=======================================
  Files         201      201
  Lines       25293    25293
  Branches     5286     5286
=======================================
  Hits        13344    13344
  Misses       9820     9820
  Partials     2129     2129

Impacted Files
Coverage Δ

subsys/bluetooth/host/hci_core.c
2.81% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Topic/jemalloc - build script to fetch and build jemalloc from its master repo

@densh Is there an easy way to hook up an sbt task to call the build.sh script? Otherwise I think it is fine to leave it is a manual step when wanting to build against the latest jemalloc.

Have you considered including jemalloc's github repo using git submodules? Why did you prefer to use manual fetch instead?
It's indeed possible to make sbt launch the script. I'll get them aquatinted with one another once we merge this.

@densh The following issues are ready and the build aside from JDK6 is green:

teach build infrastructure to automatically fetch and build latest jemalloc [done through a git clone / git pull instead of submodules for now]
decide if we want to support JDK6 [not supported for now]
ensure that casting between offheap.Size to size_t and offheap.Addr and void* are correct and work both on 32-bit and 64-bit platforms [offhead.Size -> size_t check is in place, not yet on the void* -> offheap.Addr]

I tried a bunch of things as evidenced by the string of broken CI runs. I ended up building the native code on an Ubuntu 12.04 image and as such the bindings should work on any client where the glibc version is >= 2.15
Thanks,
Adam

I think it's OK that we don't support JDK6, lets drop it officially.


Rename two build phases for consistency

https://github.com/MobileNativeFoundation/rules_xcodeproj/pull/2323#discussion_r1248148452


fix Readme for github documentation

Codecov Report

Merging #11 into master will decrease coverage by 0.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
- Coverage   39.27%   39.09%   -0.19%
==========================================
  Files          36       36
  Lines        3330     3330
==========================================
- Hits         1308     1302       -6
- Misses       1903     1908       +5
- Partials      119      120       +1

Impacted Files
Coverage Δ

pkg/redis/fake/redis.go
83.96% <0%> (-2.54%)
:arrow_down:

Continue to review full report at Codecov.

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


fix(relative path): last update date and by isn’t generated if localh…

…ost directory name has spaces in them

Motivation
Fixing the bug #2992
Have you read the Contributing Guidelines on pull requests?
Yes

LGTM thanks


Application.AfterDragDropOnSlide Event (PowerPoint)

The event doesn't fire, it's clearly defective and doesn't appear to have every worked since it's release.

Thanks for your contribution! This content has migrated to https://github.com/MicrosoftDocs/VBA-Docs. Please go to the new repo, and if you still see the issue occuring there, please open a brand new issue. Closing this issue now. Regards, lindalu


ChoicePrompt styles looks different from Facebook and Slack

Hi All,
How could I have the same (or similar) ChoicePrompt styles for Facebook and Slack? I've tried
Add(Inputs.Choice, new ChoicePrompt(Culture.English) { Style = ListStyle.SuggestedAction });
Facebook displays nicely as buttons, however Slack sill shows as list:

Option 1
Option 2
...
Thanks!

Thank you for your submission. The Microsoft Bot Framework team prefers that bug fixes and feature requests were opened as new issues on the official Bot Framework Github repo. The Stack Overflow botframework-v4 tag is a place to ask how to questions and get support from the community. Please feel free to submit an issue on GitHub, and the dev team will look into your request.


Revert "Report resource timestamp as metric timestamp"

Reverts konveyor/pelorus#513
resolves #650
resolves #651

/lgtm

Woot... testing w/
./demo.sh -b s2i
commits registered by the commitime-image-exporter are getting into prometheus
[whayutin@thinkdoe pelorus]$ oc logs -n pelorus prometheus-prometheus-pelorus-0 prometheus|grep num_dropped
[whayutin@thinkdoe pelorus]$


To many elements in dom

Is it possible to add the items dynamically to the dom only when needed, e.g. when the user searches or opens the searchresult? The problem is if you have a lot of ui-select elements on a single page you produce a lot of bindings /watches and it quickly slows down the performance. In our special case we have 6 ui-selects for currencies and another 6 for countries, this leads to +2500 items in the dom and slowing down the page.

this change might cause another serious performance issue,
if I have 500 items each time when i open and close the dropdown list all these 500 items will be re-rendered and destroyed again and again.....

//.attr('ng-if', '$select.open') //Prevent unnecessary watches when dropdown is closed

@edenchen123 originally we were using ng-show but that was causing that even if the dropdown is closed, to have all the watches fired. In case you only have 1 or 2 ui-select on the page it wont be an issue, but if using multiple controls with many items, then that was a big performance hit.


Syncronisation with working changes from other versions

I synced all done changes here necessarry for using this adapter.

@basti4557
Thanks very much for the extrem fast response.
Looks good to me except one question / feedback:
change at line 589 (new) from
    if (!inp) {
        //
    } else if (inp.indexOf('hdmi') == 0) {

to
if (inp.indexOf('hdmi') == 0){

could result in an exception if inp is null / undefined if I'm correct. Or can you ensure, that inp is always valid?
Wouldn't
if (inp && (inp.indexOf('hdmi') == 0)){

be more save?
Please comment or change if you think that chacking inp itself too is better.
In addition please answer my question from yesterday wether wss (encrypted) communication is available at all LGTVs. Otherwise this change could be  block some devices.
Thanks
Martin (mcm1957)

I'll fix the two minor linter issues during release build
@basti4557 THANKS VERY VERY MUCH for you support and work


NugetNinja should be opt in (!) - don't spam repos 😞

I'm sure your intentions are good, but please stop spamming repos with pull-requests, and create a process for maintainers to opt in to NugetNinja's pull-requests.
Most serious open-source projects have contributing guidelines, and the NugetNinja bot simply ignores the contribution guidelines set by the project maintainers.

Good idea. I made it manual to opt in my pull requests!.

Is opt in now.


Test PR for improving golden CI (1/2)

Test to check that CI will catch out of date golden files. See e.g. this run:

[!NOTE]
This PR reuses the state from https://github.com/IntersectMBO/cardano-cli/pull/720, before the commit adapting golden files for metadata tests.
Please ignore all commits of this PR except the last one, that adds the test that will make it to main in TODO

Test finished


Reload window and recent workspaces overlap on keybindings

On the mac reload window is Cmd +R, and recent workspaces is Ctrl + R.
However on Linux and I guess Windows they overlap.
Can we change the default reload command to be Ctrl + K, Ctrl + R. Or fix this overlap in some other way?

I've always used Ctrl+Shift+R for reload since it rarely conflicts with Refactor or quick open navigation.

Yeah we could try that as the default.

anyone opposed be me just making the swap?

Well the current default behavior is that only in development we bind ctrl+R to reload.
I would check if our Win devs would be pissed by this change.
So I propose to have Ctrl+Shift+R for all builds, not only development. As you suggested.

I actually run with this config for years and I am very happy with it, maybe we can make it the default:
macOS:
 {
    "key": "cmd+shift+r",
    "command": "workbench.action.openRecent"
}

Linux/Windows:
{
    "key": "ctrl+shift+r",
    "command": "workbench.action.openRecent"
}

Customers will most likely be used to Ctrl+R for open recent than reload for our product so I would be more worried about changing that one.

@isidorn I checked Windows and Linux and I see no conflict.
Windows:

Linux: I can use both Ctrl+R and Ctrl+Shift+R without issues.
Please clarify.

@bpasero Yeah if there are no conflicts I would just push those as the defaults.

Great. They are the defaults already.


Group clusters by provider in the explorer

Group clusters by provider in the explorer
Signed-off-by: azerr<EMAIL_ADDRESS>

This PR groups cluster provider in the explorer:

It improves too the selection of topic, cluster which is created by wizard. Indeed the selection can be done only if parent are expanded otherwise the selection fails (ex : don't expand items of explorer and create a topic, the topic will not be selected, this PR fixes that).


v6: build error on test

just cloned the v6 branch.
git clone -b v6 --single-branch https://github.com/erikras/redux-form

and npm install gives error  (npm run build is fine)
npm i                                                                                v6
npm WARN deprecated<EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.

><EMAIL_ADDRESS>install /Users/bsr/Work/Programming/javascript/repo/redux-form/node_modules/fsevents
> node-pre-gyp install --fallback-to-build

[fsevents] Success: "/Users/bsr/Work/Programming/javascript/repo/redux-form/node_modules/fsevents/lib/binding/Release/node-v47-darwin-x64/fse.node" is installed via remote

><EMAIL_ADDRESS>prepublish /Users/bsr/Work/Programming/javascript/repo/redux-form
> npm run test && npm run clean && npm run build

><EMAIL_ADDRESS>test /Users/bsr/Work/Programming/javascript/repo/redux-form
> mocha --compilers js:babel-register --recursive --recursive 'src/**/__tests__/*' --require src/__tests__/setup.js

/Users/bsr/Work/Programming/javascript/repo/redux-form/src/__tests__/setup.js:11
global.navigator.product = 'NotReactNative';
                         ^

TypeError: Cannot set property product of [object Object] which has only a getter
    at Object.<anonymous> (setup.js:7:1)
    at Module._compile (module.js:413:34)
    at loader (/Users/bsr/Work/Programming/javascript/repo/redux-form/node_modules/babel-register/lib/node.js:158:5)
    at Object.require.extensions.(anonymous function) [as .js] (/Users/bsr/Work/Programming/javascript/repo/redux-form/node_modules/babel-register/lib/node.js:168:7)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Module.require (module.js:367:17)
    at require (internal/module.js:16:19)
    at /Users/bsr/Work/Programming/javascript/repo/redux-form/node_modules/mocha/bin/_mocha:310:3
    at Array.forEach (native)
    at Object.<anonymous> (/Users/bsr/Work/Programming/javascript/repo/redux-form/node_modules/mocha/bin/_mocha:309:10)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Function.Module.runMain (module.js:447:10)
    at startup (node.js:141:18)
    at node.js:933:3

npm ERR! Darwin 13.4.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "test"
npm ERR! node v5.7.0
npm ERR! npm  v3.6.0
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>test: `mocha --compilers js:babel-register --recursive --recursive 'src/**/__tests__/*' --require src/__tests__/setup.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test script 'mocha --compilers js:babel-register --recursive --recursive 'src/**/__tests__/*' --require src/__tests__/setup.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the redux-form package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     mocha --compilers js:babel-register --recursive --recursive 'src/**/__tests__/*' --require src/__tests__/setup.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs redux-form
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls redux-form
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/bsr/Work/Programming/javascript/repo/redux-form/npm-debug.log

npm ERR! Darwin 13.4.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "i"
npm ERR! node v5.7.0
npm ERR! npm  v3.6.0
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>prepublish: `npm run test && npm run clean && npm run build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prepublish script 'npm run test && npm run clean && npm run build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the redux-form package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run test && npm run clean && npm run build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs redux-form
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls redux-form
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/bsr/Work/Programming/javascript/repo/redux-form/npm-debug.log

offending line was
global.navigator.product = 'NotReactNative' in setup.js

getDisplayName
✓ should read displayName
✓ should read name
✓ should default to Component
501 passing (5s)
~/W/P/j/r/redux-form ❯❯❯ git diff                                                                                v6 ✱
diff --git a/src/tests/setup.js b/src/tests/setup.js
index 9c2d05b..71f5ea4 100644
--- a/src/tests/setup.js
+++ b/src/tests/setup.js
@@ -4,4 +4,4 @@ import { jsdom } from 'jsdom'
global.document = jsdom('<!doctype html>')
global.window = document.defaultView
global.navigator = global.window.navigator
-global.navigator.product = 'NotReactNative'
+// global.navigator.product = 'NotReactNative'
~/W/P/j/r/redux-form ❯❯❯

This is strange. I've recently setup the v6 dev environment and didn't experienced any issue. Did you manage to this fix?
My dev env is a bit outdated but I'm using:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

This is probably related to a recent update of jsdom. v9.1.0 was released a few hours before you opened this issue and they reimplemented how navigator works.
As a temporary workaround you could force the version of jsdom to be 9.0.*:
npm install<EMAIL_ADDRESS>
We should update the tests to fix how we use navigator.

Fixed by #977.


Update seeds to use .create instead of .insertMany

In order to fix the issue of the passwords not hashing when the users were seeded, the .insertMany functions need to be changed to .create. This also requires updating the code to grab the IDs from the result of the create.
To test, run npm run seed and view the database in MongoDB Compass or run npm start and run some queries at /graphql in the browser.

Looks good to me! Checked queries as well. Everything is well populated and the password is hashed

Password hashing problem was resolved. Db populates.


Revert "fix: support labels for schema validation (#260)"

Reverts GoogleCloudPlatform/professional-services-data-validator#381

/gcbrun


Datetime hover tool shows UTC time by default

Hello Team,
I am plotting data on map using plot_bokeh.map(). Everyting seems to be working fine except date time column which is displaying UTC as default . I am using Hoevertool_columns to display the datetime when I hover over the points on map.
Could you please let me know how to fix this issue

Hi @koti-Maddala ,
Could you please provide a minimal code example where This issue occurs?
Best Patrik

Hello Patrick,
My bad, it was the default datetime issue on the server. It is fixed now :)


Probable Cause

Add probability calculation into graph and include on export.
This addresses #19 (shoutout to @reidjohnson for pointing me in the right direction. most of this code is his!).

lgtm

Thanks!


Apply John Papa's AngularJS style guide

See
https://github.com/johnpapa/angular-styleguide

Done


About the loss function problem

In the function of https://github.com/ExplainableML/ProbVLM/blob/cb69f28b1ab23142a1c671e004b09b5cb5d8a204/src/losses.py#L81, nn.L1Loss is inconsistent with Eq.3 in the paper, and Eq. 3 does not include the nn.L1Loss.
Waiting for your help! @udion @sgk98

Why isn't this line of code being used instead of the twenty-ninth line of code? Beta is power in the paper.
https://github.com/ExplainableML/ProbVLM/blob/cb69f28b1ab23142a1c671e004b09b5cb5d8a204/src/losses.py#L28

For stability during training, beta in power leads to unstable training hitting nans, so we used Taylor series approximation, explained in Appendix of the paper here: https://arxiv.org/pdf/2307.00398 (Eq. 16 in appendix A)


add get_config method to callbacks

Summary
Add get_config method to callback classes:
Callbacks as well as layers and optimizers have lots of arguments. The get_config method in the callbacks returns a dictionary with the configuration of the callback allowing us to inspect the parameters of the instance. This allow us to understand better the callbacks and even change them if it is necessary easily.
Objectives

This method would allow to easily obtain the configuration of a callback without knowing the arguments of the callback.
For the sake of completeness classes such as the layers, and optimizers have the get_config method, thus I think that it will be useful to add this method to callback objects.
Additionally:
It is known that keras object are not pickleable, adding a get_config method to optimizers may enable to share different optimizers configuration without the need of sharing the object themselves.

Now if we want to know the arguments of the instance we can check it by instance.argument. However not always we know or remember all the arguments. The get_config method provides an easy way to obtain the parameters. This method might affect all keras users if they want to use it.
Related Issues
#12802
PR Overview

[n] This PR requires new unit tests [y/n] (make sure tests are included)
[n] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[y ] This PR is backwards compatible [y/n]
[n] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thank you for preparing this PR.
We are no longer adding new features to multi-backend Keras, as we are refocusing development efforts on tf.keras. If you are still interested in submitting this PR, please direct it to tf.keras in the TensorFlow repository instead.
I think making callbacks serializable is a good idea.


Glk library variations

This splits out the 3 Glk library resources from #617 into their own formulae and modifies glulxe and git-if to build using them.

Looks good to me. I think this is a more maintainable design.
Anyone have concerns before I pull this?

Merged. Thank you for your contribution to Homebrew!


don't update ip address if changed

because flux deploy three nodes for same peer id
so each node try to update its own ip address to flux
this leads to gas station drain

https://github.com/cyberfly-io/cyberfly-node/commit/b2ff58b1360c1607d1fdfb71da465b37ebbd6eb5


Azure Data Studio window crash upon editing notebook SQL code syntax

I noticed that upon editing database's notebook, whole ADS window crashes. If I remove every sign one by one at time intervals in order for ADS to process it, it might still work, but it does it extremely slowly and between removing certain digits the app freezes. But if I repeatedly hit backspace, it completely crashes. My guess is that's caused by some heavy SQL transactions inside, however I don't execute them, just want to edit some of them, so no idea what's causing this problem.  Issue persists between app restarts.

Worth noting is the fact that it started to happen after last update. Before that it worked just fine.

The problem fixed itself. Upon closing the notebook and re-opening it, the app suddenly started to work properly.

I've had the same issue, I think there's a bug here but can't pinpoint it

Same issue here, it crashes even i just tried to edit text


adobe connect

Checklist

[x] I'm reporting a new site support request
[ ] I've verified that I'm running youtube-dl version 2021.06.06
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that none of provided URLs violate any copyrights
[ ] I've searched the bugtracker for similar site support requests including closed ones

Example URLs
https://vc1.tabrizu.ac.ir/p2kiia97gbco/?session=breezbreezwxk52g67sny79dfx

WRITE DESCRIPTION HERE
youtube-dl -v https://vc1.tabrizu.ac.ir/p2kiia97gbco/?session=breezbreezwxk52g67sny79dfx
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://vc1.tabrizu.ac.ir/p2kiia97gbco/?session=breezbreezwxk52g67sny79dfx']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.06.06
[debug] Python version 3.9.9 (CPython) - macOS-12.0.1-x86_64-i386-64bit
[debug] exe versions: avconv 12.3, avprobe 12.3, ffmpeg 4.4.1, ffprobe 4.4.1
[debug] Proxy map: {}
[generic] ?session=breezbreezwxk52g67sny79dfx: Requesting header
WARNING: Falling back on generic information extractor.
[generic] ?session=breezbreezwxk52g67sny79dfx: Downloading webpage
[generic] ?session=breezbreezwxk52g67sny79dfx: Extracting information
ERROR: Unsupported URL: https://vc1.tabrizu.ac.ir/p2kiia97gbco/?session=breezbreezwxk52g67sny79dfx
Traceback (most recent call last):
File "/usr/local/Cellar/youtube-dl/2021.6.6/libexec/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
return func(self, *args, **kwargs)
File "/usr/local/Cellar/youtube-dl/2021.6.6/libexec/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
ie_result = ie.extract(url)
File "/usr/local/Cellar/youtube-dl/2021.6.6/libexec/lib/python3.9/site-packages/youtube_dl/extractor/common.py", line 534, in extract
ie_result = self._real_extract(url)
File "/usr/local/Cellar/youtube-dl/2021.6.6/libexec/lib/python3.9/site-packages/youtube_dl/extractor/generic.py", line 3520, in _real_extract
raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: https://vc1.tabrizu.ac.ir/p2kiia97gbco/?session=breezbreezwxk52g67sny79dfx

Two problems here. First, yt-dl doesn't know that the page has an AdobeConnect item in it; second, even when it does, the extraction fails because the page format is not what the AdobeConnect extractor expects.
This patch enables yt-dl to find some rtmp content on the page:
--- old/youtube-dl/youtube_dl/extractor/adobeconnect.py
+++ new/youtube-dl/youtube_dl/extractor/adobeconnect.py
@@ -4,24 +4,36 @@
 from .common import InfoExtractor
 from ..compat import (
     compat_parse_qs,
-    compat_urlparse,
+    compat_urllib_parse_unquote,
+    compat_urllib_parse,
 )

 class AdobeConnectIE(InfoExtractor):
-    _VALID_URL = r'https?://\w+\.adobeconnect\.com/(?P<id>[\w-]+)'
+    # custom pattern for issue #30349
+    _VALID_URL = r'''(?x)
+            https?://
+            (?:
+                \w+\.adobeconnect\.com|
+                vc1.tabrizu.ac.ir
+            )
+            /(?P<id>[\w-]+)
+    '''

     def _real_extract(self, url):
         video_id = self._match_id(url)
         webpage = self._download_webpage(url, video_id)
         title = self._html_search_regex(r'<title>(.+?)</title>', webpage, 'title')
-        qs = compat_parse_qs(self._search_regex(r"swfUrl\s*=\s*'([^']+)'", webpage, 'swf url').split('?')[1])
+        qs = self._search_regex(r"(?:'{swfUrl}|{swfUrl}')([^']+)'".format(swfUrl=r'swfUrl\s*=\s*'), webpage, 'swf url')
+        if '%3A%2F%2F' in qs:
+            qs = compat_urllib_parse_unquote(qs)
+        qs = compat_parse_qs(qs.split('?')[1])
         is_live = qs.get('isLive', ['false'])[0] == 'true'
         formats = []
         for con_string in qs['conStrings'][0].split(','):
             formats.append({
                 'format_id': con_string.split('://')[0],
-                'app': compat_urlparse.quote('?' + con_string.split('?')[1] + 'flvplayerapp/' + qs['appInstance'][0]),
+                'app': compat_urllib_parse.quote('?' + con_string.split('?')[1] + 'flvplayerapp/' + qs['appInstance'][0]),
                 'ext': 'flv',
                 'play_path': 'mp4:' + qs['streamName'][0],
                 'rtmp_conn': 'S:' + qs['ticket'][0],

In a permanent fix, to avoid filling the _VALID_URL pattern with a list of sites, possible AdobeConnect content should be detected in the generic extractor as for many other embedded media types, but that is also an area that needs cleaning up.

i appreciate your time and and thorough reply


Insert missing space in README.md

A small space for myself but a big one for this project 🤣 🚀

I'd rather only pull in changes to master in versioned batches (which is why CONTRIBUTING.md requests that pull requests target develop), but since this is just fixing whitespace, I could just do it and credit you in the changelog next release. Cool?


Fixing tar output

Jira:https://issues.redhat.com/browse/MGDSTRM-6283
Fixing the tar input for the get rhosak operator script

Build is successful
18:20:50 Step 28/29 : RUN rm /root/anaconda* /root/original-ks.cfg
18:20:50  ---> Running in 86991435bdcf
18:20:50 Removing intermediate container 86991435bdcf
18:20:51  ---> 78a6c1fb9e33
18:20:51 Step 29/29 : WORKDIR /root
18:20:51  ---> Running in 39d7d1952346
18:20:51 Removing intermediate container 39d7d1952346
18:20:51  ---> 53b90398f836
18:20:51 Successfully built 53b90398f836
18:20:51 Successfully tagged quay.io/app-sre/managed-scripts:e1784bf
18:20:51 /usr/bin/docker tag quay.io/app-sre/managed-scripts:e1784bf quay.io/app-sre/managed-scripts:latest
18:20:51 [openshift-managed-scripts-gh-pr-check] $ /bin/sh -xe /tmp/jenkins3825402766060996190.sh
18:20:51 + '[' -z '' ']'
18:20:51 + exit
18:20:51 Setting status of b63f94083aa971d6fd995db9b0853d0e0ef8a8c0 to SUCCESS with url https://ci.ext.devshift.net/job/openshift-managed-scripts-gh-pr-check/246/ and message: ' '
18:20:51 Using context: ci.ext.devshift.net PR build
18:20:51 Finished: SUCCESS

/approve

@reetika-vyas Is this ok to be merged?

I think the /approve is the harder gate you need to pass
so
/lgtm
and it should auto merge

Can you please add a description to this?

@R-Lawton @georgettica could ensure when creating PR's we have a description describing the change? Is this related to an ticket? Please link the ticket and update the description.

sorry for pushing forward without verifying the Description is there. I will keep track of this aswell


Remove accessory type from select token screen

Remove accessry type for cells, anyway once token selected we dismiss a screen

@oa-s how do you navigate to that state in your screenshot above? For me the Select Asset screen goes away before the actionsheet is shown.


Routing incorrect for hud dropdown menu

Chrome autocorrects this issue, however @clarkjefcoat pointed out that a trailing slash is needed for the center URL in HUD otherwise the links fail in IE

this is fixed in production