A puzzling question?

sol version: 3.2.3
lua version: 5.4.2
Visual Studio 2019
Windows 10 x64

code:

struct Test1 {
    Test1() { std::cout << 1 << std::endl; }
};
struct Test2 {
    Test2() { std::cout << 2 << std::endl; }
};
sol::state lua = {};

auto t1 = lua.new_usertype<Test1>("Test1");

// --------I can do this,compile pass--------
sol::table u2t = lua["Test1"];
u2t.new_usertype<Test2>("Test2");
//-------------------------------------------

// -----------I can't do this,compile error------------
// auto t2 = t1.new_usertype<Test2>("Test2");
// t1.as<sol::table>().new_usertype<Test2>("Test2");
// ----Why ban this way,they have the same behavior----

lua.script(R"(
    local t1 = Test1.new();
    local t2 = Test1.Test2.new();
)");

I don't think I went out of my way to ban that... can you make sure to use t1.get<sol::table>().new_usertype<Test2>("Test2"); instead? The reason it's not named as is because proxy types do lookup.
In short, it does t1.get(<sol::table>("Test2"), which is different from converting the already-stored object (sol::object) into something by using .as<T>().

thanks for your anwser.


CXF-8442 Java 16 & CachedOutputStream Fix IOException closed after multiple close() & postClose() calls

see https://issues.apache.org/jira/browse/CXF-8442

Hi @reta,
Thanks for your answer and suggestions. I reverted my changes in CacheAndWriteOutputStream.
I added a variable isClosed to CacheAndWriteOutputStream and as postClose() and closeFlowthroughStream() had the same implementation I reduced it to a call to postClose().
Please review my pull request again, see the test case CacheAndWriteOutputStreamTest#testCloseMultipleTimes().

@Sevyls Thank you.


sftp on windows

I'm sorry to ask a maybe not related question to restic.
What is the simplest way to use sftp on windows ? I know putty i cannot make it work with restic.
Maybe it's on the roadmap to don't need any external ssh cmd ?

Hey, thanks for the suggestion. At the moment, restic depends on an external command ssh so that we do not have to manage the ssh configuration, which is rather complex. It's not planned, but there is the possibility to use an internal ssh implementation. but we'll tackle that problem later.

I'd like to share some information regarding using plink.exe (from the Putty family) for sftp on Windows:
I hacked src/restic/backend/sftp/sftp.go to look for a couple of environment variables that give the location of the plink executable and my private key ppk file, and if they are defined construct the ssh command line like so:
/path/to/plink.exe -s -l username -i /path/to/myprivatekey.ppk hostname sftp
I have not done extensive testing, but listing snapshots, backup, and restore seem to work.  "restic init" fails with "ssh": executable not found, so obviously there's another spot that needs some work to use plink.exe instead of the hardcoded "ssh".


Add KingIburu's file

second slide is created and left arrow will be used to move back a page.

booya


fix inquiry form

fixes https://github.com/artsy/eigen/issues/468

Does this deal with rotation correctly?


Branding does not show up correctly on grubmenu with kairos 2.1.1

**Kairos version:**
root@edge-89a33842be829ddebe91aba70105bcfc:~# cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.2 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.2 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy
KAIROS_NAME="kairos-core-ubuntu-22-lts"
KAIROS_VERSION="v2.1.1"
KAIROS_ID="kairos"
KAIROS_ID_LIKE="kairos-core-ubuntu-22-lts"
KAIROS_VERSION_ID="v2.1.1"
KAIROS_PRETTY_NAME="kairos-core-ubuntu-22-lts v2.1.1"
KAIROS_BUG_REPORT_URL="https://github.com/kairos-io/kairos/issues"
KAIROS_HOME_URL="https://github.com/kairos-io/kairos"
KAIROS_IMAGE_REPO="quay.io/kairos/core-ubuntu-22-lts"
KAIROS_IMAGE_LABEL="latest"
KAIROS_GITHUB_REPO="kairos-io/kairos"
KAIROS_VARIANT="core"
KAIROS_FLAVOR="ubuntu-22-lts"KAIROS_NAME="core-ubuntu-22-lts-k3s:v3.4.7-alpha2"
KAIROS_VERSION="1.24.6-k3s1_v3.4.7-alpha2"
KAIROS_ID="core-ubuntu-22-lts-k3s"
KAIROS_ID_LIKE="core-ubuntu-22-lts-k3s:v3.4.7-alpha2"
KAIROS_VERSION_ID="1.24.6-k3s1_v3.4.7-alpha2"
KAIROS_PRETTY_NAME="core-ubuntu-22-lts-k3s:v3.4.7-alpha2 1.24.6-k3s1_v3.4.7-alpha2"
KAIROS_BUG_REPORT_URL=""
KAIROS_HOME_URL=""
KAIROS_IMAGE_REPO="gcr.io/spectro-dev-public/aslam"
KAIROS_IMAGE_LABEL="v2.1.1_1.24.6-k3s1_v3.4.7-alpha2"
KAIROS_GITHUB_REPO=""
KAIROS_VARIANT="

CPU architecture, OS, and Version:
Linux edge-89a33842be829ddebe91aba70105bcfc 5.19.0-45-generic #46~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Wed Jun 7 15:06:04 UTC 20 x86_64 x86_64 x86_64 GNU/Linux
Describe the bug
Branding does not show up fine on the grubmenu. Please find the attached screenshot.
Kairos 2.1.1

Kairos 2.2.0

To Reproduce

Expected behavior
Branding to show up correctly on the grubmenu.
Logs

Additional context

@santhoshdaivajna thanks for reporting this. The issue is indeed present in 2.1 but was fixed in 2.2. There are no plans of backporting this fix atm so the best would be to upgrade your karios system.

We will do a 2.1.2 release for this. The CI takes a bit though, so you can expect it for Monday but I'll ping you as soon as it is ready

in the first image, the 2 extra options at the bottom are the ones coming from the grubmenu.cfg, which extends the default grub so the grubmenu override works.
What doesnt work is the default grub entry name, which is set via the elemental config. This is a known issue in kairos-agent 2.1.1 which comes from the full rework of the config to merge kairos-agent and elemental cli.
kairos 2.1.3 is the one that fixes this by loading  the elemental config instead of creating a new one.
This can be easily checked by once booting confirming that there is a file in /run/initramfs/cos-state/grub_oem_env which contains the default_menu_entry key with a value.
If that file has cOS as the value it means that the issue comes from the agent not reading the elemental config files during install.
Our default grub reads several files from the state partition in which we are able to override the default values:

grubenv (from the oem partition, not state!)
grub_oem_env
grubmenu
grubcustom

grubmenu is used to EXTEND the grub menu with extra entries, it does not contain any vars
grubcustom is used for the boot assesment, can also extend the grub menu
grubenv is used to set the entries and allow overriding values for users during a running system (grub2-editenv /oem/grubenv set next_entry=statereset)
grub_oem_env is ONLY used for the default_entry_name and ONLY created during install. It allows to setup oem values that will be applied during install and could be extended in the future to provide more configurations for the default boot.
it should be impossible to set variables via the grubmenu.cfg as that is a menu entry file not an env file and they have different formats and can only be edited via grub2-editenv.
So the fix would be to update the repos on 2.1.3 to grab the kairos-agent v2.1.3 which should fix this.

This should be fixed in https://github.com/kairos-io/kairos/releases/tag/v2.1.3 . Please let us know and re-open if necessary Thanks @mauromorales and @Itxaka !


Country

Is there a way to default queries by country?

Good question, this type of filter would be based on each individual provider.
Google Geocoding API has these additional parameters that I can include in this module.
Optional parameters in a geocoding request:

bounds — The bounding box of the viewport within which to bias geocode results more prominently. This parameter will only influence, not fully restrict, results from the geocoder. (For more information see Viewport Biasing below.)
language — The language in which to return results. See the list of supported domain languages. Note that we often update supported languages so this list may not be exhaustive. If language is not supplied, the geocoder will attempt to use the preferred language as specified in the Accept-Language header, or the native language of the domain from which the request is sent. Street-level addresses will be returned in the local language, transliterated to a script readable to the user if necessary, observing the preferred language to ensure addresses are readable to both the user and locals. All other addresses will be returned in the preferred language. Address components will all be returned in the same language, which is chosen from the first component. Should names not be available in the preferred language, the closest match will be used.
region — The region code, specified as a ccTLD ("top-level domain") two-character value. This parameter will only influence, not fully restrict, results from the geocoder. (For more information see Region Biasing below.)
components — The component filters, separated by a pipe (|). Each component filter consists of a component:value pair and will fully restrict the results from the geocoder. For more information see Component Filtering, below.


improvement: set activeTimezone when transforming dates

no issue
Set activeTimezone settings property, when running 006 migration.
Right now: we only have a default active timezone (for the blog) configured - which is UTC timezone. But when your ghost server runs in a non UTC timezone, then your blog would be in UTC and all your published_at posts dates get shifted to UTC timezone. This can be very confusing.
So we are setting the activeTimezone settings property to the server timezone (or to the one which is configured in config.js), when running the migration.
Only tested manual.

If I'm reading this correctly the timezone can be set to something that is not in our condensed list of timezones even if the timezone falls into one of our timezone groups.
Eg. for me moment.tz.guess() returns Europe/London which is correct but in our timezones.json that would equate to Europe/Dublin.
If we're OK with that then there will need to be some changes to the client to deal with settings.activeTimezone being set to something that's not in the selectable list - although that's probably a good thing as it will also allow us to later use moment.tz.guess() during the setup process to match the blog timezone to the blog owner's timezone.

Interesting.
#7143 will not allow setting any timezone string anymore, because of moment. I think we have to adapt our timezones.json.

I think #7143 needs to be changed to use moment.tz.zone(givenTimezone) to check whether a zone is valid or not? I'm working on a small client PR so that the client copes with a manual timezone override.

changed #7143 to allow all valid moment timezones
thanks @kevinansfield


Having trouble integrating the LiFx Switch into HA

Hey mate,
I copied you JSON config into my node red and modified the serial to be my switches serial number. I also installed your Photons integration but I cant seen to get the configuration correct, could you point me in the right direction?
Below is my current config.
`interactor:
host: homeassistantip
port: 6100
database:
  uri: "{config_root}/interactor.db"

daemon_options:
    limit: 30 # Limit of 30 messages inflight at any one time
    search_interval: 1800 # do a discovery every 30 minutes
    time_between_queries:
      LIGHT_STATE: 600 # label, power, hsbk every 10 minutes
      VERSION: null # The type of product can be cached forever
      FIRMWARE: 86400 # Cache the firmware version for a day
      GROUP: 780 # Cache group information for 13 minutes
      LOCATION: 9600 # Cache location information for 16 minutes

zeroconf:
  enabled: true
  ip_address: homeassistantIP
  name: homeassistant`

Above is my current config with both ip_addresses configured as my HA IP.
Here is my interactor logs
2023-02-26 16:39:22,893 INFO    whirlwind.server Hosting server at http://homeassistantip:6100 2023-02-26 16:39:22,896 INFO    interactor.zeroconf Enabling Zeroconf service discovery	hostname=homeassistant.	ipaddress=homeassistantip	port=6100	sd=homeassistant 2023-02-26 16:39:22,900 INFO    photons_transport.transports.udp Creating datagram endpoint	address=('<IP_ADDRESS>', 56700)	serial=None 2023-02-26 16:47:36,790 INFO    interactor.server Command	command=query	matcher={"serial": "d073xxxxx92"}	method=PUT	path=/v1/lifx/command	remote_ip=homeassistantip	request_identifier=e89f3ea5-997e-40cc-b332-74755fda3764	uri=/v1/lifx/command 2023-02-26 16:47:36,797 INFO    interactor.server Response	command=query	method=PUT	remote_ip=homeassistantip	request_identifier=e89f3ea5-997e-40cc-b332-74755fda3764	status=200	took_seconds=0.01	uri=/v1/lifx/command 2023-02-26 16:47:37,033 INFO    interactor.server Command	command=query	matcher={"serial": "d073xxxx92"}	method=PUT	path=/v1/lifx/command	remote_ip=homeassistantip	request_identifier=3ee5a14e-be03-4d36-b3b8-aecaf0eae134	uri=/v1/lifx/command 2023-02-26 16:47:37,035 INFO    interactor.server Response	command=query	method=PUT	remote_ip=homeassistantip	request_identifier=3ee5a14e-be03-4d36-b3b8-aecaf0eae134	status=200	took_seconds=0.0	uri=/v1/lifx/command 2023-02-26 16:47:38,035 INFO    interactor.server Command	command=query	matcher={"serial": "d0xxxxb92"}	method=PUT	path=/v1/lifx/command	remote_ip=homeassistantip	request_identifier=ee414308-c780-46b5-afac-8c8c9bc7e11a	uri=/v1/lifx/command 2023-02-26 16:47:38,037 INFO    interactor.server Response	command=query	method=PUT	remote_ip=homeassistantip	request_identifier=ee414308-c780-46b5-afac-8c8c9bc7e11a	status=200	took_seconds=0.0	uri=/v1/lifx/command
I have attached my node red config but my issue is I basically just have no idea where to put in the remote IP for my LiFx switch.

Thanks for your help, I am sure it something simple I am stuffing up but I am new to Node-Red and have no idea what I am doing. Cheers

I have obscured my switch serial number in the logs but it is correct according to the app.

Don't use this method: it's woefully inefficient, even if you can get it to work and it's exceptionally fragile as it relies on both Node-RED and Photons Interactor instead of just relying on Home Assistant. While I was happy I found this method, I never actually used it for very long.
Instead, use the method I wrote for the official Home Assistant documentation: https://www.home-assistant.io/integrations/lifx/#lifx-switch
This should only require an iOS device if you have a LIFX Switch that didn't come with a HomeKit code printed on the Switch itself or in the box, i.e. from before they were HomeKit enabled. In that case, you only need the iOS device to get the HomeKit code itself. Once you have that, everything else just uses Home Assistant.


Integrate with django-admin?

Any good ideas out there to get this to work with the django-admin command?

Not sure what the question is.  Do you mean that you want django-admin startproject to somehow generate a settings file that uses dotenv?  That’s probably out of scope for django core, and I don’t know of any hooks that would let dotenv change code.  I think many people use things like cookiecutter, or just copy settings from one project to the next.

Sorry, what I meant was is there a way to load the environment variables when using django-admin command.  The other way is to just use manage.py, but in my case I've setup my CLI aliases to use django-admin.

Ah yes, makes sense 👍


Bump to Go-1.5.3

Bump to Go-1.5.3
Security update: https://groups.google.com/forum/#!topic/golang-announce/MEATuOi_ei4
Signed-off-by: Ivan Mikushin<EMAIL_ADDRESS>

LGTM 🐶

Thank you @imikushin :+1:

@vdemeester Always welcome :)


feat: endpoint for autosave information

closes #584
Made according to the swagger spec that @lorenzo-cavazzi specified here
Example output:
{
  "autosaves": [
    {
      "branch": "master",
      "commit": "993875e950de66a1074d58cb8ce89b54f41a0941",
      "date": "2021-03-08T16:04:08+00:00",
      "pvs": false
    }
  ],
  "pvsSupport": true
}

/deploy

btw, you need /deploy not \deploy :)

You can access the deployment of this PR at https://renku-ci-nb-607.dev.renku.ch

I haven't looked much into the code, but I've tested the endpoint in the PR deployment and it works well!
I've tried with projects in groups and subgroups, all good :tada:
Would it be difficult to add also the support for PVS in the PR deplyoment, so that I can try also that part?

@lorenzo-cavazzi I deployed this at https://tasko.dev.renku.ch/ with support for pvcs. So feel free to try that out.


How can I get match_body of each hit？

Hi, I'm using elastalert to monitor my service exceptions. The alert I get, however, can not show me all the exception infos of each exception, for example
INFO:elastalert:Ran OOM Any Out Rule from 2018-05-31 10:07 CST to 2018-05-31 10:22 CST: 6 query hits (1 already seen), 5 matches, 1 alerts sent

As collecting oom exception infos, I got 5 matcher, but only received 1 alert, therefore I only got 1 exception info.
How can I get all 5 exception infos in my alert? plz give me some advice.
Here's a demo of my rule:
name: OOM Any Out Rule

type: any

index: applog-*

num_events: 1

timeframe:
  hours: 1

filter:
- query:
    query_string:
        query: "message: unable to create native thread"

alert:
- "email"

email:
-<EMAIL_ADDRESS>

Hi, I read other issues and I use agg nearly resolved this question. Some other questions, however, came up to me, my alert email is like:
Aggregation resulted in the following data for summary_table_fields ==> ['service', 'stack', 'name', '@timestamp', 'count']:

+---------+---------+------------------------+-------------------------+-------+
| service |  stack  |          name          |       @timestamp        | count |
+=========+=========+========================+=========================+=======+
| cmsk    | prd-two | r-prd-two-cmsk-62ae0f4 | 2018-05-31T05:46:50.991 | 1     |
|         |         |                        | Z                       |       |
+---------+---------+------------------------+-------------------------+-------+
| cmsk    | prd-one | r-prd-one-cmsk-b37ec09 | 2018-05-31T05:48:36.053 | 1     |
|         |         |                        | Z                       |       |
+---------+---------+------------------------+-------------------------+-------+

OOM Any Out Rule

@timestamp: 2018-05-31T05:46:50.991Z
@version: 1
_id: f4d63572-44fe-4a7f-b0d7-80836e439e41
_index: applog-2018.5.30
_type: applog
host: cnsz01vl00643
index: 1
message: something wrong like unable to create native thread, you know
name: r-prd-two-cmsk-62ae0f4
num_hits: 2
num_matches: 2
path: ~/log.log
service: cmsk
stack: prd-two

----------------------------------------
OOM Any Out Rule

@timestamp: 2018-05-31T05:48:36.053Z
@version: 1
_id: 88b84af1-fed0-488f-af74-6827df0da1a9
_index: applog-2018.5.30
_type: applog
host: cnsz01vl00643
index: 1
message: something wrong like unable to create native thread, you know
name: r-prd-one-cmsk-b37ec09
num_hits: 2
num_matches: 2
path: ~/log.log
service: cmsk
stack: prd-one

How can I remove the default 'count' field in this table ?
I want nothing but the table in my email, how can I remove the "match_body" prographs ?
I had already configured "use_local_time: true" in my .yaml, and my local_time is CST, why the "@timestamp" and other time in es-alert is still UTC...?

Plz do me a favor.

You can't.
alert_text_type: aggregation_summary_only
Because that @timestamp is coming directly from the document, use_local_time generally applies logs and messages coming from elastalert itself, but nothing that is pulling fields directly. Documentation isn't very clear on this, sorry about that.

You might also try removing summary table fields and using
alert_text_type: alert_text_only
alert_text: "{} - {} - {} - {}"
alert_text_args: ["service", "stack", "name", "@timestamp"]


async: Prepare 0.4.0 release

I added a changelog, the licenses and filled the authors in commit order.
Is there anything left to do before the 0.4.0 release?
@MathiasKoch could you add the team @rust-embedded-community/embedded-storage to crates.io?
cc: @lulf, @Dirbaio

ping @MathiasKoch

Ohh.. sorry, i missed this initially due to my inactivity by paternity leave.
Will add the team tomorrow morning. 👍

Done :+1:

Ping again! @eldruin @MathiasKoch could we get this released?
I volunteer to be a maintainer, if it helps!

Sorry for the delay. Let's get this released.


Appstudio update test-component-pac-dedt

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-dedt-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
27 seconds
build-container

✅ Succeeded
19 seconds
inspect-image

✅ Succeeded
19 seconds
deprecated-base-image-check

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
47 seconds
clamav-scan

✅ Succeeded
15 seconds
sbom-json-check

✅ Succeeded
53 seconds
label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
8 seconds
show-summary

Pipelines as Code CI/test-component-pac-dedt-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
24 seconds
build-container

✅ Succeeded
23 seconds
deprecated-base-image-check

✅ Succeeded
23 seconds
inspect-image

✅ Succeeded
12 seconds
clair-scan

✅ Succeeded
35 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
17 seconds
label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary


Add tally votes command and add an optional gasPrice arg to all cmds

Signed-off-by: Krasi Georgiev<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 5 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.003%) to 5.617%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/ethereum/ethereum.go
0
5
0.0%

Totals

Change from base Build 996156995:
-0.003%

Covered Lines:
102

Relevant Lines:
1816

💛  - Coveralls


[HOLD #20893] Display both participants of top of Money Request Report actions

HELD on https://github.com/Expensify/App/issues/20893
Currently, when a user requests money, we open a new IOUReport. On the chat report, the action appears under the first requester's name:

This is inaccurate, as the report is owned by both the requester and the payer, and could have different balances as more requests are added.
Solution
Display both names on top of the report action:

Making this a weekly since it's on hold.

I'll take this over from you Puneet, as it's tied to manual requests.

@Gonals will you be able to knock this one out this week now that the SNH freeze is over?

Once the card one merges, this one goes right after it!

Wahoo!

Quick temperature check: Where does this PR fit in your priority queue right now?

Competing with Corpay, but I'm hoping to get it in review this week.

Ok thanks!

Shall we remove the hold now that #20893 is merged, on production and awaiting payment?

This was a design tweak, and handled internally.
$1,000 payment due to @aimane-chnaif for the C+ review. Offer sent here.

Paid!


Context Issues with

I'm using React-Router in order to generate links to route to dynamically-generated routes in a dynamically generated template area. Here's the code:
`ajax('GET','/content/template/'+template+'/master.html',function(tmp){
document.getElementById('template').innerHTML = tmp;
  var mop = document.getElementsByClassName("menu");
  for (var i = 0; i < mop.length; i++) {
      if(!mop[i].getAttribute('menu-name')){
        mop[i].innerHTML = '
Menu not found.
';
        return;
      }
      var mitems = menus[mop[i].getAttribute('menu-name')];
      for(var item in mitems) {
        React.render(<Link to={mitems[item].route}>{mitems[item].text}</Link>, getAllElementsWithAttribute('menu-name',mop[i].getAttribute('menu-name'))[0]);
      }
  }

});`
All is well, and works well, until the final line of code, namely:
React.render(<Link to={mitems[item].route}>{mitems[item].text}</Link>, getAllElementsWithAttribute('menu-name',mop[i].getAttribute('menu-name'))[0]);
where, as I have seen in https://github.com/rackt/react-router/issues/400, and https://github.com/rackt/react-router/issues/513, gives

From what I have seen, this is a bug with context, however there are no cloneWithProps() in my application, and nothing seems that out of the ordinary. Any help with workarounds would be appreciated

I think this has something to do w/ React .13?

I'm having a hard time understanding what you're trying to do and why you'd want to render like that. How does the rest of your app work?
Generally speaking, I don't think it makes sense to load html from server, render it by setting innerHTML and then replacing several parts using React.render. You usually want to have a root element (e.g. document.body) where you render your whole React app, and then load data (not markup) via ajax to update the UI.

@taurose Basically this is done in this way so users are able to edit what they are familiar with, HTML.

I'm still not seeing the full picture. Do you have a top level React.render call inside Router.run? Why do you use react if your markup is (mostly?) static?
Back to the problem: As far as I understand context, it's not preserved when you do another React.render (ie. context is bound to a "render tree" produced via a single React.render call, not to single components starting another render, or the DOM). The only reliable way to pass down context is via render() in components.
Two ideas to work around that:

Don't use <Link> to begin with, and create your own component which receives the router instance via props instead of context. Should be mostly copy & paste.

Create a wrapper for Link which receives the router instance via props and "injects" it into context for Link (<Wrapper><Link ... /></Wrapper>). The component would look similar to this one, except that you wouldn't need contextTypes, and in getChildContext you would only need to return  { router: this.props.router }.

You may also have to manually rerender those links on every route change, if you want to use active classes.

React.render(<Link to={mitems[item].route}>{mitems[item].text}</Link>, getAllElementsWithAttribute('menu-name',mop[i].getAttribute('menu-name'))[0]);
You're not going to get context there, is this code inside of react component that is inside of your app rendered in a Router.run call?  If so, you could make a context wrapping component that keeps the router in context for the link.
var App = React.createClass({
  contextTypes: {
    router: React.PropTypes.func
  },

  componentDidMount () {
    var router = this.context.router;
    var NewLink = React.createClass({
      childContextTypes: {
        router: React.PropTypes.func
      },
      getChildContext () {
        return {
          router: router
        }
      }
      render () {
        return <Link {...this.props}/>;
      }
    });
    fetchYourTemplateThing((tmp) => {
      // do your stuff
      React.render(<NewLink .../>, el);
    });
  }
});

I'm having a similar problem here. I built a custom stripped-down Modal component off of the Material UI Dialog component (I probably should've named it Dialog too). I'm passing in the bio template into the Modal along with a Link component that I want to use to close the modal by redirecting to another route (/about-us). We are doing this because we want our modals to be able to be deep-linked to via a route (we may not be doing this the most effective way - if so, let me know if you know of a better one). When trying to click the close button in the modal, we get the error that @sgehly is getting (router being undefined):

Here is the code for my Modal class:
import React, {Component} from 'react';
import ReactDOM from 'react-dom';
import styles from './Modal.scss';

class Modal extends Component {

    _open() {
        this._scrollY = window.scrollY;
        this._mount = document.getElementById('mount');
        this._modalContainer = document.createElement('div');

        var animationListener = () => {
            this._modalContainer.classList.add('modal-entered');
            this._modalContainer.classList.remove('modal-enter');

            this._mount.classList.add('hide');
            this._modalContainer.removeEventListener('animationend', animationListener);
        };

        this._modalContainer.addEventListener('animationend', animationListener);

        this._modalContainer.classList.add('modal');

        document.body.appendChild(this._modalContainer);

        this._modalContainer.classList.add('modal-enter');
    }

    _close() {
        var animationListener = () => {
            this._modalContainer.removeEventListener('animationend', animationListener, false);

            ReactDOM.unmountComponentAtNode(this._modalContainer);
            document.body.removeChild(this._modalContainer);
            this._modalContainer = null;
        };

        this._modalContainer.addEventListener('animationend', animationListener, false);

        this._mount.classList.remove('hide');
        window.scrollTo(0, this._scrollY);

        this._modalContainer.classList.add('modal-leave');
    }

    _render() {
        if(this._modalContainer) {
            let modal = (

                    {this.props.children}

            );

            ReactDOM.render(modal, this._modalContainer);
        }
    }

    componentDidUpdate() {
        var open = this.props.open;

        console.log(open);

        if (open && !this._modalContainer) {
            this._open();
        }

        if (!open && this._modalContainer) {
            this._close();
        }

        this._render();
    }

    render() {
        return null;
    }
}

export default Modal;

and the instance I'm passing in the link to:
import React, {Component} from 'react'
import Modal from '../Modal/Modal';
import Avatar from '../Avatar/Avatar';
import styles from './BioModal.scss';
import Clear from 'material-ui/lib/svg-icons/content/clear';
import { Link } from 'react-router';

class BioModal extends Component {
    render() {
        var bio = this.props.bio || {};

        var overview = bio.overview,
            description;

        if (overview) {
            let overviewMarkup = () => {
                return {
                    __html: overview
                }
            };

            description = (

            );
        }

        var bioTemplate = (

                    <Avatar className={styles.avatar} src={`/public/${bio.fullSize}.jpg`} alt={bio.name}/>

{bio.name}

                    { description }

        );

        return (
            <Modal open={this.props.open}>
                <Link to="/about-us">
                    <Clear className={styles.closeButton} color={'#333'}/>
                </Link>

                { bioTemplate }
            </Modal>
        )
    }
}

export default BioModal;

You can't just render modals like that into containers and preserve context. Use an existing modal library that handles this for you, or look into how to pass through context into subtrees.

@taion thanks, figured it out via the Context section on the React site. Thank you for pointing me in the right direction. That was something I was unaware of in React.

@dwilt Could you please share the final code with the context fix? :)

@horgen sorry man, I don't have access to the code anymore as this was over a year ago :(


Fetch sites, percentiles, and data

#14 #13 #15
Tested that these actually work, but didn't commit build or indicator files.
At first, I was getting an error with site_stats step because there was an issue with sc_retrieve. It threw an error about implicit targets that were in the visualize step, and I know those haven't really been touched. So to make sure that my code in this PR works, I did the following:

built just the sites.rds by running scipiper::scmake('1_fetch/out/sites.rds.ind', remake_file = '1_fetch.yml')
replaced the sites <- readRDS(scipiper::sc_retrieve(sites_ind)) lines in fetch_site_stats and fetch_dv_data with sites <- readRDS('1_fetch/out/sites.rds')
built the last two fetch steps with scipiper::scmake(remake_file = '1_fetch.yml')

Tested that these actually work, but didn't commit build or indicator files.
In general, if you modify Drive, it's a good practice to commit build and indicator files, even if you're just testing.

But if you really want to "just test", consider setting options(scipiper.dry_put=TRUE) - then Drive isn't affected and soyou don't need to commit build/indicator files.

I forgot about the options(scipiper.dry_put=TRUE) trick - thanks! I'm currently implementing your second suggestion, which we talked about at the meeting. It's out fetching all the dv data right now.


[Node.js] Browser does not launch and missing set breakpoint

VSCode Version: 0.10.12-alpha
OS Version: Windows 10

Steps to Reproduce:
Following #3960 test tour

Create Azure Node.js Express (JavaScript) Project
Set a breakpoint in app.js at the line (or app.ts):
var app = express()
•Set a breakpoint in routes\index.js (or routes\index.ts) at the line that begins with:
res.render('index'
F5 or start debugging

Actual:

Does not launch the browser
Second break point is not hit

Launching the browser is not expected, we never open the browser for the user.
As for the breakpoint not hit - forwarding to @weinand

@vsccarl thanks for trying to go through the #3960 tour (which was based on some Visual Studio tour and is not a good fit for VS Code).
I've verified that hitting a breakpoint in app.js on line var app = express() and in routes\index.js works fine.
For the TypeScript additional setup was necessary because the template does not include any support for transpiling ts to js and producing source maps. After this done, breakpoints were working fine.
/cc @egamma


Linkchecker fails to check some links

Go to website/content/docs/developers/data-model.md and change /api/com/twitter/heron/api/tuple/Tuple.html to /api/foobar/com/twitter/heron/api/tuple/Tuple.html and run 'make site'.
Linkchecker passes. It doesn't test the link. WHen I add -v to the linchecker command and comment out rm -f $OUT_FILE from website/scripts/linckchecher.sh you see that the link is not checked because it appears external:
http://twitter.github.io/heron/api/foobar/com/twitter/heron/api/tuple/Tuple.html;
file:///Users/billg/ws/git/heron/website/public/docs/developers/data-model/index.html;;
filtered;;
The URL is outside of the domain filter, checked only syntax.;
True;
http://twitter.github.io/heron/api/foobar/com/twitter/heron/api/tuple/Tuple.html;
14;82;"Javadoc

Ideally we could map twitter.github.io/heron in linkchecker.
cc/ @lucperkins

close. #797 and #841 will fix this.


Added ParisClasseNumerique support.

I'm adding support for ParisClasseNumerique ENT.
Run screenshot

(I'd also be interested in becoming maintainer because i'm using this library a lot. I'm now developing an ENT message getter)

Hi, sorry for the delayed merge. Would you mind adding me on discord? (bain(#)5038) Just to discuss you being a maintainer, potentially discuss other things about pronotepy :)

No problem for the merge.
I sent a friend request.


Warning message for possibly future failed transaction

Log a warning message if a transaction is broadcasted less than the amount creation fee to an account that this client believes doesn't exist right now.
The best place I think is the moment the transaction is added to the transaction mempool (from this node, not via gossip net). The warning message should be in the Warn log level and say something like:
"A transaction was submitted that is likely to fail to complete because the account creation fee of $account_creation_fee is not covered by the amount $amount and this account doesn't seem to be created already."

From an OOB conversation with @bkase:

[15:45] matthew: Should we throw an error in these in GraphQL too, and require setting an additional force parameter to let it go through?
[15:46] matthew: I'd prefer to do the force thing for rosetta too, but that may violate the API design
...
[16:22] bkase: yes let's do it into compatible
[16:22] bkase: feel free to comment on that issue and we can point Omer at it
[16:23] bkase: after he does the initial logging part (that can be a hotfix since it doesn't mess with control flow)

At a high level then

we should throw an error in these in GraphQL, and require setting an additional force parameter to let it go through
this additional change should go in compatible.

^^ this change should be in a separate follow-up PR afterwards

Remaining work to be done: update graphql + CLI to allow ignore these warnings, see stale work in #8311


Possible support of L1SLOAD (RIP-7728)?

Are you interested in RIP-7728 support? I find this one perspective.
https://github.com/0xurb/l1sload - example implementation on revm (release/v45).
When optimism flow will be finished, we can do more exciting precompiles, that use state and L1 context.
Risks and open questions:
Can we require L2 nodes to rely on L1 archive nodes?

[ ] TBD

How to deal with RPC errors?

[ ] TBD

Impact of RPC latency on sequencing? How do we price this in gas?

[x] some latency (tentative) cost for used gas calculation is included per slot

Can we guarantee that the accessed L1 state is recent?

[ ] TBD

Should we allow state reads at arbitrary L1 block height?

[x] state has an access through L1Block.sol. And if EVM context storage is returns error, we can take BlockId::Latest on RPC calls.

Yeah, it's definitely something we're interested in exploring:)


Add new RSpec/IdenticalEqualityAssertion cop

Closes #1130

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Updated documentation.
[x] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

If you have created a new cop:

[x] Added the new cop to config/default.yml.
[x] The cop is configured as Enabled: pending in config/default.yml.
[x] The cop documents examples of good and bad code.
[x] The tests assert both that bad code is reported and that good code is not reported.
[x] Set VersionAdded in default/config.yml to the next minor version.

@pirj I ran this cop on real-world-rspec and found these offenses:
https://gist.github.com/tejasbubane/e64636d7c7dd97c4b056a09b09a6c61b

@tejasbubane 99% look completely legit!
expect(true).to eq(true)

:laughing:
Some are (rspec-core) testing the RSpec itself, but it's ok to flag them:
it("equality") { expect(1).to eq(1) }

The only one that is probably a false offence:
expect(fqdn_rand(3)).to eql(fqdn_rand(3))

Thank you!
BTW, did you feel the pain of removing .rubocop YAML files from that repo? :smile:

BTW, did you feel the pain of removing .rubocop YAML files from that repo?

No I ran with --only RSpec/IdenticalEqualityAssertion :)

@bquorning Thanks. Fixed as per your comments - except one.

ran this cop on real-world-rspec and found these offenses:

BTW, did you feel the pain of removing .rubocop YAML files from that repo?

No I ran with --only RSpec/IdenticalEqualityAssertion :)

Do you mind sharing how you do it, @tejasbubane ?
~/source/rubocop-rspec $ rubocop --only RSpec/Rails/AvoidSetupHook ../real-world-rspec/
Error: RuboCop found unsupported Ruby version 2.4 in `TargetRubyVersion` parameter (in /Users/pirj/source/real-world-rspec/fat_free_crm/.rubocop.yml). 2.4-compatible analysis was dropped after version 1.12.
Supported versions: 2.5, 2.6, 2.7, 3.0, 3.1

same for real-world-rspec/camaleon-cms/.rubocop.yml
After removing them:
Error: The `Layout/AlignParameters` cop has been renamed to `Layout/ParameterAlignment`.
(obsolete configuration found in /Users/pirj/source/real-world-rspec/rspec-mocks/.rubocop_rspec_base.yml, please update it)
The `Layout/IndentArray` cop has been renamed to `Layout/FirstArrayElementIndentation`.
(obsolete configuration found in /Users/pirj/source/real-world-rspec/rspec-mocks/.rubocop_rspec_base.yml, please update it)
The `Layout/IndentAssignment` cop has been renamed to `Layout/AssignmentIndentation`.
(obsolete configuration found in /Users/pirj/source/real-world-rspec/rspec-mocks/.rubocop_rspec_base.yml, please update it)
The `Layout/IndentHeredoc` cop has been renamed to `Layout/HeredocIndentation`.
(obsolete configuration found in /Users/pirj/source/real-world-rspec/rspec-mocks/.rubocop_rspec_base.yml, please update it)
The `Style/TrailingCommaInLiteral` cop has been removed. Please use `Style/TrailingCommaInArrayLiteral` and/or `Style/TrailingCommaInHashLiteral` instead.
(obsolete configuration found in /Users/pirj/source/real-world-rspec/rspec-mocks/.rubocop_rspec_base.yml, please update it)
The `Style/MethodMissing` cop has been split into `Style/MethodMissingSuper` and `Style/MissingRespondToMissing`.
(obsolete configuration found in /Users/pirj/source/real-world-rspec/rspec-mocks/.rubocop_rspec_base.yml, please update it)

I have those one-liners handy:
fd --no-ignore --hidden 'rubocop*.yml' | xargs rm

and
for repo in $(fd --type d --maxdepth 1); do; (cd "$repo"; git reset --hard HEAD); done

But I'd rather run rubocop --only and skip those unrelated errors.

@pirj Not sure how I ran with --only back then, but this worked when I tried now:
$ cat Gemfile
source 'https://rubygems.org'

gem 'rubocop', path: '../rubocop'
gem 'rubocop-rspec', path: '../rubocop-rspec' # Contains newly built / unreleased cops

$ cat .rubocop.yml
require:
  - rubocop-rspec

AllCops:
  DisabledByDefault: true

RSpec/IdenticalEqualityAssertion:
  Enabled: true

$ bundle exec rubocop --config .rubocop.yml
.....
54137 files inspected, 115 offenses detected

Thanks, that works!


Icon should be 192px not 196px

R: @jeffposnick

:100: :nine: :two: :+1:

Wait. Don't pull this yet.
On Tue, 24 Nov 2015, 20:38 Jeffrey Posnick<EMAIL_ADDRESS>wrote:

[image: :100:] [image: :nine:] [image: :two:] [image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/google/WebFundamentals/pull/2306#issuecomment-159398331
.

:hourglass_flowing_sand: ing

Sorry, yes. 192 is correct based on 1.5 x 128 dp... Just double checked
base don't other doc as well.
Lgtm
On Tue, 24 Nov 2015, 20:39 Paul Kinlan<EMAIL_ADDRESS>wrote:

Wait. Don't pull this yet.
On Tue, 24 Nov 2015, 20:38 Jeffrey Posnick<EMAIL_ADDRESS>wrote:

[image: :100:] [image: :nine:] [image: :two:] [image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/google/WebFundamentals/pull/2306#issuecomment-159398331
.

Builds appear to be stalled....

Travis is back in action. Can you merge?


Use gopkg.in for imports

I suggest using gopkg.in when importing my packages (github.com/djherbis/pkg -> gopkg.in/djherbis/pkg.v1)
This will guarantee API stability.

Hi! Thanks for your libraries.
We will simply vendor all dependencies soon (i.e. as soon as I can fix gvt), so that won't be a problem.


Use system clipboard

Is there any way to use system clipboard on yanking?

Put the following line in your neovim config file: (mine is located at: ~/.config/nvim/init.vim)
set clipboard+=unnamedplus

@progmem64, this works, thank you!


WIP: Refactor system setting

This PR reduces the complexity of the system setting system.
It only needs one line to introduce new options, and the option can be used anywhere out-of-box.

Almost ready for review.


blue and blueBright problem

chalk.blue() and chalk.blueBright() on Windows 10 cmd console output the same color - dark blue.
chalk.green() and chalk.greenBright() behave as expected.

This is fixed in v3 which I just published: https://github.com/chalk/chalk/releases/tag/v3.0.0


finder.PublicKeyFromAddress should return PublicKey

Is your feature request related to a problem? Please describe.
The key of key is needed to return what encryption methods are allowed
Describe the solution you'd like
Extend public key finder to return PublicKey

[ ] In internal/mailbox/pubkey.go change PublicKeyFromAddress(ctx context.Context, protocol, network string, address []byte) ([]byte, error) to PublicKeyFromAddress(ctx context.Context, protocol, network string, address []byte) (cypto.PublicKey, error)
[ ] use "github.com/mailchain/mailchain/crypto" package
[ ] change internal/clients/etherscan/pubkey-finder.go to new signature. Use secpk256k1 package for PublicKey
[ ] change internal/protocols/substrate/pubkey.go to new signature. Use ed25519 package for PublicKey
[ ] run make generate to update mock
[ ] update tests

Additional context
Substrate will be extended in the future to support different key types

@robdefeo Can I have this assigned to me?


[CLN] Improve JS client async errors tests

Use the jest standard Jest way for testing for rejected promises: await expect(...).rejects.throw(new Error("expected message"));.
Before, some of the tests were not failing if an error is thrown -> the tests did not really test what was expected. Skipped one such test ("collection.get should error on empty embedding"). I will fix it in a next PR.

@ibratoev sorry this didnt get merged at the time, this looks like a very nice change. im guessing you arent up for the large merge to get this up to date... (but let me know if you are!) - so im going to merge it and create an issue that links to this.

Yeah, I kind of gave up waiting for this to be merged.


Update GHA badges in README

I simply copied over the markdown generated by Github to display the badges for the configlet and CI testing actions

Strangely the badges aren't showing up correctly in the preview although this worked in other repos.


added heap sort

heap sort
time complexity O(nlogn) ~ efficient sorting algorithm

please add hacktober fest label on this


Add volume and seek control to gpmdp

Description:
Add support for volume and seek control to Google Play Music Desktop Player.  Also do a better job when GPMDP is shutdown.

🐬


Fix several vanilla rendering issues with liquids

Possibly outwith the scope of this mod and worthy of a separate one, but since this deals with liquids it might be a good idea to bring this up anyway.

Fluids attempting to connect to other diagonally adjacent fluid blocks (MC-70848)

Fluids attempting to connect to other fluidlogged blocks where they cannot (MC-127030)

Water bulging in an arrangement of four stairs (MC-127270)

And maybe make fluid textures scale up depending on what direction they flow in (up to sqrt(2) for diagonally flowing) to soften the horrible grid lines you get especially on lava (MC-129207):

probably better to leave these kinds of rendering issues to sodium

Moved to https://github.com/CaffeineMC/sodium-fabric/issues/91 since it probably fits better there - feel free to reopen if you think this mod would be better for fixing it though


Update rubocop-hq org to rubocop org

Follow up to https://github.com/rubocop/rubocop/pull/9537.

The CI / Coding Style build error is not related to this change and will be resolved by #207.


Fix dysfunctional checking for ctags command variable

It was checking for the existence of a variable gutentags_ctags_executable, while the one that was being assigned and used was gutentags_executable. This last name is now used everywhere (also changed in the docs).

Good catch, but actually it's the other way around :)  I'm slowly making the ctags aspect of Gutentags into a module inside Gutentags, so I can gradually add support for cscope on the same level, instead of making it feel like something tacked on it. And hopefully more optional modules in the future.
I fixed it with bd044f85b392519362f87b515f1b027f647b79f4 so I'll close this pull request. Thanks though!

Great! Well, I knew it was 50-50 :)


DB become read-only after decrypt

unable to write the DB after decryption.  I able to read the DB after decrypt but the DB becomes read-only when I tried to write data to a table.
SQLCipherUtils.decrypt(mcontext, mcontext.getDatabasePath(AppConstants.DB_NAME).getAbsoluteFile(),passwordInCharArray);
Here is the error message i get
android.database.sqlite.SQLiteReadOnlyDatabaseException: attempt to write a readonly database (code 1032)
#################################################################
Error Code : 1032 (SQLITE_READONLY_DBMOVED)
Caused By : Database or Journal file have been removed.
	(attempt to write a readonly database (code 1032))
#################################################################

See this test case for how I am testing the current functionality.
In order to try to address your problem, I would need either:

Details about what is different between your scenario and that test case, or
A test case that can reproduce the problem, or
A sample project that can reproduce the problem

Without any of those, I have nothing to go on in order to investigate what is causing your difficulty.

thanks for your quick response, I have fixed it by reopening the database.


[ISSUE #3709]Resolve export metadata errors while multiple brokers.

Make sure set the target branch to develop
What is the purpose of the change
#3709
When there are multiple brokers in the cluster, export the metadata file using -c, prefixed with the broker name.

Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.04%) to 53.229% when pulling 7cda63f9ec091f0b491fc0a19bf0b4bc4698c804 on zhangjidi2016:fix_exportMetadata into ed5f4e4eac71849bc479aee2db7cc931d262a184 on apache:develop.


(Property ‘FirebasePlugin’ does not exist on type ‘Window’)

I watched the steps exactly but I get an error.
app.component.ts
  import { Component } from '@angular/core';
  import { Platform } from 'ionic-angular';
  import { StatusBar } from '@ionic-native/status-bar';
  import { SplashScreen } from '@ionic-native/splash-screen';

  import { HomePage } from '../pages/home/home';

 @Component({
  templateUrl: 'app.html'
 })

 export class MyApp {
    rootPage:any = HomePage;

   constructor(platform: Platform, statusBar: StatusBar, splashScreen: SplashScreen) {
     platform.ready().then(() => {

      statusBar.styleDefault();
      splashScreen.hide();

       window.FirebasePlugin.getToken(function(token) {
      // save this server-side and use it to push notifications to this device
         console.log(token);
        }, function(error) {
       console.error(error);
     });

       Window.FirebasePlugin.onNotificationOpen(function(notification) {
         console.log(notification);
      }, function(error) {
      console.error(error);
     });

      });

      }
   }

`

do not use capital W

Same here, was this solved?

I solved using another plugin : cordova-plugin-fcm

Caused by using TypeScript with out the proper types. You can get around this by doing:
const plugin = (<any>window).FirebasePlugin;
// or
const plugin = (window as any).FirebasePlugin;

I solved using another plugin : cordova-plugin-fcm

FCM breaks on Android 7.1.1 for Ionic. @rwoody solution works.

Caused by using TypeScript with out the proper types. You can get around this by doing:
const plugin = (<any>window).FirebasePlugin;
// or
const plugin = (window as any).FirebasePlugin;

thank Mr.


#130 | Ziyang | Fix wrong status shown in search result

Closes #130.
Screenshot

IBM Cloud toolchain: Delivery Pipeline deployed promotions to prod, including commit 33e3efa211d3843bfb34058ba61acc9691f41713


Set default toolchain version correctly when installing from our copy of rustup-init.sh

Resolves https://github.com/travis-ci/travis-ci/issues/7510.

Tested: https://staging.travis-ci.org/BanzaiMan/travis_staging_test/builds/555372#L170-L171


Write wrapper script to retrieve any available source of funding

It would be nice to have a wrapper script that would retrieve any available funding data from our various source-specific scripts for a given repo or project name.

Agreed! I certainly thought this is where we are heading :)


Support OpenAPI 3

JSON and YAML format

Requires https://github.com/apache/camel-k-runtime/issues/232


BAP Remove direction from codec

(Reopen of https://github.com/zephyrproject-rtos/zephyr/pull/42446 due to GH bug)
Direction (dir) field has been moved to the endpoint instead. This cleans up a significant amount of functions and macros that otherwise determined the direction in other ways.
The purpose is to avoid bidirectional audio streams as they are not defined by the BAP spec (audio streams are only unidirectional).
fixes https://github.com/zephyrproject-rtos/zephyr/issues/41194

Difference between this PR and the old one is the 4th commit


Suppor for display of image analysis results

You may want to rebase before you do the PR, I think the switch in project namespaces messed up the commit log a little bit, and that might be why you had issues in the previous PR. Something similar happened to me in the other project and it appeared to be fixed after a rebase, even if it is a rebase onto the original commit that it branched from in the first place.


New version: JohannesMillan.superProductivity version 7.13.1

Result: Installation Failed

Manifest
Add/Remove Programs

Name
superProductivity

Version
7.13.1

Publisher
Johannes Millan

ProductCode

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 6457
Microsoft Reviewers: Open in CodeFlow

Hi @vedantmgoyal2009,
This installer doesn't actually install the package:


Upgrade mysql

Signed-off-by: kuba--<EMAIL_ADDRESS>
Upgrade go-mysql-server to the latest version da01e280582214ca93e769d49d271536aa56a0d3
It fixes #349

Btw. I've added simple script (so far can handy) to upgrade a revision, because managing secret knowledge about "some files with SHA1 references" is not what I'd like to know :)
For instance to upgrade mysql, I've just run:
./rev-upgrade.sh 24a5925748096dd3116b0073833a438ecb0c34f9 da01e280582214ca93e769d49d271536aa56a0d3
where 24a5925748096dd3116b0073833a438ecb0c34f9 is a old sha and ``da01e280582214ca93e769d49d271536aa56a0d3` is a new sha.
The script is not a magic. It replaces one sha by another and if everything is ok, run dep ensure.

I've improved the script. Now it takes a project's path and new revision.:
./rev-upgrade.sh "gopkg.in/src-d/go-mysql-server.v0" "dcc1c537eb7e2bc20ce69d03732e8aa8feaa8612"
It parses Gopkg.lock to get the old revision for a given project and replaces old by new (like before).


⚠ Bofrost.fr has degraded performance

In 9931d07, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1954 ms

Resolved: Bofrost.fr performance has improved in 06fc68b after 21 minutes.


deploy container service return error

Describe the bug

Steps to reproduce

create a template with https://github.com/walrus-catalog-sandbox/terraform-kubernetes-containerservice commit
create a service with just input image: nginx
get error below

Error: Attempt to get attribute from null value

  on .terraform/modules/c1/variables.tf line 255, in variable "containers":
 255:     condition     = alltrue([for c in var.containers : c.profile == "" || contains(["init", "run"], c.profile)])

This value is null, so it does not have any attributes.

Error: Attempt to get attribute from null value

  on .terraform/modules/c1/variables.tf line 255, in variable "containers":
 255:     condition     = alltrue([for c in var.containers : c.profile == "" || contains(["init", "run"], c.profile)])

This value is null, so it does not have any attributes.

Error: Attempt to get attribute from null value

  on .terraform/modules/c1/variables.tf line 261, in variable "containers":
 261:         for p in(c.ports != null ? c.ports : []) : (0 < p.internal && p.internal < 65536) && try(0 < p.external && p.external < 65536, true)

This value is null, so it does not have any attributes.

Expected behavior
Should success deploy
Environment

Walrus version:
https://github.com/seal-io/walrus/commit/abc217a55cf7a6c0c39d4f8dec61d32ced2436b7
Additional context
Request Payload
POST https://localhost:8447/v1/projects/488560421570084869/environments/488560958977867781/resources

{
    "projectID": "488560421570084869",
    "project": {
        "id": "488560421570084869"
    },
    "environment": {
        "id": "488560958977867781"
    },
    "description": "",
    "labels": {},
    "type": null,
    "name": "c1",
    "template": {
        "name": "terraform-kubernetes-containerservice",
        "version": "main",
        "id": "",
        "template": {
            "id": "488560908813991941"
        }
    },
    "attributes": {
        "containers": [
            {
                "profile": "run",
                "ports": [],
                "resources": {
                    "cpu": 0.25,
                    "memory": 256
                },
                "execute": {
                    "command": [],
                    "args": []
                },
                "envs": [],
                "files": [],
                "mounts": [],
                "checks": [],
                "image": "nginx"
            },
            null
        ],
        "context": {},
        "infrastructure": {
            "domain_suffix": "cluster.local",
            "gpu_vendor": "nvidia.com"
        },
        "deployment": {
            "replicas": 1,
            "rolling": {
                "max_surge": 0.25,
                "max_unavailable": 0.25
            },
            "timeout": 300,
            "sysctls": []
        }
    }
}

Probably the same as https://github.com/seal-io/walrus/issues/1542

Verified could success deploy with UI dev-3e67638


[ZEPPELIN-955] Redirect docs/0.6.0-incubating-SNAPSHOT to docs/0.6.0-SNAPSHOT/

What is this PR for?
Redirect docs/0.6.0-incubating-SNAPSHOT to docs/0.6.0-SNAPSHOT/
What type of PR is it?
Improvement
Todos

[x] - Create .htaccess with rewirte rule

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-955
How should this be tested?
Screenshots (if appropriate)
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

LGTM

merge if there're no more discussions.


fix: s2-vue 保持和react组件一致的tooltip summary渲染逻辑

👀 PR includes

✨ Feature

[ ] New feature

🎨 Enhance

[ ] Code style optimization
[ ] Refactoring
[ ] Change the UI
[ ] Improve the performance
[ ] Type optimization

🐛 Bugfix

[x] Solve the issue and close #1575

🔧 Chore

[ ] Test case
[ ] Docs / demos update
[ ] CI / workflow
[ ] Release version
[ ] Other ()

📝 Description
s2-vue 明细表圈选多个 dataCell 不应该显示总和
🖼 Screenshot

Before
After

❌
✅

🔗 Related issue link

🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[x] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-vue-v1.2.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-vue-v1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Docs] Add documentation for merge.conf

Describe the enhancement requested

Add documentation for usage of merge.conf used by the PR merge script

Component(s)
Developer Tools

Issue resolved by pull request 41621
https://github.com/apache/arrow/pull/41621


fix(@angular/cli): enable full node module resolution for project deps

As discussed in issue #6248.
Closes #6248

I've validated this fix for my use cases, and it looks great! Thanks for your help @clydin.


resource filters: allow blacklist and tag filtering

What does this PR do?
A requested feature for this integration is the ability to filter on tags coming from vSphere.
This PR gives users the ability to filter on any tag on a whitelist and/or blacklist way.
Having the ability to use blacklist is a requirement for tags, indeed, at the contrary of other filters, a resource can have multiple tags. So it's not possible to rely on negative matching regexes.
example: If you need to match all resources with role:database but you want to ignore those with env:staging, there is no other way than having whitelist and blacklist.
Motivation
Custom request.
Additional Notes
I also reversed the meaning of the is_resource_excluded_by_filters method as it would be overly complex to keep that negation with the addition of blacklists.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Great ideas, I've updated, let me know how that works 👍


added support for double-tap-hold (aka "one-finger pinch") zoom

like in google maps.

flutter rest actually passes on my machine, not sure why did it fail on travis..

@johnpryan ? :)

can you update with master and run dartfmt?

@johnpryan , done

@johnpryan merged with latest changes

@johnpryan merged with master again, is there something wrong - conceptually - with this PR?


execute uninstall didn't remove anes from the description's file

Hi, I'm trying to uninstall the facebook ane by calling uninstall, and here's the output:
$ apm uninstall com.distriqt.facebook.Core
Uninstall package : com.distriqt.facebook.Core
✓ Removed package : com.distriqt.facebook.Core
Uninstall package : com.jetbrains.kotlin
✓ Removed package : com.jetbrains.kotlin
Uninstall package : com.android.installreferrer
✓ Removed package : com.android.installreferrer
Uninstall package : androidx.core
androidx.core :: Required by another package - skipping uninstall
Uninstall package : com.distriqt.Bolts
✓ Removed package : com.distriqt.Bolts
Uninstall package : com.distriqt.Core
com.distriqt.Core :: Required by another package - skipping uninstall

I tried to regenerate the description file and these anes are also listed in the extensions section.

Thanks for reporting this, I'll look into it.
What version are you using?

I'm using 1.1.1-alpha

This is happening due to the logic around the extensions node when generating the app xml. Currently we take the approach that we only add extensions to this list (never remove) when generating the app xml, as the extension may be a local / external / custom extension that wasn't delivered through apm.
This is one of the concerns we had originally around the app xml generation about what to use as the "template".
I think to resolve this one the simplest approach is to pass the app xml into the uninstall command, eg:
apm uninstall com.distriqt.facebook.Core src/MyApp-app.xml

So at uninstall time we can process the app xml as well.
The more involved resolution would be to change the logic behind the extensions node generation to remove any extensions not installed via apm. For this to work we would have to add a config for other non-airpackage extensions.

The original idea here was to use the config/application-descriptor.xml as the base template and only have the external extensions in there. So what you can try is copying your app xml to this location, removing the android / iphone nodes and just having the base AIR content in there. If you have any local extensions, add them into the extensions node but not any that are installed via apm. This way when a package is uninstalled it would no longer be added into the generated descriptor.
But the problem we saw with this (and why we have not encouraged this approach) is the issue with maintaining 2 app descriptors. The one that you use to generate your app, and are probably working on eg MyApp-app.xml, and the one that apm uses to generate this file. If you make changes to MyApp-app.xml but not the config//... then apm could overwrite your changes.

Potentially we should implement both of the solutions.

Actually I've come up with a better approach using the new "lock" file which should be seamless for the user. Will release this with the 1.1 update later today.


fix grape-entity repo link

No need to CHANGELOG a text change ;)

Removed changelog entry.

@hedgesky … thanks


Support synchronous reactions

This change allows for firing on the main thread to act synchronously. This helps avoid a number of race conditions and other problematic behavior that occurs when mixing inherently synchronous work (the main thread) with asynchronous behavior.
Note: This is potentially a breaking change, as people may have logic that does not work properly once the reactions are synchronous and serial. Those were probably originally bugs anyway, but will become more evident with this change.

Both this change and #17 are addressing race conditions. Could you perhaps try your situations with only the change from that PR and see if it addresses your concerns? If so, we'll get that one in. If not, let's chat offline for more detail.

I did try that first, and my issues persisted. Let's talk more if needed.

Closing for now, in favor of #20.


Add snapshot testing to CLI & set up AWS mock

Which issue does this PR close?

Related to https://github.com/apache/datafusion/issues/13456#issuecomment-2515916144
Rationale for this change
We currently don't test whether our external integrations actually work: as a result https://github.com/apache/datafusion/pull/13576 has happened.
What changes are included in this PR?
Integration tests for S3.
Are there any user-facing changes?
No.

The test is not run automatically in CI and thus someone has to remember to run it. I predict this means it sill slowly bit rot (aka not get run and break and no one will notice)

I believe it does work in the CI (hence so many commits in this PR to make it work 😀)
https://github.com/apache/datafusion/actions/runs/12435578988/job/34722247404?pr=13672


Proposal: Convert uProxy to Typescript ∩ AtScript ∩ Flow

There are now at least 3 major efforts designed to provide type-checking of javascript, together with support for ECMAScript 6 features (like classes and arrow-functions), in a way that works with existing browsers: Typescript (from Microsoft), AtScript (from Google), and Flow (from Facebook).  All 3 use nearly-identical syntax and highly similar type checking.  (This is not a coincidence, as all are inspired by the Harmony/ES6 and ES4 standardized languages.)
The future of these three systems is unclear.  They are similar enough that it is difficult to tell them apart, but not so similar that one can simply take a program written for one and use it in the other.  For example, Typescript has enum, whereas the other two do not.
The maintainers of these languages have made noises about converging on a common standard.  For example:

We're also working with the Flow and Angular teams to make sure that TypeScript is the best language for working with a broader range of libraries, including declarative frameworks like the upcoming Angular 2.0 release.

Given the residual incompatibilities, it seems likely that existing (e.g.) Typescript code may not be compatible with the ultimate standard.  However, I think code written within the common core of all three languages will likely be future-proof, or nearly so.
In addition to future-proofing, writing in the intersection also confers some immediate advantages, because the three tools have different emphases:

Typescript focuses on IDE support and readable error reporting for dense type annotations.
AtScript focuses on runtime type checking, for properties that cannot be verified at compile time.
Flow focuses on type-tracing and propagation, for sparse type annotations.

Using all three would improve the validation of our code, both at compile time and at runtime.
No special effort would be required for us to identify the language-subset that is in the intersection of these three.  We would simply run all three tools on our code.  The transition could be made gradually, by enabling new typechecking tools but making their output advisory-only until all major errors are resolved.  (Indeed, even our use of Typescript's typechecking is currently only advisory.)
Possible roadblocks:

Flow has limited support for some advanced ES6 features like const

but they're working on it

Need to replace enum
Typescript still has limited support for some advanced ES6 features (like Symbol, which could replace enum)

but they're working on it

The syntax for modules (i.e. require) would also have to be sufficiently compatible across all three tools.

Implementation note: AtScript is designed as an extension of Traceur, an ES6-to-ES5 compiler.  See also: http://kangax.github.io/compat-table/es6/

I've been advised that it's too early for this, and instead we should stick to Typescript and wait for the language vendors to sort out a roadmap.


README: Document use with Eldev

As tentatively agreed in discussion for issue #55.
You can see a working example of it being used here: https://github.com/doublep/iter2

I could have sworn I had merged this. Thanks!


User story 3.2

This PR introduces the functionality that allows inventory managers to modify the user information of stock-keepers as described in user story 3.2 (#50)
Inventory managers can go to the employees tab and select a stock keeper to view their information and edit it.
Please note that the id, email, and role fields for a stock keeper can not be changed by an inventory manager.
Once the edited information is saved, the new user information will be displayed under the stock keeper's profile.

Codecov Report

Merging #203 (cc4a7f9) into master (f7d449f) will decrease coverage by 0.04%.
The diff coverage is 93.87%.

@@            Coverage Diff             @@
##           master     #203      +/-   ##
==========================================
- Coverage   93.56%   93.51%   -0.05%
==========================================
  Files          35       35
  Lines         823      863      +40
==========================================
+ Hits          770      807      +37
- Misses         53       56       +3

Impacted Files
Coverage Δ

server/user_account/permissions.py
92.30% <90.32%> (-1.31%)
:arrow_down:

server/user_account/tests.py
100.00% <100.00%> (ø)

server/user_account/views.py
88.42% <100.00%> (+0.29%)
: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 f7d449f...cc4a7f9. Read the comment docs.

Codecov Report

Merging #203 (cc4a7f9) into master (f7d449f) will decrease coverage by 0.04%.
The diff coverage is 93.87%.

@@            Coverage Diff             @@
##           master     #203      +/-   ##
==========================================
- Coverage   93.56%   93.51%   -0.05%
==========================================
  Files          35       35
  Lines         823      863      +40
==========================================
+ Hits          770      807      +37
- Misses         53       56       +3

Impacted Files
Coverage Δ

server/user_account/permissions.py
92.30% <90.32%> (-1.31%)
:arrow_down:

server/user_account/tests.py
100.00% <100.00%> (ø)

server/user_account/views.py
88.42% <100.00%> (+0.29%)
: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 f7d449f...cc4a7f9. Read the comment docs.


update-makefile-dont-buf-every-time

I noticed after the update to streamline CMake & the Makefile that running any make command that depends on buf runs make buf every time, which results in all the GRPC C++ classes being compiled every time, which dramatically slows down the workflow of incremental C++ builds (when doing TDD for example).
This PR makes it so that buf (now called grpc/buf will touch a file called grpc/buf once it succeeds which should make it so that this commend doesn't continue to get run once it succeeds once.

Changes look fine, but just want to confirm there isn't something else going on.
When I originally switched this, it was to make sure things never got built with out-of-date generated files, which has/had been a big problem in the past. In timing it, make buf (on a second run) takes 0.82 seconds on my desktop, and 1.35 seconds on my macbook. You said it dramatically slows things, so I'm curious if you're seeing different behavior than I am, or if you're just doing something different where 1.4 seconds is too long. I'd definitely assumed a 1% slowdown was acceptable.

Its not that the buf step is slow. It's that it generates C++ source files that then need to be recompiled, even if the file contents have not changed. I'll take timings on how much time that takes & get back to you.

Jinx! :-)


ograc: init at 0.1.6

Description of changes
init ograc
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

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

1 package built:

ograc

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/803


Tooltip with corresponding colours

Decided it'd be great with the tooltip colours to correspond with the colours of the icons :partying_face: Probably need another colour for the yellow-500, aka. Settings. Anyways, I'm pleased with the result! :parachute:
Please, by any means, provide criticism! I'm more than happy to re-do this in a more coolified-edition!
PS. I'm sorry about lines 347, 362 & 407. I tried to stop the Tailwind extension from re-ordering, which succeeded for most the stuff, but not these.
PSS. I've marked this as ready for review, even though the CONTRIBUTING.md told me not to, but it seems incomplete.

Thanks for the PR! It looks awesome! :)


error while running command: noteVault vault should exist

Please select the area your bug applies to. (Multiple selections are Possible)

[ ] Onboard. Initial vault setup and import/export pods
[ ] Create. Note creation, lookup, snippets and templates
[ ] Retrieve. Backlinks, references, graph view
[ ] Structure. Refactoring, multi-vault and schemas
[X] Publish. Sharing your repo with the world
[ ] Misc (Choose this if your not sure)

Describe the bug
error while running command: noteVault vault should exist.
To Reproduce
Steps to reproduce the behavior:

Dendron: Site Build

Expected behavior
Site is built at x location
Screenshots

Desktop (please complete the following information)

OS: Windows 10
Browser: chrome

Additional context
Add any other context about the problem here.
Dendron Log file
Please attach the output of > Dev:Dendron: Open Logs her
dendron.log

Hi @Sameep1234, can you try the resolution here? https://github.com/dendronhq/dendron/issues/1600#issuecomment-954618991

Hi @Sameep1234, can you try the resolution here? #1600 (comment)

Yes, I tried that but the error still persists. Any alternate solutions?

Hi @Sameep1234, can you try the resolution here? #1600 (comment)

Sorry for the inconvenience but it worked. I was installing it in the wrong directory. Thank you for helping out.


vJoy failed to install on Windows11

vJoy kept failed to install on my device even the latest release. Please help me! Here is some detailed information about my device.
版本	Windows 11 专业工作站版
版本	21H2
安装日期	2021/8/17
操作系统版本	22000.194
体验	Windows 功能体验包 1000.22000.194.0

hi for me same vjoy not working since windows 11 update and i dont know how to fix the problem.Anyone know how to fix it Thx?

I had had enough of bad experience on Windows 11, so I just downgraded to Windows 10. :tada: Problem solved!

any fix?

I'm really keen for a fix to this too. njz3 or anyone else, is someone interested in taking a look at this? Thanks :)

@NPU-Franklin I've just seen that the author is not maintaining the project anymore.
I used this fork https://github.com/njz3/vJoy and installed it successfully under Windows 11

Did you have to do anything special to get this working?

Did anyone else get the vJoySetup-2.2.1-signed.exe to work for them on Windows 11? It doesn't seem to be working for me.

No it isn't working, the install fail, it is unable to create the devices.

Awesome, <IP_ADDRESS> by jshafer817 worked for me too! Thanks!

Sweet, don't want to buy a new signing certificate and go through all that again, but I will if needed. 2.2.1 has some newer force feedback features.. Hmmm

Spent some hours yesterday to dig into the Windows 11 issue for v2.2.1, and can confirm it is related to the driver itself not being loaded by windows 11 (while it works on windows 10), and not the installer process.
During the install phase, vjoy installer asks to load the driver vjoy.sys file after copying all the file. The -5 errors come from Windows 11 indicating the file is either not loadable (driver compatibilty issue), or there is not enough memory to load it (?!?).
Sadly, I cannot test on a computer equiped with Win 11 and test-signing enabled to confirm where the driver issue is located.

vJoy <IP_ADDRESS> by jshafer817 installed correctly on Windows 11, and works well.

@MichalBerner https://github.com/jshafer817/vJoy/releases/tag/v<IP_ADDRESS> That is signed for Windows 10, but people say it works with Windows 11. Not sure about @njz3 status is on his version as his supports more force feedback stuff.

You are right. Version <IP_ADDRESS> works with Windows 11. I can confirm.
I should be more precise with my question.
What I meant was if there is a plan to release the signed version of vJoy driver with extended FFB support (<IP_ADDRESS>).

@MichalBerner I was going to renew my signing cert, I even paid the money. But then they wanted me to "revalidate" which requires letters from an attorney or etc. It was a bit of a pain the first time around and so I decided not to renew. As long as <IP_ADDRESS> is working in Windows 11 I am happy. But when vjoy broke in Windows 10 1903, I put down like $400 and a friend pitched in $300. I think this is why the original developer stopped messing with vjoy as he stated he didn't want to mess with the certificate requirements and I totally get that.

totally get this..suck a shame great little app..thanks for all the efforts!...used this for Oculus and pinball VR...doesnt work for me now on windows 11....
anyone got it working or another way to Oculus controllers please share....

As of July 25 of 2022, is not working, on Windows 11 says:

A lot of the issue is signing the program again. I don't have to resign my fork, but the njz3 version needs to be signed, although disabling driver enforcement should work without having to sign the program. Signing costs $700.... Want to spend $700??

Before even trying to sign it, I need to found the explanation why the driver does not load. Unfortunately I don't have a second computer with win11 to debug this kernel loading issue.

I do get it, thanks for the info.

I tried compiling my fork with Visual Studio 2022 and the latest WDK and
SDK and.... no dice... same error as you.
On Mon, Jul 25, 2022 at 3:39 PM njz3 @.***> wrote:

Before even trying to sign it, I need to found the explanation why the
driver does not load. Unfortunately I don't have a second computer with
win11 to debug this kernel loading issue.
—
Reply to this email directly, view it on GitHub
https://github.com/shauleiz/vJoy/issues/57#issuecomment-1194599983, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6CD2LKPKMXTD7PZUSJWU3VV33OJANCNFSM5FEDTUBQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Justin Shafer
Onsite Dental Systems
7704 Sagebrush Ct. S.
North Richland Hills, TX. 76182
(817) 909-4222

Just confirming & updating date, loaded & running in Windows 11
OS Name Microsoft Windows 11 Pro
Version	10.0.22621 Build 22621
side note MOST pass issue have been with Windows HOME version
I surgest NEVER get the HOME version as I ALWAY had my hands tied when dabbling with things.
Will try the other version too.

As of July 25 of 2022, the v<IP_ADDRESS> version is working, on Windows 11 (I'm using the Windows 11 Version 21H2 Build 22000.795)
I Would love to see the @njz3 version working that has great improvements (https://github.com/njz3/vJoy/releases/tag/v<IP_ADDRESS>) But that gives an error on installing the driver. Even if is a paid software, I would pay for it...
Or even @ianb888 with his latest fork.

https://github.com/njz3/vJoy/issues/9 the workaround for installing v2.2.1 can be found here. I just went through it and it installed so now to check it out.

ok so it installed but I can't get it to do anything. I run Vjoy configure and it runs but when I click add device I get nothing, I can check the box that say enable vjoy but the apply button is greyed out. Thoughts??

Just tried the workaround and even if the installer succeed, the driver is actually not loaded as nothing appears in the device manager of Windows.
Faking the reg HID seems to make the installer beleived it was loaded (the registry keys appears when the driver loads), while actually it did not.

Awesome, <IP_ADDRESS> by jshafer817 worked for me too! Thanks!

We still don't really know why......
On Thu, May 11, 2023 at 1:22 AM HDXUT @.***> wrote:

Awesome, <IP_ADDRESS> by jshafer817 worked for me too! Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/shauleiz/vJoy/issues/57#issuecomment-1543404160, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6CD2I45PFVA62QH6F7DDLXFSARTANCNFSM5FEDTUBQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Justin Shafer
Onsite Dental Systems
7704 Sagebrush Ct. S.
North Richland Hills, TX. 76182
(817) 909-4222

No it isn't working, the install fail, it is unable to create the devices.
But vJoy <IP_ADDRESS> from jshafer817 seems to work

here we are in 2023, this works as of 08/08/2023. thanks.

And none of us really know why... because I signed both projects.... One was compiled in a newer SDK and Visual Studio. The other was signed with the same exact stuff as the original, with one additional commit. Weird.

And none of us really know why... because I signed both projects.... One was compiled in a newer SDK and Visual Studio. The other was signed with the same exact stuff as the original, with one additional commit. Weird.

Hello there.
What I had to do in order to get vjoy to download ( I am on W11 ) I had to download the Version- <IP_ADDRESS> in order for it to work as none of the other versions seemed to want to download.

Ok, good news.
After many months (years?), I finally setup a windows 11 computer for kernel debugging to see what happens .
This required me to compile and install vjoy with testsigning enabled (using a test certificate), using VS2022 and latest WDK available to date (10.0.22621.0).
The dowside of this upgrade is that win32 is not accepted anymore, only x64 is available. So OS Windows target version must be >10 (ie windows 10, windows 11, and possibly future major OS).
So, after debugging step by step, I discovered that the main loading issue is related to this function DriverEntry function, where a piece of code is called at the wrong time:

Basically, we ask for a memory allocation, while driver memory pool is not yet created...
So I am happy to say that we probably have a fix for vjoy 2.2.x

Great work man!!!!

The fixed version also installs perfectly on win10

Good morning,
thanks for your work.
I have win 11, I can install Vjoy 2.2.1 using registry patch but when I start Vjoy everything is disabled. Will there be patches in the future?
Thanks again

Current v2.2.1 cannot be installed on Win11 as there is a bug inside the code that does the initialization of the driver.
The fix is in the new source code, but not published as an installer as I do not have the necessary certificate to sign the driver.

Here is a download link to the TESTSIGN binary version v<IP_ADDRESS> for feedbacks.
https://drive.google.com/file/d/1Vu2JICPl9xMlvjQvZLP05dBodJmepk6o/view?usp=sharing
To use it, first enable TESTSIGN mode on your windows computer by entering Bcdedit.exe -set TESTSIGNING OFF in an elevated command prompt.
https://learn.microsoft.com/en-us/windows-hardware/drivers/install/the-testsigning-boot-configuration-option
Then download my test certificate that you can download from here:
https://drive.google.com/file/d/1QAWUyx7mptCcIt99xyQ3mjhte9KgxxDY/view?usp=sharing
Install the certificate by double-clicking on it and select the "Trusted Root Certification" store.
https://learn.microsoft.com/en-us/windows-hardware/drivers/install/installing-test-certificates
Finally, proceed to vJoy installation as usual.

Great work!

great job, it installs.
However..
when I launch Force EmuWheel the program gives me this error :
[ERROR] vJoy driver version does not match the version of 'vJoyInterface.dll'[ERROR] Could not initialize vJoy.

The error is really self explaining....

yes, i figured it out even if i don't know anything about programming, but how to fix it?

EmuWheel software expect an old version of the driver. You have to recompile.

Here is another download link to the TESTSIGN binary version v<IP_ADDRESS> for feedbacks.
https://drive.google.com/file/d/1SEYtmJMelM_2xMC0SeO8neArsTy6_fh2/view?usp=drive_link
And the corresponding SDK:
https://drive.google.com/file/d/1r1MUxAatZxZW-fnKETdP7weKKwXJMGkL/view?usp=drive_link
It includes a fix for gamepad javacript API on browser (see https://github.com/njz3/vJoy/issues/12), by reducing the HID reported values of axes and continous hat switches to 16 bits instead of 32 bits.
This may break compatibility with existing feeder software (to be confirmed).

I've tried <IP_ADDRESS>, as well as the two testsign builds by njz3. I can get them all to install, kinda. They hang at the end. The kdmf installer is stuck. Once I close it it says its successful and will often work.
But as soon as I restart it is dead and never works again. I have to uninstall and reinstall.
What is it about a restart that causes it to completely stop working from that point on?

I've tried <IP_ADDRESS>, as well as the two testsign builds by njz3. I can get them all to install, kinda. They hang at the end. The kdmf installer is stuck. Once I close it it says its successful and will often work.
But as soon as I restart it is dead and never works again. I have to uninstall and reinstall.
What is it about a restart that causes it to completely stop working from that point on? I had it working for the last year or so, but it was flaky. Tried installing the njz3 versions above and now none of them seem to work.

Have you try my test builds with TESTSIGN mode enabled?
Please look here for step by step instructions :
https://github.com/njz3/vJoy/issues/12

@njz3 Although I got both the test signed builds you linked above to install on Windows 11, the configure utility works as expected, and the vjoy devices appear in control panel USB Controllers etc, they don't seem to work with UCR as the feeder. The vJoy device(s) appear as output devices and can be mapped to as expected, but no response in the vJoy monitor app. For reference, vJoy <IP_ADDRESS> from jshafer817 does work as expected.

Here is another download link to the TESTSIGN binary version v<IP_ADDRESS> for feedbacks. https://drive.google.com/file/d/1SEYtmJMelM_2xMC0SeO8neArsTy6_fh2/view?usp=drive_link And the corresponding SDK: https://drive.google.com/file/d/1r1MUxAatZxZW-fnKETdP7weKKwXJMGkL/view?usp=drive_link It includes a fix for gamepad javacript API on browser (see njz3#12), by reducing the HID reported values of axes and continous hat switches to 16 bits instead of 32 bits. This may break compatibility with existing feeder software (to be confirmed).

Hello ! I've tried <IP_ADDRESS> and it install correctly on latest Windows 11 with TESTSIGNED mode, but I use irFFB and in this mode IRacing prevent to start race with an anti-cheat and when disabling TESTSIGNED mode Windows disallow Vjoy driver to start due to not verified signature... Hope a fully signed version will drop ! :p

Hey folks
we decided to go through the signing process and just released a fully signed version in our fork here.
We just built the master branch which was already at version v<IP_ADDRESS> and tested it on several win10 and win11 machines with great success.
Hope this helps others and puts the great work of @njz3 (and others) to use :)

@BrunnerInnovation many thanks!

@BrunnerInnovation I just see that you have based your certified build on my master rep. Would it be possible to do another build integrating my latest fixes ? (those are mainly in this branch : https://github.com/njz3/vJoy/tree/dev/w7_pr3)
I can perform the merge to master today.

@njz3  yes, I saw that you made fixes in another branch, but I had troubles building it. And since the master built just fine and no issues were found while testing I decided to just go with the current state of the master branch.
But sure, if you merge the changes we could do another run of the certification process and maybe release a v<IP_ADDRESS>?
The process of submitting the driver files in the Microsoft Partner backend isn't that complicated or time consuming now that we finally know how to do it :D

:) nice work
On Tue, Dec 19, 2023, 6:19 AM Brunner Elektronik AG <
@.***> wrote:

@njz3 https://github.com/njz3 yes, I saw that you made fixes in another
branch, but I had troubles building it. And since the master built just
fine and no issues were found while testing I decided to just go with the
current state of the master branch.
But sure, if you merge the changes we could do another run of the
certification process and maybe release a v<IP_ADDRESS>?
The process of submitting the driver files in the Microsoft Partner
backend isn't that complicated or time consuming now that we finally know
how to do it :D
—
Reply to this email directly, view it on GitHub
https://github.com/shauleiz/vJoy/issues/57#issuecomment-1862657748, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6CD2PHJGHFEF6EKF5U6ULYKGA55AVCNFSM5FEDTUB2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBWGI3DKNZXGQ4A
.
You are receiving this because you were mentioned.Message ID:
@.***>

@njz3 yes, I saw that you made fixes in another branch, but I had troubles building it. And since the master built just fine and no issues were found while testing I decided to just go with the current state of the master branch. But sure, if you merge the changes we could do another run of the certification process and maybe release a v<IP_ADDRESS>? The process of submitting the driver files in the Microsoft Partner backend isn't that complicated or time consuming now that we finally know how to do it :D

The explanation is that the "master" branch was updated to compile for w10/w11 only, while the /dev/w7_pr3 is meant to be compiled for w7 up to w11, requiring build tools older than the ones that are readily available from Microsoft when installing VS2022.

Ok I thought so... And I actually tried for several hours to get all required build tools, but with no luck.
And since those older platforms aren't relevant to us I stopped to spend time on it. It would of course be nice to have a repo still supporting all old platforms going forward, but I'm afraid the time and effort required isn't really justified. Retro gaming scene can still use the older releases available. Or what are your thoughts about that?

I only know some industrial platforms like Beckhoff and Siemens that were using win7 (mostly embedded versions WES7) where it could be relevant, but that's a really niche.
I will try to backport my most important changes to the master branch w10/w11 and let you know when done.

Hi,
I came across this while looking into https://sourceforge.net/projects/dxace/ which has a vjoy dependency.
I've previously avoided vjoy in win 11 for obvious reasons 😱
Any chance of an update on vjoy status with win11?
Is this V<IP_ADDRESS> https://github.com/njz3/vJoy/releases/tag/v<IP_ADDRESS> the definitive working win 11 version, or ?
Is it even worth the pain of trying to get vjoy working on win 11 atm?
Thanks! (also for the great work above, I read the thread and can feel your pain.)

Awesome, thanks!

No it isn't working, the install fail, it is unable to create the devices.
But vJoy <IP_ADDRESS> from jshafer817 seems to work

i Just logged in to say thank you.


add distiluse-base-multilingual-cased-v2 to leaderboard?

Hi mteb, follow your leaderboard I discorver this repo https://huggingface.co/sentence-transformers/distiluse-base-multilingual-cased-v2, which does really well on sentence similarity task.
I try to search to add model to your repo but have no idea where to add...

Someone already benchmarked universal-sentence-encoder-4 & added it (https://huggingface.co/spaces/mteb/leaderboard); I'd expect that one to be similar
You can do it like that person & benchmark it yourself and add the scores to a random repo on your account and they will show up on the leaderboard.

They are not the same. The universal-sentence-encoder-4 is focused only on English but distiluse-base-multilingual-cased-v2 is multilingual. The training process is different becasue distiluse use student/teacher schema and distiluse model is based on multilingual universal sentence encoder 4 which is not same as universal-sentence-encoder-4

And I was trying to benchmark distiluse-base-multilingual-cased-v2 but when I used this simple snippet
from mteb import MTEB
from sentence_transformers import SentenceTransformer

# Define the sentence-transformers model name
model_name = 'sentence-transformers/distiluse-base-multilingual-cased-v2'

model = SentenceTransformer(model_name)
evaluation = MTEB(task_types=['Classification', 'Clustering', 'PairClassification', 'Reranking', 'Retrieval', 'STS'])
results = evaluation.run(model, verbosity=2, output_folder=f'results/{model_name}')

on our university cluster I got an segmentation fault error:
OpenBLAS warning: precompiled NUM_THREADS exceeded, adding auxiliary array for thread metadata.
OpenBLAS warning: precompiled NUM_THREADS exceeded, adding auxiliary array for thread metadata.
OpenBLAS warning: precompiled NUM_THREADS exceeded, adding auxiliary array for thread metadata.
OpenBLAS warning: precompiled NUM_THREADS exceeded, adding auxiliary array for thread metadata.
OpenBLAS warning: precompiled NUM_THREADS exceeded, adding auxiliary array for thread metadata.
/var/spool/pbs/mom_priv/jobs/1982979.infra-pbs.SC: line 22: 97810 Segmentation fault      (core dumped) TOKENIZERS_PARALLELISM=false python script_to_run.py
acn17.karolina.it4i.cz: # PBS Epilogue Report
acn17.karolina.it4i.cz: # Issues found in kernel log
acn17.karolina.it4i.cz: [ 7457.776637] python[115279]: segfault at 10 ip 00002b69cb10d175 sp 00002b6a902b7990 error 4

where I am not sure if the problem is in the implementation of virtual environment because I was able to get results for these tasks:
AmazonCounterfactualClassification.json      MassiveIntentClassification.json
AmazonPolarityClassification.json            MassiveScenarioClassification.json
AmazonReviewsClassification.json             MTOPDomainClassification.json
Banking77Classification.json                 MTOPIntentClassification.json
EmotionClassification.json                   ToxicConversationsClassification.json
ImdbClassification.json                      TweetSentimentExtractionClassification.json

I will try to investigate it further. If you have any suggestion for possible directions please let me know.

Hmm you could try decreasing the batch_size e.g. results = evaluation.run(model, verbosity=2, output_folder=f'results/{model_name}', batch_size=8)

I was able to run some of the additional measurements but I do not own the distiluse-base-multilingual-cased-v2 model - is it possible to report it anyway?
mteb_metadata.md

Yes that'd be great! Can you upload all the result files here?
(Or you can directly add them to this repo in a new folder: https://huggingface.co/datasets/mteb/results/tree/main/results)

@petrLorenc seems like the model has been added. I will close this issue for now.