[ogre] Cannot open include file

Is your feature request related to a problem? Please describe.
when downloading ogre with vcpkg
./vcpkg install ogre
all of the header files are not correctly included, they are all included with quotations "" rather than with brackets <> and the location of the header files are not accurate either.
for example:
a header in Ogre.h will have this by default
#include "OgreAnimation.h"
when it should have
#include <OGRE/OgreAnimation.h>
Proposed solution
i can rewrite hundreds of header files manually to maybe get ogre to work but a update may break the changes...
Describe alternatives you've considered
i'm very new to c++ and i just wanted to dabble with it, sorry...
Additional context
i am running on windows 10

I was able to get it running from building the source code in this helpful guide! It's a long process so if you could get vcpkg to work for ogre it would be cool but I'll be able to work on it on the meantime, so no rush! :)

Hi @celadaris ! How are you consuming OGRE? When OGRE is used from a project with a CMake build system, if I recall correctly the OGRE include directories are passed to the compiler, so the #include "OgreAnimation.h" works fine.

@traversaro
Thanks for your focus on this issue.
I also tried to build the project on my local. If I used #include "OgreAnimation.h", which can be compiled successfully.
But if I used #include "OGRE/OgreAnimation.h", which posted C1083 error.
To my confusion, all the headers are installed to include/OGRE/ folder and the include path also points to include/OGRE in OGREConfig.cmake:
get_filename_component(OGRE_INCLUDE_DIRS "${OGRE_PREFIX_DIR}/include/OGRE" ABSOLUTE)
I used ogre as follows:
find_package(OGRE CONFIG REQUIRED)
# Add source to this project's executable.
add_executable (CMakeProject21 "CMakeProject21.cpp" "CMakeProject21.h")
target_link_libraries(CMakeProject21 PRIVATE OgreMain OgreBites OgrePaging OgreVolume)

The path in building like this:
OGRE_LIBRARY_DIRS:F:/tool/vcpkg/installed/x64-windows/lib
OGRE_INCLUDE_DIRS:F:/tool/vcpkg/installed/x64-windows/include/OGRE

So I don't know what's wrong here.

i'm going to re-install it and try and provide better details soon...

perhaps i gave a bad example by specifically mentioning #include "OgreAnimation.h" but i have multiple other header files that cannot be opened when trying to use ogre in a very simple manner (which is what is shown in the video). But then after i recorded, i try to manually fix the header files and then more header files show up as errors... the code I've written up is supposed to simply open a black window. i can run the code just fine on my other computer that i built the ogre source code with, but vcpkg gives me errors when i try to use it... I will include a video and a link to the code i used.
https://github.com/celadaris/OgreTutorial/blob/main/OgreTutorial.cpp
https://user-images.githubusercontent.com/12479149/103394306-51804f00-4aed-11eb-8528-a189b1bbfd05.mp4

Hi @celadaris
Thanks for your test and the video.
Have you tried to update the headers like this?
#include "Ogre.h"
#include "Bites/OgreApplicationContext.h"

yes, here is a pic..
in the meantime I tried running ./vcpkg install ogre:x64-windows and no luck with that either

It's so weird. I can build successfully if I included the headers without OGRE.

it's okay, thanks for your time! i have notifications turned on so I'll be around.

I also should have mentioned that on the build i did with cmake from the source code, it also works without OGRE the pic i provided earlier was using vcpkg.

@celadaris
Sorry for the delay.
Also sorry for my misunderstanding about this issue.
Since OGRE subfolder has been included in OGRE_INCLUDE_DIRS in OGREConfig.cmake. It's unnecessary to include OGRE anymore when using headers.
We just need to use like this: #include "OgreAnimation.h".
So the current behavior is correct. Sorry again for my confusion.

@celadaris
Sorry for the delay.
Also sorry for my misunderstanding about this issue.
Since OGRE subfolder has been included in OGRE_INCLUDE_DIRS in OGREConfig.cmake. It's unnecessary to include OGRE anymore when using headers.
We just need to use like this: #include "OgreAnimation.h".
So the current behavior is correct. Sorry again for my confusion.

Okay so then referring back to the video I made. How would I get my code to run without errors about opening the source files?

Okay so then referring back to the video I made. How would I get my code to run without errors about opening the source files?

hi, sorry for the delay as well. I'm not really sure I understand your latest post because i feel as though the information your asking for has already been presented from when i posted the video. Perhaps it was hard to understand because this is one of the very few times that ive ever tried to work out a code related problem with someone so my posts may not be very well thought out.
I'll try to re-answer some of your questions but please let me know if maybe these aren't the answers your looking for, perhaps due to a misunderstanding on my part.

Could you please help provide your whole project sample here?

this is the project sample i wanted to try to run, i said earlier that "the code I've written up is supposed to simply open a black window."

Have you tried cmake project?

"i can run the code just fine on my other computer that i built the ogre source code with" (i built it with cmake in this helpful guide)

Does it work normally?

yes when it's built with cmake... "but vcpkg gives me errors when i try to use it"

Sorry for the long delay. I have tested your project on my machine. There are some errors.
But the headers should be included like this:
#include "Ogre.h"
#include "Bites/OgreApplicationContext.h"

And update #include "OgreRTShaderSystem.h" as #include "../RTShaderSystem/OgreRTShaderSystem.h" in ../Bites/OgreSGTechniqueResolverListener.h file.

I have the same problem anyone found a solution yet seems like i will have to instead use cmake to build the project

I have the same problem anyone found a solution yet seems like i will have to instead use cmake to build the project

If you refer just to the include problems, I guess that when using a non-CMake build system, a possible workaround is to pass <vcpkg_dir>/installed/x64-windows/include/OGRE as additional include path explicitly (change x64-windows with your triplet if you are using another triplet).

As @traversaro said above, you need to add <vcpkg_dir>/installed/x64-windows/include/OGRE to C/C++ -> General -> Additional Directories explicitly in your MSbuild project.
For the link error, it is due to lack of OgreMain.lib.
Since OgreMain.lib is installed in lib/manual-link directory, also need to add <vcpkg_dir>/installed/x64-windows/lib/manual-link/OgreMain.lib as Linker-> Input -> Additional Dependencies.
So we only need to fix the problem that the header cannot be found.
Should update #include "OgreRTShaderSystem.h" as #include "../RTShaderSystem/OgreRTShaderSystem.h" in ../Bites/OgreSGTechniqueResolverListener.h file.
I will submit a PR to fix this.

Hi, i have a couple more issues with ogre. I will try my best to walk you through reproducing the errors that i have, i made a short video on it here sorry for the bad quality, vimeo compressed the video when i uploaded it.

@celadaris
Thanks for your details.
Seems the original problem that the header file cannot be found and the link error has been solved now, right?
I looked through your video, the current problem you met is that some plugins cannot be built in your project. I noticed that you commented these plugins in plugin.cfg file, then you can continued to build.
Can you provide the link to get ogre-sdk-v1.12.9-msvc141-x64.zip?
I can only get the latest version https://www.ogre3d.org/download/sdk/sdk-ogre.
I'm not sure if the problem still exists on the latest version 1.13.0. I need to reproduce this on my local first on the version 1.12.9.

Seems the original problem that the header file cannot be found and the link error has been solved now, right?

my original problem was that i did not know i had to do the following:

add <vcpkg_dir>/installed/x64-windows/include/OGRE to C/C++ -> General -> Additional Directories explicitly in MSbuild project.
add <vcpkg_dir>/installed/x64-windows/lib/manual-link/OgreMain.lib as Linker-> Input -> Additional Dependencies

but your instructions solved that!
As for updating #include "OgreRTShaderSystem.h" as #include "../RTShaderSystem/OgreRTShaderSystem.h" in ../Bites/OgreSGTechniqueResolverListener.h file. that is a new issue that I still have to manually do every time, which i believe you're working on.
The issues I'm having with plugins are non-related to the header file.
I did not realize until just now that there is a page dedicated to downloading previous MSVC SDK builds (as shown in the image), no need for the method i used in the video, here is the download. If however you would prefer to focus on the latest version of ogre and porting that instead I'm alright with that.

Thanks for your clarification. I will try this on my machine and will update the status once completed.

I can reproduce this on my machine with 1.12.9. But since the .dll files are from external binaries instead of built by vcpkg.
We recommend to use all the binaries from vcpkg. I noticed these .dll files are also provided by vcpkg.
So could you please try to copy these .dll files to your project and rebuild this?

copying the .dll files from vcpkg\installed\x64-windows\bin  into the name folder of the project worked! i have a black window which is what was originally requested :partying_face:. The original problem is pretty much completely solved
We can close this and make a new thread if u want, but now using resources.cfg causes an error. resources.cfg is needed for anything that you would want to do with ogre besides open a black window.
if you solve this problem and the black window runs then maybe we can try this neat project i made a while back, to test ogre :D (warning: i did not test it in 1.12.9 so it might not be the best test x.x)

I tried adding all of the additional Include Directories as seen here
I tried adding all of the additional Dependencies as seen here

It helped a little, there's probably something in there that needed to be added, but still results in an error when trying to use resources.cfg maybe im just doing something wrong 🤷♂

I think you can open an new issue to report the new problems you encountered.


I18N rule conditions

Codecov Report

Merging #80 (86472a3) into main (07d90b8) will decrease coverage by 0.47%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #80      +/-   ##
==========================================
- Coverage   65.66%   65.18%   -0.48%
==========================================
  Files          39       39
  Lines        1095     1103       +8
  Branches      354      356       +2
==========================================
  Hits          719      719
- Misses        376      384       +8

Impacted Files
Coverage Δ

...-probe-ui/src/components/Diff/RulesDiffContent.tsx
0.00% <0.00%> (ø)


Support tilesets

A few things to make this happen:

Need some way to mark images as "tiles" so that they get emitted with the right metadata
Imported tiles should preserve the metadata
(bonus) We should add some way to specify a category name (and update the project import parsing to read it)

metadata/hamburger icon in the top left corner of the selected image (same size as delete, opposite it) that pops open a dialog with tile, category, etc fields? obscures more of the preview but the thing you're previewing is also on the canvas in front of you, just big

yeah, i like it!


Friendlier KAFKA_TOPIC

add instructions to create config var for topic
blow up on boot if topic env var isn't set (rather than waiting for a POST to the resource)

lgtm


Add files via upload

Related data links assets and personal data control (parent company) (telecommunications)

import "@stdlib/ownable";
import "@stdlib/deploy";
message Withdraw{
amount: Int as uint32;
}
contract Lottery with Ownable, Deployable {
const sendModeCommentOnly: Int = 1 | 2;
owner: Address;
winPercent: Int as uint8 = 45;

init(owner: Address, winPercent: Int) {
    self.owner = owner;
    if (winPercent>=0) {
        self.winPercent = winPercent;
    }
}

receive() {
    let ctx: Context = context();
    let betAmount: Int = ctx.value;

    if (betAmount < 100000000 || betAmount ><PHONE_NUMBER>){
        self.reply("Bet amount must be between 0.1 TON and 1 TON".asComment());

       return;
    }

    let potentialWinAmount: Int = betAmount * 2;
    if (myBalance()<=potentialWinAmount){
        self.reply("Insufficient funds in contract to cover potential win".asComment());

        return;
    }

    let randomValue: Int = random(0,100);
    let isWin: Bool = ( randomValue < self.winPercent );

    if (!isWin) {
        send(SendParameters{
            to: ctx.sender,
            value: 0,
            mode: self.sendModeCommentOnly,
            body: "You Lose!".asComment()
        });

        return;
    }

    send(SendParameters{
        to: ctx.sender,
        value: potentialWinAmount,
        mode: self.sendModeCommentOnly,
        body: "You Win!".asComment()
    });

}

receive(s: String) {}
receive(s: Slice) {}

receive("withdraw") {
    self.requireOwner();

    send(SendParameters{
        to: self.owner,
        value: 0,
        mode: SendRemainingBalance,
        body: "Withdraw successful!".asComment()
    });
}

get fun balance(): Int{
    return myBalance();
}

}


:new:usr: Adding Report for Apache Flume

In addition, please use the correct gitchangelog conventions. It is "new:usr: ..." instead of ":new:usr: ...".

This pull request updates many other students' files, and repeats other students' commits, so it's really difficult to understand what @sakorrap's additions are.
@sakorrap, please make a backup of the repo, refork and submit your paper again.


script fails when certain dependencies are already installed

Script exited when installing google chrome because I had already installed it.

This error happens when an application was installed some other way than brew. This happens when you get a machine that has some applications installed before we run./install.sh. I'm not sure this is preventable, but I do wish that Homebrew had a better error message than this.
Closing for now.


Private PYPI upload + dynamic versioning

adds dynamic versioning based on git commit + git hash
adds make priv-pypi-upload target

We better use versioneer lib to do this. Keeping it open as a WIP for now.


sendFabricEvent error thrown (intermittently) after ending an apprtc 2 way call

Browsers and versions affected
Chrome M53 Beta 53.0.2785.46
Mac 10.11.6, CrOS
Description
sendFabricEvent: SDK is not initialized or no Fabrics added error thrown after ending a 2 way apprtc call
Steps to reproduce

Establish a 2 way apprtc call between Mac and CrOS
End the call after a while

Expected results
No JS errors should be seen after ending a peer2peer apprtc call
Actual results
The following error is thrown
sendFabricEvent: SDK is not initialized or no Fabrics added.
gc @ callstats.min.js:3
PeerConnectionClient.sendCallstatsEvents @ apprtc.debug.js:3204
PeerConnectionClient.close @ apprtc.debug.js:2931
Call.hangup @ apprtc.debug.js:2296
AppController.hangup_ @ apprtc.debug.js:1971
Attached screenshot
CC @KaptenJansson
Notes
In that AppRTC call session, ICE connection dropped. Not sure if that's related, but thought might be helpful to mention it!
JS Console logs attached as well.

JS logs.txt

Thanks for filing, I will fix this.

This is fixed.


ingester  crash

Describe the bug
A clear and concise description of what the bug is.

is it a bug?
Version (please complete the following information):

OS: [e.g. Linux]  Linux
Jaeger version: [e.g. 1.8]  1.20
Deployment: [e.g. bare metal, Docker, Kubernetes, ...]  K8S 1.16

What troubleshooting steps did you try?
Try to follow https://www.jaegertracing.io/docs/latest/troubleshooting/ and describe how far you were able to progress and/or which steps did not work.
I have tried several times
Additional context
Add any other context about the problem here.

@yangvipguang Have you specified the ingester.deadlockInterval property? The message appears to indicate you haven't.
See description of this flag in https://www.jaegertracing.io/docs/1.20/cli/#jaeger-ingester.
I'll close this issue for now - but if that doesn't fix your issue please reopen with details of how you are running the ingester.

@objectiser  yes i just found that  i have fixed it .

I think we can have a better error message.


[Feature Request] Changing colors on grids

Feature description:
Allow the user to right-click on any grid with a colored dye, and be able to change the color of any kind of grid to one of the 16 colors, so it can fit with the aesthetic of a room, for example.
Version (make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Refined Storage: 1.2.26

I apologize in advance if this has already been added in a 1.11 or 1.12 release, this post will be deleted promptly if that is the case.

No.

Any particular reason?

Maybe more important things to do? If you really want it codes are here at the open. Do it yourself.


Register program type specific helper functions to uBPF in interpreted mode

This PR addresses #370 and second issue of #291 .
This PR adds support for program type specific helper functions in interpret mode. Here are the changes:

The program object now maintains an array of helper function IDs that are used by the program. Due to limitations in uBPF code, the helper function IDs have to be moved to a (0 - 63) range. The array indices are the "modified" helper IDs to be used by uBPF whereas the real helper IDs are stored in the array values.
When a program info provider (re)attaches the helper addresses are stored in the trampoline table. The index of the trampoline entry is chosen such that it can be easily derived from the helper ID; and a full lookup of the trampoline table is not needed when trying to retrieve address of a helper function.
Finally, when a byte code is loaded, the addresses of program type specific helper function are retrieved from the trampoline table. The trampoline table is updated every time a provider reloads post byte code load.

I have tested this using the sample extension driver and loading the test program for sample extension loaded in interpreted mode. I have also tested after enabling HVCI.
API_Test and ebpf_client tests were ran as well.

@shankarseal please resolve conflicts so this can be merged


Correction for raw affiliation ENS de Lyon, CNRS UMR 5239, Laboratory of Biology and Modelling of the Cell, Lyon, France

Correction needed for raw affiliation ENS de Lyon, CNRS UMR 5239, Laboratory of Biology and Modelling of the Cell, Lyon, France
raw_affiliation_name: ENS de Lyon, CNRS UMR 5239, Laboratory of Biology and Modelling of the Cell, Lyon, France
new_rors: 04zmssz18;029brtt94;02feahw73;01bj4fd12
previous_rors: 04zmssz18;029brtt94;02feahw73
works_examples: W3169390442
contact:<PHONE_NUMBER>ccc397ec05764115ae2170:23642792ed03399e453207d276a6 @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


MongoError['No primary node is available!']

Hi, I am getting the exception while running the project, I am using play2.2.0. Please check the generated log by Typesafe console below. Please give me some solution for this.
--- (Running the application from SBT, auto-reloading is enabled) ---
[info] play - Listening for HTTP on /<IP_ADDRESS>:9000
(Server started, use Ctrl+D to stop and go back to the console...)
[1] warning javax.* types are not being woven because the weaver option '-Xset:weaveJavaxPackages=true' has not been specified
[info] application - ReactiveMongoPlugin starting...
[info] application - ReactiveMongoPlugin successfully started with db 'modern-web-template'! Servers:
[localhost:27017]
[info] play - Application started (Dev)
[DEBUG] [10/30/2014 09:32:13.168] [play-akka.actor.default-dispatcher-5] [EventStream(akka://reactivemongo)] logger log1-Logging$DefaultLogger started
[DEBUG] [10/30/2014 09:32:13.168] [play-akka.actor.default-dispatcher-5] [EventStream(akka://reactivemongo)] Default Loggers started
[error] r.api.Failover - Got an error, no more attempts to do. Completing with a failure...
reactivemongo.core.actors.Exceptions$PrimaryUnavailableException$: MongoError['No primary node is available!']
at reactivemongo.core.actors.Exceptions$PrimaryUnavailableException$.(actors.scala) ~[na:na]
at reactivemongo.core.actors.MongoDBSystem$$anonfun$pickChannel$4.apply(actors.scala:508) ~[na:na]
at reactivemongo.core.actors.MongoDBSystem$$anonfun$pickChannel$4.apply(actors.scala:508) ~[na:na]
at scala.Option.getOrElse(Option.scala:120) ~[na:na]
at reactivemongo.core.actors.MongoDBSystem.pickChannel(actors.scala:508) ~[na:na]
at reactivemongo.core.actors.MongoDBSystem$$anonfun$receive$1.applyOrElse(actors.scala:252) ~[na:na]
[error] application -
! @6k39kii01 - Internal server error, for (GET) [/users] ->

you need to start the mongod
$ user@host /c/Program Files/MongoDB 2.6 Standard
$ ./bin/mongod --dbpath data

Hi Guys, yes as @keepscoding  pointed out you need to have a mongo instance running.  Going to close this.


Failed to minify the code from this file

I have a project which is running fine with local development mode when I try to build for production mode with npm run build getting following error, any guidance will be appreciated.
Development Environment :

npm --version : 5.6.0
node -v : v8.11.1

.babelrc as below
{
  "presets": ["es2015", "stage-2", "react", "env"],
  "plugins": [
    ["babel-plugin-root-import",
     "babel-plugin-transform-decorators-legacy",
     "transform-object-rest-spread",
     "transform-class-properties"]
  ],
  "env": {
    "debug": true
  }
}

react-scripts build

Creating an optimized production build...
Failed to compile.
Failed to minify the code from this file:
./node_modules/parse-domain/lib/parseDomain.js:25

Read more here: http://bit.ly/2tRViJ9
Let me know if more details are required.

I have the same issue

fixed by using v1.2.0

Thank you @abenchi  it's working.

Thanks @abenchi , v1.2.0 works with minification. It'll be best of course, if the latest version can continually support this :+1:

nice job! 💯
Close ticket?

Yay 🎉


Remove "ifndef undef".  If they are not defined, there is no need to undefine them, yes?

These looks like weird morphs of coda.h.cmake.in

The coda.h.cmake.in file is automatically generated from coda.h.in by a rule in Makefile.am.
So I can't accept pull requests on this file alone.
Unless you provide a patch for the Makefile.am to produce the coda.h.cmake.in without these defines, things will have to stay as they are.


Question wuwu

I'm sorry to bother you.Can I know the relationship between the model under the folder mpinets and the TemporalUNet used. Thank you very much.

Hi,
The model is totally unrelated to our framework. It is the model for MPINETs policy (one of the baselines in our paper). We shall remove it in the upcoming commit. Thanks for informing us!


Added app description and fix minor typo on readme

Hey!
Thank you for the contribution.
Can you remove the part of Hacktoberfest from the description?
Just make a clear description about the project, regardless the reason.

Thanks for the review @origranot . I have make a changes according to your instructions.


No info on readme for tape

I would recommend using istanbul:
istanbul cover tape test/*.js

I used istanbul cover --report cobertura test/**.js


Crashes on Windows 11 with

Describe the bug
I use wry in an application like this:

fn webview_cb(webview: &WebView, arg: &Cmd, data: &mut RpcData) {
    match &arg {
        Cmd::OpenScript { lines } => {
            // ...
        }
    }
}

fn main() -> wry::Result<()> {
    use std::env;
    use wry::{
        application::{
            event::{Event, WindowEvent},
            event_loop::{ControlFlow, EventLoop},
            window::WindowBuilder,
        },
        webview::WebViewBuilder,
    };

    let mut userdata = RpcData::default();

    let initial_screen = ui::render_entire_screen(&mut userdata);

    let resizable = true;
    let debug = false;
    let app_html = include_str!("app.html")
        .to_string()
        .replace("<!-- REPLACED_ON_STARTUP -->", &initial_screen);

    let event_loop = EventLoop::with_user_event();
    let proxy = event_loop.create_proxy();
    let window = WindowBuilder::new()
        .with_title(APP_TITLE)
        .with_maximized(true)
        .build(&event_loop)?;

    let webview = WebViewBuilder::new(window)?
        .with_html(app_html)?
        .with_devtools(false)
        .with_navigation_handler(|s| s != "http://localhost/?") // ??? - bug?
        .with_ipc_handler(move |_window, cmd| match serde_json::from_str(&cmd) {
            Ok(o) => {
                let _ = proxy.send_event(o);
            }
            Err(e) => {
                println!("{e}");
            }
        })
        .build()?;

    event_loop.run(move |event, _, control_flow| {
        *control_flow = ControlFlow::Wait;

        match event {
            Event::WindowEvent {
                event: WindowEvent::CloseRequested,
                ..
            } => {
                *control_flow = ControlFlow::Exit;

                let _ = std::fs::remove_file(std::env::temp_dir().join("dgb").join("auth.json"));

                if let Ok(original_value) = original_value.as_ref() {
                    env::set_var(GTK_OVERLAY_SCROLLING, original_value);
                }
            }
            Event::UserEvent(cmd) => {
                webview_cb(&webview, &cmd, &mut userdata);
            }
            _ => {}
        }
    });}

It doesn't matter what version of wry or WebView2 I use, the application always crashes with:
Error: WebView2Error(WindowsError(Error { code: 0x80070057, message: Wrong parameter.
, win32_error: 87 }))

Steps To Reproduce
See https://github.com/projekt-dgb/dgb/actions/runs/4765767255 or https://github.com/projekt-dgb/dgb/blob/421e597afb7afdc0c79d36eff2fdbd02d9775d68/src/main.rs#L6789 for code
Download and unzip https://github.com/projekt-dgb/dgb/suites/12397174748/artifacts/659100197
Run dgb.exe
Expected behavior
The application launches like on MacOS:

Platform and Versions (please complete the following information):
OS: happens on both Windows 8.1 and Windows 11
Rustc:
Would you want to assign yourself to resolve this bug?

[X] Yes
[ ] No

I have tried wry 0.16.2, wry 0.22.0 and wry 0.28.0, as well as running on Windows 8.1 and Windows 11. I use wry 0.22.0 for Mac builds because 0.28.0 just runs an infinite loop on Mac and just shows a blank white window (another bug, but I can ship the older version for now).

The issue was that I was (a) calling with_html() with a > 2 MB string and calling .with_maximized(true) on the Window (doesn't work on Mac). Works now, thanks.


Enriched IoT Edge module messages can not access Device Twin values

I tested message enrichment with messages coming from Azure IoT Edge modules.
The enrichment only works for Module Twin values. Device Twin values (not 'overruled' in the module twin) are not mapped.
I expected the enrichment to works with the Device Twin as 'fall-back'.
Now I have to copy and update common device-level values in each module twin which is error prone.
Is this the right observation? Is this behavior by design?
ps. See also https://sandervandevelde.wordpress.com/2021/05/10/message-enrichment-for-azure-iot-edge-modules/ for the way I tested it.

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

ID: 09122ce7-4c60-89f8-205f-a9f0a4796fe7
Version Independent ID: 860539b6-44f5-5d09-b371-02864d4366d2
Content: Overview of Azure IoT Hub message enrichments
Content Source: articles/iot-hub/iot-hub-message-enrichments-overview.md
Service: iot-hub
GitHub Login: @robinsh
Microsoft Alias: robinsh

Hello @sandervandevelde We received a response from the product team on this query.

Yes. It is by design. The device twin values are not inherited.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.
Please also do visit Microsoft Q&A the home for technical questions and answers at Microsoft.

@SatishBoddu-MSFT The article is written with direct-connected IoT devices in mind and is not describing this behavior related to Edge devices and their modules.
I suggest adding this to the article.

Hello @sandervandevelde Thanks for the feedback on doc enhancement, we will keep you posted!


Push the new ARGOS DB webpage code to production.

https://argosdb.org/ -This link shows the old website. New changes have been made and pushed to the main branch. Kindly Update the link

@rajamazumder

@Reeya123 @kee007ney @rajamazumder  --- I have updated the code for the repo. Anyone should be able to deploy the code in any of the argosdb servers (dev/test/beta/prd) following the instructions given in the README.md at the root of the repo.

Hello @rykahsay @HadleyKing . I am unable to follow the steps correctly in order to push my updated code to production. After making changes in the code, I ran the command python3 start.container.py as instructed in the readme.
Error: PS C:\Users\gupta\OneDrive\Desktop\argosdb-main\app> python3 start_app_container.py
The syntax of the command is incorrect.

<EMAIL_ADDRESS>build
react-scripts build

'react-scripts' is not recognized as an internal or external command,
operable program or batch file.
I am assuming this is because it thinks that react script isn't in the node modules but I see it in the package.json file. I don't understand where I am going wrong.
Also after running the above part,  exactly where would I make the changes that are asked in the below statement?
-->To serve the APP at a given public domain name such as www.argosdb.org, add the following lines to your apache VirtualHost directive (assuming the "app_port" you have selected in the conf/config.json configuration to be 5090)

@Reeya123
I have reassigned this issue to @kee007ney. The instructions in the README.md need to be followed on the ARGOS server. I do not believe you have access.

@HadleyKing do you have permission to do this? I don't think that I do
@kee007ney are you able to push this today?

@steph-sing Robel asked Dacian to give me permissions on the server to do it. Just sent a follow up now to ask for an ETA. Whenever I get permissions I'll push.

Thank you for the update Jonathon

@Reeya123 New version was pushed. Header portion is removed, but spacing still looks wrong. I'm guessing this is not what you want to see. @HadleyKing will look at the React framework to see if it's an issue here.

Hadley, Pat, and I talked this over. Hadley figured out that the HTML was duplicated in /home/keeneyjg/argosdb/app/src/components/home_page.jsx. When we remove that, it works fine. @Reeya123, I don't know if you added that content to that page or not.
Closing this as complete.

just checked and looks to be fixed. Thanks guys!


open doesn't use arguments/options

if you look here https://github.com/infinitered/bluepotion/blob/70007244adc1a0c03924cd015104b30ac9e79ac6/lib/project/pro_motion/activities/pm_navigation_activity.rb#L26-L33
you can see nothing is done with the options
my current work around is to init the object, change the property and then call open

You're right. Should definitely be fixed. Seems like a straightforward pull request...want to give it a go, @jsilverMDX ? Something like:
options.keys.each{|k| frag.send("#{k}=", options[k]) if frag.respond_to?("#{k}=") }

Yes, I will make a demo app and try to fix this before work today.
On Sat, Oct 17, 2015 at 11:53 AM, Jamon Holmgren<EMAIL_ADDRESS>wrote:

You're right. Should definitely be fixed. Seems like a straightforward
pull request...want to give it a go, @jsilverMDX
https://github.com/jsilverMDX ? Something like:
options.keys.each{|k| frag.send("#{k}=", options[k]) if frag.respond_to?("#{k}=") }
—
Reply to this email directly or view it on GitHub
https://github.com/infinitered/bluepotion/issues/101#issuecomment-148937947
.

Thanks @Jamon.
On Sat, Oct 17, 2015 at 4:35 PM, Jonathan Silverman<EMAIL_ADDRESS>wrote:

Yes, I will make a demo app and try to fix this before work today.
On Sat, Oct 17, 2015 at 11:53 AM, Jamon Holmgren<EMAIL_ADDRESS>
wrote:

You're right. Should definitely be fixed. Seems like a straightforward
pull request...want to give it a go, @jsilverMDX
https://github.com/jsilverMDX ? Something like:
options.keys.each{|k| frag.send("#{k}=", options[k]) if frag.respond_to?("#{k}=") }
—
Reply to this email directly or view it on GitHub
https://github.com/infinitered/bluepotion/issues/101#issuecomment-148937947
.

:+1:

seems fixed to me.


⚠ Bofrost.nl has degraded performance

In 207efbc, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 3116 ms

Resolved: Bofrost.nl performance has improved in 9a85f05.


Update with per sc details for managed modem profiles

Add table below under managed modem profile names-
 | **Spacecraft Title**    | **Aqua**  |**Suomi NPP**|**JPSS-1/NOAA-20**| **Terra** |
 | :---                    |  :----:   |    :----:   |     :----:       | :----:    |
 | `noradId:`              | 27424     | 37849       | 43013            | 25994     |
 | `centerFrequencyMhz:`   | 8160,     | 7812,       | 7812,            | 8212.5,   |
 | `bandwidthMhz:`         | 15        | 30          | 30               | 45        |
 | `direction:`            | Downlink, | Downlink,   | Downlink,        | Downlink, |
 | `polarization:`         | RHCP      | RHCP        | RHCP             | RHCP      |

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

ID: fa1b40c1-7456-dd6b-563e-5bf2b7912ad9
Version Independent ID: 2534bf43-bcfe-e738-8c03-ae7263cf28db
Content: Configure the RF chain - Azure Orbital
Content Source: articles/orbital/modem-chain.md
Service: orbital
GitHub Login: @hrshelar
Microsoft Alias: hrshelar

@hrshelar
Thank you for bringing this to our attention. I've assigned this issue to the author who will investigate and update as appropriate.

Addressed by https://github.com/MicrosoftDocs/azure-docs-pr/pull/235286. #please-close


Laravel 5.1 Upgrade

Any chance of getting this upgraded to 5.1 now that it's in full release?

Sure. :shipit:


Fix ixmp4 tests and support Python 3.12

Please confirm that this PR has done the following:

[x] Tests Fixed
[x] Documentation Added
[x] Name of contributors Added to AUTHORS.rst
[x] Description in RELEASE_NOTES.md Added

Description of PR
FYI @meksor, @phackstock, @danielhuppmann.
This PR fixes the test failure reported in https://github.com/iiasa/ixmp4/issues/50 by configuring dask to not convert None to string[pyarrow_numpy]. This is following the suggested workaround in https://github.com/dask/dask/issues/10631#issuecomment-1915417729, which is not a sustainable solution, which might be provided by https://github.com/dask-contrib/dask-expr/issues/691.
Coincidentally, I ran the tests in a local venv of Python 3.12 and they all seem to pass (after adapting a matplotlib option in one tutorial), so this PR might close #798. For that, the steps suggested in setup.cfg still have to be executed because of minimum version bumps and the release notes/docs have to be updated as suggested above.
@phackstock please let me know if you want me to finish the PR or if you want to take a look (since the python support might be non-trivial and a new release would then probably be warranted).

I'm wondering about the pytest-legacy workflow: won't these lines install the legacy packages first, but then update them?

Please note that while applying ruff to the code base, I didn't take the time to make all your functions compatible with the code complexity limit we set for ixmp4. Instead, please search for # noqa: C901 to discover which functions have been excluded from this measurement.
Also, I'd recommend using mypy to automatically detect issues like the following:
    def validate(
        self,
        criteria: dict = None,
        *,
        upper_bound: float = None,
        lower_bound: float = None,
        exclude_on_fail: bool = False,
        **kwargs,
    ) -> pd.DataFrame:

The type hints are off. Someone could think it's okay to not set e.g. criteria, but doing so would most likely result in a failure.

Sorry, looks like I missed one notebook. Feel free to cancel the tests and restart.

I'm wondering about the pytest-legacy workflow: won't these lines install the legacy packages first, but then update them?

No, the second pip install will not upgrade existing packages if they are compatible with the package dependencies.

Re @glatterf42' comment about the validate() signature - the signature is correct and criteria is not a required argument, instead a user should use keyword arguments directly. The criteria argument is kept as a legacy-compatibility fix and will be removed in the next major release.

@glatterf42, thanks for the quick updates on this PR. To me it currently combines three sets of changes that should be their own PRs:

Use & apply ruff for formatting and linting
Officially update pyam to support python 3.12
Fix the failing ixmp4 test (that one should not be necessary anymore after iiasa/ixmp4#53)

So if you don't mind, I'd suggest that you close this PR and open two new ones for points 1. and 2. above.
Once we have a new release of ixmp4 we can then remove the ixmp4 version pin to see if the tests run.

As discussed just now, #821 is for ruff and we'll just remove the workaround for the failing test from here once ixmp4 releases 0.7.1, so no need for a third PR.

Closing in favor of #821

Okay, then I'll open another PR with commits cherry-picked from here to support Python 3.12. Note to self: I also need to update the badge in index.rst.


Hardcoded scripts folder name causes script to fail

Error when running script acendan_reaperblog_Duplicate items or tracks depending on focus and time selection.lua:
"This script requires ACendan Lua Utilities! Please install them here:
Extensions > ReaPack > Browse Packages > 'ACendan Lua Utilities'"
I had Lua Utilities installed and error persisted.
Going to the source I found that path to scripts was written with lowercase S:
reaper.GetResourcePath()..'/scripts/ACendan Scripts/Development/acendan_Lua Utilities.lua'
both in acendan_Lua Utilities.lua and acendan_reaperblog_Duplicate items or tracks depending on focus and time selection.lua.
I changed "scripts" for "Scripts" everywhere and the script worked.
Please, change the path by default, make case-insensitive folder seeker or add a mention of this in docs to repo.
Thank you for the script, it is awesome!

Hi @Marc-Winner,
Thanks for letting me know about this issue! Out of curiosity, what platform/operating system are you on? You're the first person to report having this problem to me; which is a bit of a surprise.
Cheers!

Hey Marc! Recently I set up a Linux Virtual Machine for the first time and ran into the same issue you mentioned. Just pushed out an update to all of my scripts that referred to the lowercase path. Should hopefully work as expected now!

Glad to hear it!
Keep up the good work!
Best regards,
Marsel
On Sat, Jul 16, 2022, 21:36 Aaron Cendan @.***> wrote:

Hey Marc! Recently I set up a Linux Virtual Machine for the first time and
ran into the same issue you mentioned. Just pushed out an update to all of
my scripts that referred to the lowercase path. Should hopefully work as
expected now!
—
Reply to this email directly, view it on GitHub
https://github.com/acendan/reascripts/issues/2#issuecomment-1186261489,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE6D7I3JQL6WGGOFZ3IEFXLVUL6JJANCNFSM5WSFVXZA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello,
I use Ubuntu 21.04. Yeah, I don't know how naming of folders change from OS
to OS. Maybe, other users are just so advanced that they corrected it
silently)
Best,
Marsel
On Wed, May 25, 2022, 04:37 Aaron Cendan @.***> wrote:

Hi @Marc-Winner https://github.com/Marc-Winner,
Thanks for letting me know about this issue! Out of curiosity, what
platform/operating system are you on? You're the first person to report
having this problem to me; which is a bit of a surprise.
Cheers!
—
Reply to this email directly, view it on GitHub
https://github.com/acendan/reascripts/issues/2#issuecomment-1136611941,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE6D7I63RMKTAZRNX2UTSX3VLV743ANCNFSM5WSFVXZA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Show end state of game

all non-mined cells are uncovered
a mined cell is uncovered

I'll try to implement it similary to the original minesweeper game. This means:

all non-mined cells are uncovered
every mine is marked as flagged
a mined cell is uncovered
all mines are uncovered and the clicked mine has a red background color

Closed by #67


Watch 3Blue1Brown in class?

https://www.youtube.com/watch?v=aircAruvnKk&feature=youtu.be
I've found this video to be understandable to even people who haven't worked with ML before and helps demystify what is a "neural network". Maybe people should watch before coming to class next time to help make some of these concepts like gradient descent, hidden layers, etc etc?
While the video is nearly 20min could be helpful to watch together so that students can ask questions along the way? or maybe helpful if students have to respond to the video with an illustration of how all those components work?

A helpful reminder that students should watch this video between Monday and Wednesday this week. I agree that stopping and starting the video with questions / discussion would make it much more meaningful to watch in class, maybe I can pick out some parts of we can watch it on Wed this week.


[3.x] Fix non-latin layout scancodes on Linux, adds access to physical scancodes.

Backport of #18020, without breaking changes.
Existing Functions properties are not renamed, new property is physical_scancode.
For the reference, list of changes to existing API (in both cases physical scancode is used only if remapped scancode is 0):

InputEventKey::action_match uses physical scancode, if remapped scancode is 0 (Physical key are added to InputMap with scancodes == 0, remapped keys with physical_scancode == 0).
On Linux/X11, physical scancode is used in place remapped, if scancode is 0 (not remapped correctly by XLookupString).

Modified test project from 46751: Test.zip

Screenshots:
Dvorak:

Russian (phonetic) layout:

Results are the same on macOS, Linux (Ubuntu 20.10) and Windows.

Fixes #46751 and fixes #46750

Thanks!

Does this mean we'll be able to bind the default keys based on key positions instead of the letters of the keys?  Would be great for non-qwerty keyboard users!

Does this mean we'll be able to bind the default keys based on key positions instead of the letters of the keys? Would be great for non-qwerty keyboard users!

Yes 🙂

Awesome!  I saw there's a physical_key that gets written to the project file, but it seems to always be 0.  Maybe it's only supported in Linux at the moment?

@jitspoe Should work on all platforms, but key and physical_key are mutually exclusive. You should click "Physical Key" to add one to the input map (and key will be 0).

Is there a way to get the display string from a physical_key?  Can't seem to find anything, so I made a proposal here: https://github.com/godotengine/godot-proposals/issues/3571

I really appreciate this!! Was really hoping I wouldn't have to wait until 4.0 for this and thankfully that is now the case.


Revert 1 feature/assignment4

Current coverage is 100.00%

Merging #2 into develop will not affect coverage as of 6b8589d

@@            develop      #2   diff @@
=======================================
  Files             6       6
  Stmts           110     110
  Branches         20      20
  Methods           0       0
=======================================
  Hit             110     110
  Partial           0       0
  Missed            0       0

Review entire Coverage Diff as of 6b8589d

Powered by Codecov. Updated on successful CI builds.


TextField cursor on iOS displayed twice

Steps to Reproduce
Unclear... appears randomly.
Expected results: Only one cursor
Actual results: Two cursors
No errors in the logs :-(

Analyzing app...
No issues found! (ran in 33.6s)
Flutter Doctor on other machine. Sorry, can't reach my macbook until next week.

[✓] Flutter (Channel stable, v1.17.1, on Linux, locale de_DE.UTF-8)
• Flutter version 1.17.1 at /home/krille/Lokal/HADTF/flutter_linux_v1.2.1-stable/flutter
• Framework revision f7a6a7906b (vor 9 Tagen), 2020-05-12 18:39:00 -0700
• Engine revision 6bc433c6b6
• Dart version 2.8.2
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /home/krille/Android/Sdk
• Platform android-29, build-tools 29.0.2
• Java binary at: /snap/android-studio/88/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
• All Android licenses accepted.
[✓] Android Studio (version 3.6)
• Android Studio at /snap/android-studio/88/android-studio
• Flutter plugin version 45.1.1
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
[!] Connected device
! No devices available
! Doctor found issues in 1 category.

Hi @ChristianPauly

Steps to Reproduce
Unclear... appears randomly.

With this the issue is hardly actionable. Could you try to notice some pattern of what and how you are typing/selecting and check if it happens with another locale / another device

/ with the minimal code sample containing only TextField?
import 'package:flutter/material.dart';

void main() => runApp(MaterialApp(home: MyApp()));

class MyApp extends StatefulWidget {
  @override
  _MyAppState createState() => _MyAppState();
}

class _MyAppState extends State<MyApp> {
  @override
  Widget build(BuildContext context) {
    return SafeArea(
      child: Scaffold(
        body: Center(
          child: TextField(),
        ),
      ),
    );
  }
}

Without additional information we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.

I also has the same problem, appears randomly too, both in iOS and android devices while developing flutter web.


Use standard GNU- and BSD-style options

I’ve never seen the -p=8080 notation – it puzzled me a bit. I guess a more mainstream format will be more developer-friendly :)

@henrytseng here’s some more reading on the topic: https://jesse.sh/the-art-of-unix-programming/#command-line-arguments.

You can disclude the =.  I'll update the documentation


Ocp

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

90 of 141 (63.83%)  changed or added relevant lines in 3 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 58.981%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/render/render.go
56
70
80.0%

pkg/syncdaemon/syncdaemon.go
29
66
43.94%

Files with Coverage Reduction
New Missed Lines
%

internal/controller/nicconfigurationtemplate_controller.go
7
85.92%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.005%

Covered Lines:
1875

Relevant Lines:
3179

💛  - Coveralls


Further extend readme

I am a new to using source code and compiling it, only used precompiled bootstrap before. I would appreciate a quick start guide in the readme on how to use bootstrap with less on meteor.

:+1:

Please let me know if this is now enough for you:
https://github.com/huttonr/bootstrap3/blob/master/README.md

Super!! thanks.


Byte order marks in source files

Continuing from the discussing that started in #406.
My view is that BOMs can be annoying and are completely pointless for UTF-8, but rarely cause problems in recent years. After a quick Google search I just discovered that the Unicode 5 standard actually discourages their use with UTF-8 files, I'm glad that is their position because it makes sense.
However, I know older versions of Visual Studio insert BOMs and adds them on save to files without one. I didn't test out Visual Studio but the following web site makes it sound like VS still inserts them:

The free Fix File Encoding extension prevents Visual Studio 2017/2015/2013/2012 from adding BOM to UTF-8 files.
(https://vlasovstudio.com/fix-file-encoding/)

If VS doesn't insert them anymore we could set the charset EditorConfig property and normalise all files to be without one, however it doesn't appear that is that case which means if someone uses any other text editor to work on the source code you won't get BOMs so we'll always have a mix of with and without.
@fir3pho3nixx @stakx if you'd like to look into/confirm any of that feel free, but I think we should just slowly walk away from this beehive and close this issue 😉.

I think the important thing is that files with special characters are factually encoded as UTF-8. Whether there is a BOM in every file, only in some files, or in none of them is of lesser importance IMHO. I don't have any strong feelings on the topics, so I am perfectly happy to withdraw from that topic.
One last suggestion, though: Perhaps it would make sense to add charset = utf-8 to .editorconfig?

Hey I don’t mind honestly. Glad I was right to begin with but there is more important to be getting on with. I just feel bad because I accidentally trolled @davidshen84’s PR. Sorry :/

I agree with both of you.
@jonorossi, you've already mentioned charset = utf-8 for .editorconfig, but perhaps even that can wait.
@fir3pho3nixx, don't feel bad, something good has come out of this. We now know our position regarding encoding—we all favor the ideal of UTF-8 files without BOM—so if this topic ever crops up again, we'll already know where we stand.

you've already mentioned charset = utf-8 for .editorconfig, but perhaps even that can wait.

@stakx Yep, as I mentioned above if we were going to set this we'd have to be confident Visual Studio no longer adds them, otherwise we'd have every other editor doing as we asked with VS adding them back.
 What about vs-code, atom, sublime et al.

Pre-commit hook in git would be legendary.

Complete aside:
Just to show there is a human side to me(@stakx), the amount of fights I have had in the office about git installations with windows guys(checkout as-is, commit as-is) over my career has probably wasted enough time perhaps for the man hours required to supply all the programmers around the world with a single cup of coffee in one day.
Today, I found out AGAIN that LF-CR screws up bash files on Linux. Encoding is the least of our worries!
I wish everyone would checkout as-is and commit unix.

@fir3pho3nixx - I can sympathize. It's true that these little issues sometimes needlessly try our patience when we collaborate across different platforms.
"Be liberal in what you receive, be conservative in what you send" is a great guideline for successful interoperation, and I'm under the impression that tools these nowadays are generally getting better at implementing it and using sane defaults.
Things might get even easier once Microsoft flushes all their CR LFs down the toilet and adopts UNIX-style line endings (... and, who knows, perhaps even a good command line processor).  What about vs-code, atom, sublime et al.

@fir3pho3nixx sorry no idea what you are referring to. You'll find most sane editors will keep the encoding with or without a BOM as the file is on disk, whereas Visual Studio would/still does (?) add one no matter how the file is on disk. All I was pointing out is that if you instruct well behaved editors (via EditorConfig) to always save without BOMs and Visual Studio does indeed still write them you'll be in a worse position of the first line constantly showing up in diffs being flipped back and forth.

Today, I found out AGAIN that LF-CR screws up bash files on Linux. Encoding is the least of our worries!
I wish everyone would checkout as-is and commit unix.

Aren't we all glad we don't have to deal with core.autocrlf anymore, I remember the days of using Git and having to deal with that nightmare, especially with .NET being Windows focused and the repository having CRLFs. I also remember in earlier versions of Visual Studio you couldn't set checkout as-is because solution files would fail to load if they didn't have CRLFs as they aren't XML. There seems to be a common theme here, that Visual Studio has never played well with others.

[...] and Visual Studio does indeed still write them [...]

I suspect that the problem with Visual Studio may be that it doesn't apply EditorConfig rules to item template files when a new project item is created. And it seems that the default item template files (e.g. for a C# class) have a UTF-8 BOM and use four spaces for indentation (which is one reason why you might see indentation mistakes whenever someone adds a new file to a project that's configured to use tabs).

@fir3pho3nixx sorry no idea what you are referring to.

Let's par le.

You'll find most sane editors will keep the encoding with or without a BOM as the file is on disk, whereas Visual Studio would/still does (?).

Why is Visual Studio a "go to" point for you? A sane editor for you is not the same as the next guy. Vs will now save it in the same way it found it. It still moans about LF-only endings on windows.

All I was pointing out is that if you instruct well behaved editors (via EditorConfig) to always save without BOMs and Visual Studio does indeed still write them you'll be in a worse position of the first line constantly showing up in diffs being flipped back and forth

I am thinking pre-commit hooks using git or really forcing things down to the client using .gitattributes.

Aren't we all glad we don't have to deal with core.autocrlf anymore

Yes sir!

There seems to be a common theme here, that Visual Studio has never played well with others.

I use dotnet core on Linux. Tell me about it.

You could write your own text converter in bash using sed to remove the BOM. That is Linux though.
Guys over here are saying pre-commit hooks are the way: https://stackoverflow.com/questions/15780867/is-it-possible-to-make-git-ignore-utf-8-bom-when-commit


[Feature]: spark-dependencies job should support OpenSearch in addition to ES and Cassandra

Requirement
A Jaeger deployment allows for automatic span interdependency analysis to build a system architecture tree. I would like to see the job responsible for generating this architecture to support our OpenSearch backend storage.
Problem
According to the documentation, spark-dependencies is only supporting ES and Cassandra, not OpenSearch.
Proposal
To implement or document the correct parameters for the job to run against an OpenSearch backend.
Open questions
No response

@ronnybremer There is already an issue related to this subject in the spark-dependencies project.


Archive rdf4j-tools

A former independent repository https://github.com/eclipse/rdf4j-tools says "this repository is now closed", can't it also be archived on GitHub?

Yes, it can, but we're taking it one step at a time :) Same holds for rdf4j-storage btw

One reason to temporarily keep these repos open is that makes it easier for us to run hotfix releases on previous versions. We will likely close once release 3.1 is out the door and we can confidently drop hotfix support for these older releases.

Logged request with Eclipse maintainers to mark as archived: https://bugs.eclipse.org/bugs/show_bug.cgi?id=559899


'readFile' function mistakenly reported as being unused

When I run jshint in this code with the unused option set to vars it reports the following function as being defined but never used:
readFile("as");

function readFile(text) {
    console.log(text);
}

I noticed that the declaration order and the function name matter. If I define the function first or if I change the function name the error is not reported.

Simple pattern to reproduce this behaviour is:
var used; fn;
function fn() {
    used;
    function used() {}
}

// Output: line 4, col 14, 'used' is defined but never used.

Problem also occurs if you have declared variable of same name as function in outer scope, /*globals ... */or predef option included.
Please fix it, this thing is really annoying.


Arm template Reference - Event hub missing API version

There is a hole new API 2018-01-01-preview that is not documented for the eventhub and eventhub Namespace in the Arm Template reference.
missing information og Iprestrictions, defaultACL and more.

@mthoger Where's the doc?


Added logger for bascule decorators

Codecov Report

Merging #41 into master will decrease coverage by 0.7%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
- Coverage   45.88%   45.17%   -0.71%
==========================================
  Files           5        5
  Lines         255      259       +4
==========================================
  Hits          117      117
- Misses        135      139       +4
  Partials        3        3

Impacted Files
Coverage Δ

main.go
7.04% <0%> (-0.21%)
:arrow_down:

Continue to review full report at Codecov.

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


Adding missing methods to Apply optics - Part2

Fixes part of #439
Added methods for Iso, Lens, Optional, Traversal, Getter

👍 Thanks


event in the table

Hey!
I try to open a popver with a click on a button inside a  of a rowsorter table. But it doesn't work. Same things for modals and any actions except tooltips.
Any solution for me?
Thank you!

try new version please. and try "handler" option. for example, set "handler" option to "td".

Don't know how GitHub works (don't really care) so I'll post an issue here.
Version as of Nov 30 2015. The mouse down event on handler is somehow captured so no underlying element is getting the bubble event calls. Worked it older version.
e.g. Click the rowSorter handler, the bubble stops. If you have a on('mousedown') function on the table row, you are screwed. mouse move and mouse up work.
Should also have enable/disable option for entire plugin. I have problems when the handler is doing other things with other plugs-ins. e.g. My Slider plugin also user the Handler to slide the handler left and right. (think of the flash UI or recording tracks).
Update: I had to change table selector to document to get it working:
line 45 change to: document.on(downEvent, settings.handler, downFunc);
line 190 added: document.off(downEvent, upFunc);


Support for hiding navigation header when headerMode: float

Currently it's supported only with headerMode: screen. We should support it on float also and hide the header with animation.

Let's merge headerMode: 'none' | 'screen' | 'float' and header.visible: boolean, to header.mode: 'none' | 'screen' | 'float'.

@ericvicenti ^ ?

Only problem I see is that header.visible changes the header on a per-screen basis, and headerMode changes it for the whole navigator. What happens if you transition from a float screen to a screen with mode: 'screen'?
The animation for that isn't very easy.. what would it look like?

@ericvicenti the only difference between them is when the animation is happening, right? it shouldn't be visible if changed just before the animation starts, .i.e when the state changes.

To be clear, you are suggesting to have the floating header simply hide when the next header.mode:screen screen enters? Such that either A) the floating header instantly disappears, or B) both headers are visible during the transition?
Depending on the animation implementation, B might be quite nice.

Working on a similar transition, I would love to see B implemented

The doc's say you should be able to do headerMode: 'none' but this doesn't seem to work for me.

i would like to be able to hide the header when the mode is float. we could just have it as is now, and just enable the hiding. i guess thats simpler than the suggestions above.

With react-router-flux it was working quite well for my use case: Starting of with no header visible on the main screen. Then navigate to the next screen by using the "screen mode animation", which will have a header then. All animations ontop of this view are of type "float" then

yea i was using react native router flux too. feels more native now, but also feels weirdly set up. the api of flux is better I think. I like some stuff here but even though this is muuuuuch better than previous navigator attempts, it still feels half way there. I guess it will get better with time :)

 static navigationOptions = {
    header: {
      style: { position: "absolute" }
    }
  };

Anyone see any problems with using this hack in order to hide the header on a specific view?

@mdramos I gave that a try, and it shifted down the content of the hidden header page during the animation.

I'll be working on this in the morning. I'll keep you posted!

Having worked on this today I think it needs some further discussion.
To fit into what's already in the code I worked on a method named "shouldEmbedHeader" which determines whether the header should float outsides all scenes or sit inside the current scene depending on the navigationOptions of the current scene and the one previously shown.
When we transition from headerVisible: false to headerVisible: true it keeps the header inside the view (as if it were mode: 'screen') but then seamlessly swaps it out to a floating header when the transition is complete.
There's various edge cases to deal with so it's not ready to release but I'm concerned that it already feels very much like a hack so I'd like to get some thoughts from the core developers.
@skevy @grabbou @satya164 @cooperka

...to be clear I'm attempting to allow any combination of modes and visible header and allowing for modal as we can natively.

There are may hacks
Try this solution
const style = StyleSheet.create({        hideText:{              display:"none"        }   })
  static navigationOptions = {         header: <Text style={style.hideText} ></Text>     };

complete solution:

complete solution:
app.js:
auth: {
screen: StackNavigator({
auth: { screen: AuthScreen },
login: { screen: LoginScreen }
})
},
auth: (shows no header)
class AuthScreen extends Component {
static navigationOptions = {
header: null
}
login: (shows header)
class Login extends Component {...

@fritz-c  did you find a solution? or switched to another navigation library?

check out this example: https://github.com/react-navigation/react-navigation/blob/master/examples/NavigationPlayground/js/StackWithHeaderPreset.js
you can hide header on specific screens and it will transition as expected between header/no header

Why is this issue closed? on Android the header is still shown.
<Stack.Navigator
initialRouteName="Register"
mode="modal"
headerMode="float"
screenOptions={{
header: ({ scene, previous, navigation }) => (

),
}}
>
<Stack.Screen
name="Register"
component={RegisterRecyclablesFormScreen}
options={{ headerTitle: 'Register' }}
/>
<Stack.Screen
name="Scan"
component={ScanBarcodeScreen}
options={{
header: () => null,
}}
/>
</Stack.Navigator>
Could there be some clarification as to how to hide a header that works with headerMode = 'float'?

I've tried to use this on the screen that overrides the float header... but doesn't works!
React.useLayoutEffect(() => {
navigation.setOptions({
headerShown: false,
});
}, [navigation]);
any workaround for this in v5???


INPUT and SELECT components do not respect GRID item space assignation

[ X] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
When a INPUT or SELECT components are inside of GRID component, they should follow the layout established by the GRID component.
Current Behavior
Right now <INPUT and <SELECT are rendered without fill the space assigned by the GRID and they must be CSS styleshed 1 by 1. This is worst when the <INPUT has autocomplete because lot of CSS must be used for something that is expected: any component/html inside of a GRID item should use all the space assigned to it.
Steps to Reproduce (for bugs)
https://24zr5plwwy.codesandbox.io/
/* eslint-disable flowtype/require-valid-file-annotation */

import React, { Component } from 'react';
import withRoot from '../components/withRoot';
import Grid from 'material-ui/Grid';
import { InputLabel } from "material-ui/Input";
import { Input } from "material-ui";
import Checkbox from 'material-ui/Checkbox';

class Index extends Component {

  render() {
    const factor = {
      "field_id": "1",
      "editLabel": "This is an edit label"
    }
    return (

current behavior: the input do not fill the space assigend by the grid

        <Grid container>
          <Grid item xs={11}>

              <InputLabel style={{ background: 'yellow' }} htmlFor={factor.field_id}>{factor.editLabel}</InputLabel>
              <Input style={{ background: 'cyan' }} id={factor.field_id} onChange={this.handleChange.bind(this)} type="number" />

          </Grid>
          <Grid item xs={1} style={{ background: 'orange' }}>
            <Checkbox onChange={this.handleChange.bind(this)} style={{ background: 'cyan' }} />
          </Grid>
        </Grid>

{"expected behavior: the input field (and selects) should fill all the space assigned to it by the <GRID item>"}

        <Grid container>
          <Grid item xs={11}>

              <InputLabel style={{ display: 'block', background: 'yellow' }} htmlFor={factor.field_id}>{factor.editLabel}</InputLabel>
              <Input style={{ display: 'block', background: 'cyan' }} id={factor.field_id} onChange={this.handleChange.bind(this)} type="number" />

          </Grid>
          <Grid item xs={1} style={{ background: 'orange' }}>
            <Checkbox onChange={this.handleChange.bind(this)} style={{ background: 'cyan' }}/>
          </Grid>
        </Grid>

    );
  }

  handleChange(event) {
    alert(event.target.value);
  }
}

export default withRoot(Index);

An example is in: https://codesandbox.io/s/24zr5plwwy

Context
I believe this is a problem because <INPUT and <SELECT are not following the space assignation rules of the <GRID component, making the <GRID component not so useful to build forms.
Your Environment

Tech
Version

Material-UI
1.0.13beta

React
last version 15.x and 16.0.0

browser
last chrome and firefox on apple and ubuntu

The current behavior is the expected behavior. You need to use the FormControl component in this case. Here is an example https://codesandbox.io/s/8kllkwpvnl.
<Grid container>
  <Grid item xs={10}>

      <FormControl fullWidth>
        <InputLabel style={{ background: 'yellow' }} htmlFor={factor.field_id}>{factor.editLabel}</InputLabel>
        <Input style={{ background: 'cyan' }} id={factor.field_id} type="number" value="3" />
      </FormControl>

  </Grid>
</Grid>

Excellent.
The documentation for option fullWith says "If true, the label will take up the full width of its container.". Perhaps it should say something like "If true, the label and the input/select element will take up the full width of its container." or similar.
I will use that. Thx

@pabloa I like the suggestion. Do you want to update our documentation?

No :)
But I will do it. Let me find out how to do it and I will.
P
On Thu, Oct 5, 2017 at 12:00 PM, Olivier Tassinari<EMAIL_ADDRESS>
wrote:

@pabloa https://github.com/pabloa I like the suggestion. Do you want to
update our documentation?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/callemall/material-ui/issues/8544#issuecomment-334560552,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAk_CzmiLEW5CKYFT1S-Sf2B25MV49lwks5spSdUgaJpZM4PuWWC
.


ENH: examine first line of file for magic string

(Sorry not sorry about the whitespace changes due to my use of black)


[Enhancement]: Navbar addition

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
We can add a navbar to the top to make it look more visually appealing.
Describe the solution you'd like
I would likes to add a navbar with Home, contact us and other buttons.
Therefor i request the project admin @kunjgit  to assign me this issue and label under gssoc
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @aggrhythm !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @aggrhythm ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Update API error message returned when none of a client's claims auth…

This updates the API error message referenced in the ticket to follow the proposed format. I searched for other error messages using the same form as the original message but found none.

Reopening this PR

These changes update the API error messages referenced in the ticket and another with the same form to be more concise and removes non-applicable wording. In addition, tests that include the affected error messages were also updated to align with the new language.


Pass options to fakeserver create 784

This PR fixes the issues from my previous PR #785.
This allows fakeServer to be created with a whitelisted set of settings and adds a configure method which allows the server to be configured from an object literal.
So sorry for the repeated PR's. Realized that in order to safeguard completely against Object.prototype extensions I need to check hasOwnProperty on both the whitelist and settings.

issue #784

Very nice. Could you squash it down to just one commit before we merge it?


The ascii art should write down in line

right now, the ascii art writes down in column, the art being wrote in line would be better

Yo, je vais essayer


Added MouseHover project

Hey Corrected and pushed @thinkswell
https://shivamdangi7.github.io/MouseHover/
Added my contribution a small js based kinda game.

Hey @thinkswell i have corrected
Please review it today is the last day plz

Thanks !


Upgrade to latest version of JET

It would be nice to upgrade to the latest version of JET, since it uses depth-limited stack traces, and may actually be helpful with fixing inference issues. Right now, the log is virtually unreadable.

Closed by #3288.


npm install failed

D:\workspace>npm install --save obj2gltf
npm ERR! Windows_NT 10.0.15063
npm ERR! argv "D:\\app\\nodejs\\node.exe" "D:\\app\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "install" "--save" "obj2gltf"
npm ERR! node v6.10.2
npm ERR! npm  v3.10.10

npm ERR! shasum check failed for C:\Users\bugki\AppData\Local\Temp\npm-11172-df6ce304\registry.npmjs.org\cesium\-\cesium-1.33.0.tgz
npm ERR! Expected: c2471be888dad49a45d47a28cd8c2742eccc34ef
npm ERR! Actual:   1e0a0de7817d955da39ae12cea62cd755325a813
npm ERR! From:     https://registry.npmjs.org/cesium/-/cesium-1.33.0.tgz
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     D:\workspace\npm-debug.log

Thanks for the bug report.  This looks like it may be a corrupted cache issue.  Can you try running these two commands (neither should output anything to the screen):
npm cache clean obj2gltf
npm cache clean cesium

and then re-running npm install --save obj2gltf?
 Windows_NT 10.0.15063
npm ERR! argv "D:\\app\\nodejs\\node.exe" "D:\\app\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "install" "--save" "obj2gltf"
npm ERR! node v6.10.2
npm ERR! npm  v3.10.10
npm ERR! code ENOSELF

npm ERR! Refusing to install obj2gltf as a dependency of itself
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     D:\workspace\obj2gltf\npm-debug.log

Both of these are expected.  In the first case, you are installing as a developer dependency of another node project, but no package.json exists (what makes me assume this isn't what you were attempting to do).  The second case is because you are trying to do the same thing, but add a dependency of obj2gltf to itself.
Are you simply trying to install obj2gtlf to use it as a command line tool?  If so, then the command line you need is npm install -g obj2gltf, the -g indicates you want to install it system wide as a cli tool.  After running that command, you can then just run obj2gltf.
If you're trying to do something else, please provide details.  Thanks.

I'm going to close this, since it's not a bug in obj2gltf, but if you are still having problems @linuxaged, please let us know.


replacing DispatchDB instances with datastore factory

[ ] I have added the tests to cover my changes.
[ ] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

@FyzHsn @AlejandroEsquivel this is a WIP to replace the DispatchDB() instances with the DataStore factory as we discussed on Wednesday
Please feel free to work on this branch, there are still things that need to be updated, but hopefully it's a starting point.

This pull request is stale because it has been in review for at least three days. It will be closed in two more days.

@scottwn Found a syntax error - maybe has something to do with the failing tests. But otherwise, looking good!

@FyzHsn @AlejandroEsquivel I'm seeing this error in logs every time I GET /result/<dispatch_id>.
  File "/Users/scott/covalent/covalent_dispatcher/_service/app.py", line 103, in get_result
    with workflow_db.session() as session:
AttributeError: __enter__

Stackoverflow suggests that it's because I'm not calling the function (I am calling it since the syntax fix that @FyzHsn identified) or because of SQLAlchemy <1.4 (we are on 1.4.37). Any idea how to fix this?

@AlejandroEsquivel another question, why is Session capitalized? shouldn't it be lowercase since it's a class attribute?
https://github.com/AgnostiqHQ/covalent/blob/07f7fb83de078c82d5e1f06f4aab0ae01b868cb2/covalent/_data_store/datastore.py#L54

This pull request is stale because it has been in review for at least three days. It will be closed in two more days.

@AlejandroEsquivel another question, why is Session capitalized? shouldn't it be lowercase since it's a class attribute?
https://github.com/AgnostiqHQ/covalent/blob/07f7fb83de078c82d5e1f06f4aab0ae01b868cb2/covalent/_data_store/datastore.py#L54

@scottwn this is as as far I understand Session is a class, and it also seems to be capitalized in SQLAlchemy documentation so I am capitalizing it for consistency, you can typically import Session directly see here, however you always need to specify an engine to initialize the session. I am using the sessionmaker method to create a new Session reference which can be used in the exact same way but without needing to specify an engine (docs on this here)

@FyzHsn @AlejandroEsquivel I'm seeing this error in logs every time I GET /result/<dispatch_id>.
  File "/Users/scott/covalent/covalent_dispatcher/_service/app.py", line 103, in get_result
    with workflow_db.session() as session:
AttributeError: __enter__

Stackoverflow suggests that it's because I'm not calling the function (I am calling it since the syntax fix that @FyzHsn identified) or because of SQLAlchemy <1.4 (we are on 1.4.37). Any idea how to fix this?

@scottwn if there is a flaw in how the context manager is written one thing to try would be using the Session class directly, therefore:
https://github.com/AgnostiqHQ/covalent/blob/19661f137e0cb04f1bdb0d5db7ecbabc304628f6/covalent_dispatcher/_service/app.py#L103
could be with workflow_db.Session() as session: or with workflow_db.Session.begin() as in the docs https://docs.sqlalchemy.org/en/14/orm/session_basics.html#using-a-sessionmaker

@cjao @FyzHsn this is what's failing in the tests
>           assert (
                load_file(storage_path=lattice_storage_path, filename=lattice_row.executor_filename)
                == "dask"
            )
E           AttributeError: 'Lattice' object has no attribute 'executor_filename'

Was executor_filename removed in v11 (should the test be changed)? Or is the test correct and I have to fix the lattice table?

Hi @scottwn, the test should be updated. V11 split executor_filename into two fields, executor and executor_data_filename.


PHPUnit 7 support

After upgrading to PHPUnit ~7.0 I get the following error:
PHP Fatal error: Declaration of
VCR\PHPUnit\TestListener\VCRTestListener::addError(PHPUnit\Framework\Test $test, Exception $e, $time)
must be compatible with
PHPUnit\Framework\TestListener::addError(PHPUnit\Framework\Test $test, Throwable $t, float $time): void in
[project-path]/vendor/php-vcr/phpunit-testlistener-vcr/src/VCRTestListener.php on line 32

Any plans on releasing support for PHPUnit 7?

Ok it seems like it's only the method declarations that have been updated in PHPUnit, I'll open a PR shortly with a fix.

Nevermind, found there's already an existing PR on this.


Passing traceparent header doesnt appear to work

Hello.
I have followed this guide (and others) but I cannot successfully make a http json request for a grpc endpoint (which has traceparent present in a header of the GET request) to propagate through this OTEL "instrumented" grpc-gateway" following this boilerplate guide.
Use case:
An ingress controller is OTEL enabled and generates a trace/span (sent to OTEL collector) upon http request to a service running in k8s.
The ingress routes that request to the grpc-gateway (compiled for the service) on port 80.
GET /v1/service-name/api/ HTTP/1.1
traceparent: 00-000000000000000010c04a5dc44899bb-83fda3a85333b7a8-01
User-Agent: Go-http-client/1.1
This works when the request is sent to an auto-instrumented flask application (python) so I know the format of the OTEL w3c context is correct
but I am trying to get the grpc-gateway to propagate this so that the span it creates while handling the http request (before it passes toe the grpc service endpoint) is linked as a child of the parent (provided from the inbound http get traceparent).
This guide suggests that setting the otel test map propagator will do that but it doesnt work for me.
I've code inspected the UnaryClientInterceptor here: https://github.com/open-telemetry/opentelemetry-go-contrib/blob/5cc3715df180510fa37f11bd6921c7e546dfcf78/instrumentation/google.golang.org/grpc/otelgrpc/interceptor.go#LL63C61-L63C61
And it seems to not extract the OTEL context but create a new root span and inject that forwards.
This would not propagate the context which is what I am seeing.
Whereas the UnaryStreamInterceptor:
https://github.com/open-telemetry/opentelemetry-go-contrib/blob/5cc3715df180510fa37f11bd6921c7e546dfcf78/instrumentation/google.golang.org/grpc/otelgrpc/interceptor.go#L302
does correctly extract the context
ctx = extract(ctx, cfg.Propagators)
	name, attr := spanInfo(info.FullMethod, peerFromCtx(ctx))
	ctx, span := tracer.Start(
		trace.ContextWithRemoteSpanContext(ctx, trace.SpanContextFromContext(ctx)),

Does this boilerplate guide suggest the it propagates a request (with a traceparent header in the http request) through?
The trace example (which uses a stdout trace exporter) shows no propagation, it displays the trace created by the UnaryClientInterceptor but not in the use case where the http request already has a W3C context present.

I'm seeing similar issue. When calling the grpc endpoint, I see what is expected:
Isito Ingress -> Istio Sidecar -> gRPC-Gateway -> gRPC-Gateway -> Service

However, when calling the HTTP endpoint:
gRPC-Gateway -> gRPC-Gateway -> Service

Traces don't seen to propagate from Istio to the gRPC-Gateway when using HTTP.

because there is no propagation for grpc gateway to extract trace info from http request header then inject trace to grpc request metadata


[Bug] Fix broken test for cache eviction with staging engine

https://github.com/octoml/mlc-llm/blob/batch-serving/serve/tests/unittest/test_staging_engine.py#L216-L217

@elvin-n I understand the problem you encountered in https://github.com/octoml/mlc-llm/pull/158, and the possible solution 1. But I don't understand what you meant in 2., "Do decode inferencing in the worker, but do not return it back to the SequenceGenerationOutput until we achieve number of already generated tokens". Can you elaborate what you meant?

Oh maybe what you meant was: "Replace the clamping in the worker with len(generated_token_ids) steps of decoding, to recover the state before eviction". We now have a better solution using evaluate_multi_query, which can do the same thing as multiple steps of decoding.

Consolidate it into https://github.com/octoml/mlc-llm/issues/163 since the underlying issue is the same.


IndexAdapter breaks with Signed indices

Hello !
I know this is probably a stupid use case, but the code I'm working with does store indices as signed integer, and IndexAdapter seems broken when given such a type of indice
This seems due to
template <typename T, bool ZeroCopy = sizeof(T) == sizeof(unsigned int)>
struct meshopt_IndexAdapter;

unsigned int does have the same size as int, so the adapter does not perform a copy here
The assumption that the size being equals means the types are the same seems dangerous there
Is this something you would be willing to fix, or is this too obscure of a use case ? 😄
Edit: Sorry, I might have been wrong here, I'm trying to debug an issue where I get some seemingly corrupt index value, but that doesn't seem to be the issue after all... I'll investigate, but this might need to be closed

I believe it's correct to skip the copy here - negative values are obviously not correct inputs, but that would usually be flagged by assertions in various algorithms.

Yep, I misdiagnosed an issue I was having. Found the true source, sorry for the noise !


Use google maps provided @Types file and make agm more extensible

Issue description
Make AGM extensible and Use @Types/googlemaps so that features can be added easily
Steps to reproduce and a minimal demo of the problem
In its current form AGM uses custom mapTypes sometimes these Types slightly veer slightly from the googlemaps types and may be the source of some known bugs. I propose we use the @types/googlemaps dependency instead of custom types. This makes AGM easily extensible in most cases. I have made a pull request #1461 that is refactored to use googlemaps and also identified a number of variations from the @types library
What steps should we try in your demo?
you can build https://github.com/wdunn001/angular-google-maps and use it to try it I currently don't have all the tests passing but it runs just fine I use it in my current project.
alternatively you can npm install --save @agme/core which I set up for my use
Current behavior
agm uses custom mapTypes
Expected/desired behavior
use the typings provided by google
angular2 & angular-google-maps version
@latest
Other information
in addition, this pr identified numerous logic issues such as recasting of already typed values and redundant splice logic. some changes were required to make this compatible with googlemaps such as using a map(f => l.add(f)) to populate the features  with the built in google maps funtion.
I found enum's to be particularly problematic I implemented a switch case that could probrably be switched to a map.

Anyone help?
It seems that there is a namespace conflict between trivial google object windows.google and namespace google.maps. in the @types/googlemaps package.
The compiler says
error TS2503: Cannot find namespace 'google'
I'm using these package versions
"@agm/core": "^1.0.0-beta.3",
"@types/googlemaps": "^3.30.12",
and here below there is is my service:
import { Injectable } from '@angular/core';
import { MapsAPILoader } from '@agm/core';

import Geocoder = google.maps.Geocoder; // <-- this conflict with the line below and in the constructor
declare const google: any;              // <--

@Injectable({ providedIn: 'root' })
export class GeocoderService {

  private geocoder: Geocoder;

  constructor(private mapsApiLoader: MapsAPILoader) {
    this.mapsApiLoader.load()
      .then(() => {
        this.geocoder = new google.maps.Geocoder(); <--
      });
  }
}

this will actually require structural changes to the rest of the project to perform.  here is an example of circle-manager using google map types.
import {Injectable, NgZone} from '@angular/core';
import {Observable, Observer} from 'rxjs';
import {AgmCircle} from '../../directives/circle';
import {GoogleMapsAPIWrapper} from '../google-maps-api-wrapper';

@Injectable()
export class CircleManager {
  private _circles: Map<AgmCircle, Promise<google.maps.Circle>> =
      new Map<AgmCircle, Promise<google.maps.Circle>>();

  constructor(private _apiWrapper: GoogleMapsAPIWrapper, private _zone: NgZone) {}

  addCircle(circle: AgmCircle) {
    this._circles.set(circle, this._apiWrapper.createCircle({
      center: {lat: circle.latitude, lng: circle.longitude},
      clickable: circle.clickable,
      draggable: circle.draggable,
      editable: circle.editable,
      fillColor: circle.fillColor,
      fillOpacity: circle.fillOpacity,
      radius: circle.radius,
      strokeColor: circle.strokeColor,
      strokeOpacity: circle.strokeOpacity,
      strokePosition: circle._strokePosition,
      strokeWeight: circle.strokeWeight,
      visible: circle.visible,
      zIndex: circle.zIndex
    }));
  }

  /**
   * Removes the given circle from the map.
   */
  removeCircle(circle: AgmCircle): Promise<void> {
    return this._circles.get(circle).then((c) => {
      c.setMap(null);
      this._circles.delete(circle);
    });
  }

  setOptions(circle: AgmCircle, options: google.maps.CircleOptions): Promise<void> {
    return this._circles.get(circle).then((c) => c.setOptions(options));
  }

  getBounds(circle: AgmCircle): Promise<google.maps.LatLngBounds> {
    return this._circles.get(circle).then((c) => c.getBounds());
  }

  getCenter(circle: AgmCircle): Promise<google.maps.LatLng> {
    return this._circles.get(circle).then((c) => c.getCenter());
  }

  getRadius(circle: AgmCircle): Promise<number> {
    return this._circles.get(circle).then((c) => c.getRadius());
  }

  setCenter(circle: AgmCircle): Promise<void> {
    return this._circles.get(circle).then(
        (c) => { return c.setCenter({lat: circle.latitude, lng: circle.longitude}); });
  }

  setEditable(circle: AgmCircle): Promise<void> {
    return this._circles.get(circle).then((c) => { return c.setEditable(circle.editable); });
  }

  setDraggable(circle: AgmCircle): Promise<void> {
    return this._circles.get(circle).then((c) => { return c.setDraggable(circle.draggable); });
  }

  setVisible(circle: AgmCircle): Promise<void> {
    return this._circles.get(circle).then((c) => { return c.setVisible(circle.visible); });
  }

  setRadius(circle: AgmCircle): Promise<void> {
    return this._circles.get(circle).then((c) => { return c.setRadius(circle.radius); });
  }

  createEventObservable<T>(eventName: string, circle: AgmCircle): Observable<T> {
    return new Observable((observer: Observer<T>) => {
      let listener: google.maps.MapsEventListener = null;
      this._circles.get(circle).then((c) => {
        listener = c.addListener(eventName, (e: T) => this._zone.run(() => observer.next(e)));
      });

      return () => {
        if (listener !== null) {
          listener.remove();
        }
      };
    });
  }
}

+1
I want MVCObject…specially #set #get
agm types only defines addlistener…

i followed this approach :
https://stackoverflow.com/a/55451667/1427338
/* /src/google-maps.d.ts */
import '@types/googlemaps';

declare global {
  interface Window {
    google: typeof google;
  }
}

/* /tsconfig.json  */
{
  ...
  "compilerOptions": {
    ...
    "typeRoots": [
      "node_modules/@types",
      "src/google-maps.d.ts"
    ],
   ...
  }
}

export classMyComponent implements OnInit {

    lat: number ;
    lng: number ;
    drawingManager: google.maps.drawing.DrawingManager;

    ngOnInit() {
        this.drawingManager = new google.maps.drawing.DrawingManager({
            // ...
        })
    }
}

NB: Don't forget to restart the active ng serve session


cron table overload

Thank you for your module, I installed it, but my cron schedule table has more than 20000 pending jobs.
I know your module prevents that several cron jobs run at the same time, but how can I prevent that so many cron jobs are created.
Looking forward to hearing from you
Brigitte

When did you install it? I found instances where if old data is in the from table it will create extra entries but that was fixed this morning.

Dear Martin,
Thank you so much for your answer, I downloaded the files on march 19th  and installed it yesterday.
I had old data in the cron schedule tab. I deleted all entries this afternoon and now I have again 5000 entries pending.
Should I reinstall the new version, you corrected today ?
Kind regards
Brigitte
De : Martin Pachol<EMAIL_ADDRESS>Envoyé : March-21-18 5:01 PM
À : magemojo/m2-ce-cron
Cc : brigittevelten; Author
Objet : Re: [magemojo/m2-ce-cron] cron table overload (#17)
When did you install it? I found instances where if old data is in the from table it will create extra entries but that was fixed this morning.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/magemojo/m2-ce-cron/issues/17#issuecomment-375123840 , or mute the thread https://github.com/notifications/unsubscribe-auth/AiGIKy1xvZPckJtox5lIyc2b5PcveA1Xks5tgtu7gaJpZM4S2H0q .  https://github.com/notifications/beacon/AiGIK9Oh0-V5ncf1IPmPMMzAlltuzSuRks5tgtu7gaJpZM4S2H0q.gif

Yes, please try the latest, that will force everything to UTC time and should prevent that. The new version also has cron reporting in the admin.

Dear Martin,
Thank you so much, I really appreciate your help.
Brigitte
De : Martin Pachol<EMAIL_ADDRESS>Envoyé : March-21-18 10:52 PM
À : magemojo/m2-ce-cron
Cc : brigittevelten; Author
Objet : Re: [magemojo/m2-ce-cron] cron table overload (#17)
Yes, please try the latest, that will force everything to UTC time and should prevent that. The new version also has cron reporting in the admin.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/magemojo/m2-ce-cron/issues/17#issuecomment-375179177 , or mute the thread https://github.com/notifications/unsubscribe-auth/AiGIK5x-7b0cRwYecQCV8JDsHGGqqF4Rks5tgy3agaJpZM4S2H0q .Image supprimée par l'expéditeur.

No problem, going to close this issue out.


Update to v0.0.4

Changes

Update to v0.0.4
Make requirements consistent with upstream
Update tests
Change recipe format to comply with new recipe standards

Package Info
Upstream: https://github.com/anaconda-distribution/anaconda-linter/tree/0.0.4
Change log: https://github.com/anaconda-distribution/anaconda-linter/blob/0.0.4/CHANGELOG.md

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_29lr22y1r4/clone/recipe/meta.yaml:60: missing_license_url: The recipe is missing a license_url
WARNING: /tmp/abs_29lr22y1r4/clone/recipe/meta.yaml:60: missing_doc_source_url: The recipe is missing a doc_source_url
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_d19r1hfmyb/clone/recipe/meta.yaml:58: missing_license_url: The recipe is missing a license_url
WARNING: /tmp/abs_d19r1hfmyb/clone/recipe/meta.yaml:58: missing_doc_source_url: The recipe is missing a doc_source_url
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_a8n4i_rx11/clone/recipe/meta.yaml:58: missing_doc_source_url: The recipe is missing a doc_source_url
WARNING: /tmp/abs_a8n4i_rx11/clone/recipe/meta.yaml:58: missing_license_url: The recipe is missing a license_url
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_870df339ut/clone/recipe/meta.yaml:59: missing_doc_source_url: The recipe is missing a doc_source_url
WARNING: /tmp/abs_870df339ut/clone/recipe/meta.yaml:59: missing_license_url: The recipe is missing a license_url
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_94bzq1cql5/clone/recipe/meta.yaml:59: missing_license_url: The recipe is missing a license_url
WARNING: /tmp/abs_94bzq1cql5/clone/recipe/meta.yaml:59: missing_doc_source_url: The recipe is missing a doc_source_url
Warnings were found


claim aud is not compliant with RFC7519

It is stated that aud is an array of strings, ref: https://tools.ietf.org/html/rfc7519#section-4.1.3
Please note It is impossible to assign an array of strings: https://github.com/dgrijalva/jwt-go/blob/dc14462fd58732591c7fa58cc8496d6824316a82/claims.go#L19

Ignore: duplicate of: https://github.com/dgrijalva/jwt-go/issues/348


[WIP][POAE7-287]Add shell to detect AEP so as to cache backend fallback

@offthewall123  Could you please help to review this prototype?

Add [OAP-CACHE] to the title and submit an issue and add issue id. Can refer to this PR's title
https://github.com/Intel-bigdata/OAP/pull/1740


Ospp 2024/004 renyi fix crypto

PR
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our Commit Message Guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

@18003228289 PR标题编写得不合规范，请参考贡献指南


Get 1.8 build compiling without errors

Have already created 1.8.9 branch for this and have started updating for forge 1722.

Good checklist of gotchas at:
http://jabelarminecraft18.blogspot.com/p/minecraft-forge-converting-mods-from.html

As of f00e8a7, we are down to 187 errors :stuck_out_tongue_closed_eyes:

As of 0c978bfe, it compiles. But a lot of things (AI and Layer rendering) were disabled to get to this point.


feat: add tooltip to pools and refactor loans tooltip

Interbtc UI Pull Request Template
Description
Add tooltip to pools and refactor loans tooltip existing code.

@peterslany I still get 0% trading APR in pools


Feature added generator for action and srv

With this PR you can also generate .action and .srv files
The generator has two steps, it converts the file into separate msgs files in a tmp dir and uses then the same generator logic as before

@MrBlenny
Hello,
in the last commit you changed the naming of the generated msgs to _ActionResult
The problem with this is, that on the generation step we don't find now the msgs, because of the other name.
What was the thoughts behind this changes ?
Thanks for the response


Do we need to use global or can we test for window

Scrumfun are using Vite and they hit an issue here because Vite does not expose global.
Can we guard this checking for window first?

we can add it by default if it does not exist

Ill take care of it