The connect step

I am trying to understand the XNL/XCMP protocol. Currently, I am stuck in the connect operation which requires some key which you derive from TRBOnet.Server.exe when it isn't set. Is there any other way to get this key to pass the connection?

The other option is to reverse engineer the constants here:
https://github.com/pboyd04/Moto.Net/blob/master/Moto.Net/Mototrbo/XNL/Encrypter.cs#L64C13-L70C1
And put them in your app.config file. I haven't included them in my attempt to not get sued by Motorola.

I am working with XNL/XCMP protocols and need help obtaining the encryption keys referenced in the Encrypter.cs file (XNLConst1, XNLConst2, etc.). Is there a way to derive or access these keys without using TRBOnet.Server.exe?
Any guidance or documentation would be appreciated.
Best regards,

The best way to get them is via some sort of side band attack. It is a version of Tiny Encryption Algorithm (TEA) if you want to Google the best attacks. Worst case given input and output (wireshark) of licensed Mototrbo software like RDAC or the referenced TRBONet you can brute force the constants. That would need about a year or so to perform on my current GPU that is a couple of generations out of date. I'm not going to provide those constants unless I get a letter from Motorola legal saying that they aren't proprietary and I won't be sued for releasing them.

Thank you for your response. I understand the constraints regarding the encryption keys. However, I am willing to cover any costs associated with obtaining the necessary keys through official channels. Could you please provide further details on how I can proceed or direct me to the appropriate contact to facilitate this process?
Your assistance is greatly appreciated.
Best regards,

I'm honestly not 100% certain on how you would get started on paying for this. I would probably start here: https://www.motorolasolutions.com/en_us/about/environmental-social-corporate-governance-esg/our-business/technology-leadership/technology-licensing/licensing-process.html
And hopefully they can help you or direct you where to go.


Typescript failed after install

VSCode Version: Code 1.12.2 (19222cdc84ce72202478ba1cec5cb557b71163de, 2017-05-10T13:20:36.315Z)
OS Version: Windows_NT ia32 6.3.9600
Extensions: none

Steps to Reproduce:

With VS code closed, Install Typescript using normal methods
Open VS Code and error showed at the top of IDE

What error are you seeing?

Closing as we need more information to investigate this


debug console scrollable horizontally, just like code

From @joeblew99 on November 23, 2015 15:11
would make debugging much smoother is the debug console window was scrollable horizontally.
in a code window its nice that i can scroll easily to see long longs.
If there is a way to maybe force the debug console window to flow onto the next line that would be also good.
Copied from original issue: Microsoft/vscode-go#66

Yes - one or both of these does seem necessary to make the debug console usable.

Thanks.
This ide is really nice to work in compared to the others I have tried for golang.

We are reluctant to add horizontal scrolling as no other terminal / repl has this and would be strange.
Flowing text to the next line - word wrap is the way to go. We will look into this.

Certainly understand. Thanks for feedback.

If the debug console cannot be docked horizontally, is it possible to hide it / keep it minimized?

Not at the moment, will be possible in the future.


Add add e2e functional test coverage

This improves the functional test coverage so that we test all condition steps until the overall condition becomes True.

/test horizon-operator-build-deploy-kuttl
tls: handshake failure


[Discrete Simulator Project] How do you differentiate level 5 from the rest?

The generation of Arrival Events for level 5 is different, how do you guys know when it is level 5 or when it is other levels? Do you create a another simulator (e.g. Simulator5 for level 5) or use another constructor ?

For me, the simulator has several variables
ArrayList of arrivalTimes
ArrayList of restTimes
ArrayList of serviceTimes
In level 5, I only know arrivalTimes (the only one that can be constructed from the beginning). So other ArrayLists will be empty. So i just check
if (restTimes.size() != 0) {
         NOT LEVEL 5
} else {
         LEVEL 5
}

For me, the simulator has several variables ArrayList of arrivalTimes ArrayList of restTimes ArrayList of serviceTimes
In level 5, I only know arrivalTimes (the only one that can be constructed from the beginning). So other ArrayLists will be empty. So i just check
if (restTimes.size() != 0) {
         NOT LEVEL 5
} else {
         LEVEL 5
}

Oh wow, that is a good idea. Thank you Albert !

Hi I think another way is to have a Randomgenerator that takes in arrivalRate, serviceRate, restTime for your level 5 constructor, while ArrayLists for arrivalTimes, serviceTimes and restTimes can be empty. For the other levels RandomGenerator can be set to an empty array or optional, so you can check for Randomgenerator to see if it's level 5

Don't even risk overloading LOL. I made a new method entirely named simulate5

I overloaded my simulate method that takes in all the new variables and created a new method from there

Don't even risk overloading LOL. I made a new method entirely named simulate5

Why is overloading a risk ?

Hi I think another way is to have a Randomgenerator that takes in arrivalRate, serviceRate, restTime for your level 5 constructor, while ArrayLists for arrivalTimes, serviceTimes and restTimes can be empty. For the other levels RandomGenerator can be set to an empty array or optional, so you can check for Randomgenerator to see if it's level 5

Nice ! I think that is a good idea, rather than checking for the variables

I overloaded my simulate method that takes in all the new variables and created a new method from there

What do u put for the variables that are not used for lower levels? (e.g. probabilities of greedy, etc)

Hi I think another way is to have a Randomgenerator that takes in arrivalRate, serviceRate, restTime for your level 5 constructor, while ArrayLists for arrivalTimes, serviceTimes and restTimes can be empty. For the other levels RandomGenerator can be set to an empty array or optional, so you can check for Randomgenerator to see if it's level 5

This was what i did as well, then anywhere in your implementation of your simulate method, when you need to differentiate between level 5 and other levels you can always check if the Optional is empty. Of course, you can't use Optional::isEmpty or Optional::isPresent, but there are many ways to get around this

Me? I just combined everything together. Customer get Lazy for their service time and I wrapped all the information into an immutable SimulatorData class that generates customers and rest times.


docs: explain how to set document config

there is a small things that users need to be aware of when setting configs, so i am explaining that here.
TODO: small test done

@samsja what will be the effect of pydantic v2 on this?

@samsja what will be the effect of pydantic v2 on this?

this way will still be compatible with pydantic v2 but pydantic will show a warning saying it is deprecated. I suggest we update the docs after the merge


Testing skip coverage for forks

@em-pe Didn't work I assume?

@szczeles No it did not as it executes build in target project. I'm still looking for a way to distinguish fork build from the standard one.


journal: add log namespace support

Closing for now, this requires newer python3-systemd and systemd versions, which will be available in F32.
https://app.clubhouse.io/aiven/story/17541/make-sure-tlog-auditing-doesn-t-end-up-in-journal-on-the-node-side-to-avoid-recursion


Allow for unsubscription

Right now there is no way to unsubscribe an element from redux state changes. This could become a memory leak issue and needs to be addressed.

Done


TextArea

Hi  AOA Plz Help Me what Was the Format Of TextArea To Show Blank Message In texrarea Field Plzz Upload here Texara code Of Java Script in presence
?

TeachingCareer: {
        presence: true,
    },

Its not Working
HownTo show Blank Message For This ??

Try this:    presence: { message: 'should not be blank',allowEmpty: false }


error in code

rdio_export_to_sqlite.py
click.secho("collection.csv doesn't exist, bailing", fg=red)
should be
click.secho("collection.csv doesn't exist, bailing", fg='red')
I think anyway!!

For small stuff like this it's usually okay to make a PR, makes it way easier to fix, just have to click the merge button. I edited the file directly in github and made a PR https://github.com/jacobian/rdio2spotify/pull/4.

Fixed in #4


Memory leak with addCurve

Intensive use of addCurve looks to leak memory.
Here is a script from @t20100 reproducing the problem.
    def run(self):
        """Method implementing thread loop that updates the plot"""
        while self.running:
            time.sleep(0.1)

            # Run plot update asynchronously
            x = numpy.arange(1000)
            data = numpy.random.random(4000).reshape(4, 1000)
            for index in range(4):
                concurrent.submitToQtMainThread(
                    self.plot1d.addCurve,
                    x,
                    data[index],
                    resetzoom=False,
                    legend=str(index))
            #concurrent.submitToQtMainThread(self.plot1d.clear)

Here is the result in Flint after few times (memory (KB) per time is second)

Have you tried to change backend (Matplotlib and OpenGL) to see if the problem is at the backend side or at our side?

Yes it's the same. With OpenGL the leak is smaller, but there is still something.

Here is a minimalistic sample code to reproduce:
import numpy

from silx.gui import qt
from silx.gui.plot import PlotWidget

app = qt.QApplication([])
plot = PlotWidget()
plot.show()

x = numpy.arange(1000)
y = numpy.random.random(1000)
legend = 'curve'

plot.addCurve(x, y, resetzoom=False, legend=legend)
curve = plot.getItems()[0]

def timeout():
    curve.setData(x, y)

timer = qt.QTimer()
timer.timeout.connect(timeout)
timer.start(100)

app.exec_()

Hi, I've tried the "minimalistic" script and I do not see any memory leak.
I monitored the process memory with:
taurus trend 'eval:@p=psutil.Process($PID)/p.memory_info().rss/1024'

... where $PID is the process ID of the silx plot script
Note: the script does not change the values of x,y (just sets the same again and again). Is that on purpose?
just in case I tried recomputing x,y inside the timeout function so that new values are used. No mem leak either.
I also tested the equivalent code with pyqtgraph with same negative results.

Note: the script does not change the values of x,y (just sets the same again and again). Is that on purpose?
Yes idea was to check if there is any memory leak in the machinery, whether data is different or not should not change the behavior.

I'm using mprof run from memory-profiler (which looks to use psutil), and I see a small increase over time:

But I'm not sure I'm looking at the right thing, or maybe memory usage stop increasing after a while...

Hi, here is what I get with mprof run + mprof plot using the "minimal" script (I have only run it for ~2min, but I think it is safe to say that there is no memleak)

For comparison, this is what I get from doing the same with pyqtgraph

note: in my tests I avoiding doing any other thing with the PC at the same time, to avoid "contaminating" the results with moving windows, etc.

OK, it looks like what I would expect.
I run it without doing anything else and here is the result, it stabilises after a while:

BTW, I'm on macos so things are probably different underneath.

Sounds like you don't check the same thing as me.

I tried to simplified the pattern as much as I can... but probably too much as I got lured by the increase when doing quick tests.

Here is mine.
If you use a single curve you don't have the leak.
Spend so much time for a so simple sample that i can't really believe this result.
import sys
import os
import numpy
from silx.gui import qt
from silx.gui.plot.PlotWindow import Plot1D

def create_window():
    plot = Plot1D(None)

    x = numpy.zeros((1,))

    def update():
        plot.addCurve(x, x, legend="a")
        plot.addCurve(x, x, legend="b")

    timer = qt.QTimer(plot)
    timer.setInterval(1)
    timer.timeout.connect(update)
    timer.start()

    return plot

def main():
    qapp = qt.QApplication([])
    window = create_window()
    window.setWindowTitle(f"{os.getpid()}")
    window.setVisible(True)
    sys.exit(qapp.exec_())

if __name__ == "__main__":
    main()

Found that matplotlib.cbook.CallbackRegistry.disconnect is never called.
As result the dict self._func_cid_map[s] grow up always, storing weakref.
But most probably the problem come from something else.

Here is a pure MPL example.
I think it is more or less what we do in silx.
import sys
import os
from PyQt5 import Qt

from matplotlib.backends.backend_qt5agg import FigureCanvasQTAgg
from matplotlib.figure import Figure
from matplotlib.container import Container

class PlotCanvas(FigureCanvasQTAgg):

    def __init__(self, parent=None, width=5, height=4, dpi=100):
        fig = Figure(figsize=(width, height), dpi=dpi)
        self.axes = fig.add_subplot(111)

        FigureCanvasQTAgg.__init__(self, fig)
        self.setParent(parent)

        self.l1 = None
        self.l2 = None

        timer = Qt.QTimer(self)
        timer.setInterval(1)
        timer.timeout.connect(self.tick)
        timer.start()

    def tick(self):
        if self.l1 is not None:
            self.l1.remove()
        if self.l2 is not None:
            self.l2.remove()

        self.l1 = Container([self.axes.plot([1,2], [1,2])])
        self.l2 = Container([self.axes.plot([2,1], [1,2])])

        self.draw()

def main():
    app = Qt.QApplication(sys.argv)
    window = PlotCanvas()
    window.setWindowTitle(f"{os.getpid()}")
    window.setVisible(True)
    return app.exec_()

if __name__ == "__main__":
    result = main()
    sys.exit(result)

It looks like it is fine if we use another container, that's really matter of timing.
class MyContainer:

    def __init__(self, axes, artists):
        self.axes = axes
        self.artists = artists

    def remove(self):
        artists = self.artists # without this 2 lines, it still fails
        self.artists = None  # need python expert here o_O or a sleep
        for a in artists:
            a.remove()

Found a fix. Sounds like a pure mpl bug.

If you are about to release silx soon, would you like a kind of monkey patch before the fix in matplotlib?
I am afraid i will have to do it anyway for Flint, so i can patch silx instead.
What do you think?

I'd prefer a workaround on our side rather than a monkey patch... but in this case it's probably complicated.
Looking at the history of CallbackRegistry, it didn't changed much in the last few years... so I expect we can safely monkey patch it for a wide-enough range of versions without testing with each individual version.

We did that in Flint: https://gitlab.esrf.fr/bliss/bliss/-/merge_requests/3469. It's fine for me.
Also, up to you if you want to close the issue, or let it open until it is not fixed in matplotlib.

OK, then unless someone else thinks that we need to fix it from silx, I'd leave it as it is since it should be fixed in matplotlib.

The patch was released with matplotlib 3.4.0 RC1.
Then i close this issue.


refreshToken function isn't invoked

First of all, Thank you @agordn52 for this great framework.
I'm following the docs, and what has been mentioned in the issues/discussions.
refreshToken function isn't being invoked at all.
  @override
  refreshToken(Dio dio) async {
    AuthTokens? tokens = await StorageKey.userToken.read<AuthTokens>();
    if (tokens?.refreshToken == null) {
      await Auth.logout();
      routeToInitial(navigationType: NavigationType.pushAndForgetAll);
      return;
    }
    dynamic response = (await dio.post(baseUrl + "/auth/refresh",
            options: Options(
                headers: {"Authorization": "Bearer ${tokens?.refreshToken}"})))
        .data();
    //  Save the new token
    await StorageKey.userToken.store(response);
  }

Noting that the approach was mentioned, I have to retry manually.
Did I miss anything?
If we refresh on request instead, would it be better? but what if I run parallel requests?

Update [29-01-2024]
This is the interceptor approach.
class BearerAuthInterceptor extends Interceptor {
  @override
  void onRequest(
      RequestOptions options, RequestInterceptorHandler handler) async {
    if (!options.path.contains("auth/refresh")) {
      await api<ApiService>((request) => request.refreshTokens());
    }
    AuthTokens? tokens = await StorageKey.userToken.read<AuthTokens>();
    String? token = options.path.contains("auth/refresh")
        ? tokens?.refreshToken
        : tokens?.accessToken;

    if (token != null) {
      options.headers.addAll({"Authorization": "Bearer $token"});
    }
    return super.onRequest(options, handler);
  }

the refreshTokens function checks if a refresh token is needed then proceed.
if condition to eliminate the recursion.
I had to skip the previous approach, but I still do not know how reactive this approach is. Especially with parallel requests scenario.

Hello @agordn52,
Yes, and during the debug, it is being invoked. However, refreshToken isn't invoked at all I have no clue why.

Hi @KhaledAlMana,
Thanks for sharing your code.
I see a few things wrong that should be changed, please try the below 👍
@override
Future<RequestHeaders> setAuthHeaders(RequestHeaders headers) async {
    AuthTokens? tokens = await StorageKey.userToken.read<AuthTokens>();
    if (tokens?.accessToken != null) {
      headers.addBearerToken(tokens?.accessToken ?? "");
    }
    return headers;
}

@override
Future<bool> shouldRefreshTokens() async {
  AuthTokens? tokens = await StorageKey.userToken.read<AuthTokens>();
  if (tokens?.accessToken == null) {
    return true; // if accessToken is null then you should want to get a new 'AuthTokens'
   }
   // If the token is expired, where expiredAt is epoch time
  // add 15 seconds to the expiry time to account for latency
  if (DateTime.now().isAfter(tokens?.expiresAt?.subtract(Duration(seconds: 15)) ?? DateTime.now())) {
        return true;
  }
  return false;
}

@override
refreshToken(Dio dio) async {
    // debug is not hitting this function, nor printing the below.
    print("refreshing token");
    AuthTokens? tokens = await StorageKey.userToken.read<AuthTokens>();
    print(tokens?.refreshToken);
    if (tokens?.refreshToken == null) {
      await Auth.logout();
      routeToInitial(navigationType: NavigationType.pushAndForgetAll);
      return;
    }
    dynamic response = (await dio.post(baseUrl + "/auth/refresh",
            options: Options(
                headers: {"Authorization": "Bearer ${tokens?.refreshToken}"})))
        .data; // date() not a function, use data instead
    //  Save the new token
    await StorageKey.userToken.store(response);
}

I'm going to close this as resolved.
If you need help understanding networking, the docs on nylo.dev are up-to-date so you'll be able to learn more 👍

Sorry, I have been busy lately.
I got the  .data;  issue but the problem is that the function refreshToken is still not being called.
After the latest update, I tested it again, it works fine.
Thank you @agordn52 for your support.


Prevent MapView from re-rendering on Marker position change

Is this a bug report?
No
Have you read the Installation Instructions?
Yes
Environment
react-native: 0.58.6
react: 16.8.4
react-native-maps: 0.24.0

Steps to Reproduce

Render a MapView with dynamic latitude and longitude
Add a Marker
Change the Marker position

Expected Behavior
I would like the MapView to not re-render when the Marker position change, as it causes the map to focus on the initial region.
Actual Behavior
When I change the Marker position, the entire map seems to re-render and focus on the initial region.
Reproducible Demo
  componentDidMount() {
    navigator.geolocation.getCurrentPosition(
      position => {
        this.coords = position.coords
      },
      error => console.warn(error)
    )
  }

  setLocation(e) {
    this.setState({ selectCoordinates: e.nativeEvent.coordinate })
  }

  render() {
    if (!this.coords) return null

    return (
      <View>
        <MapView
          provider={PROVIDER_GOOGLE}
          style={styles.map}
          region={{
            latitude: this.coords.latitude,
            longitude: this.coords.longitude,
            latitudeDelta: 0.015,
            longitudeDelta: 0.0121
          }}
          onPress={e => this.setLocation(e)}
        >
          {this.state.selectCoordinates && (
            <Marker coordinate={this.state.selectCoordinates} />
          )}
        </MapView>
      </View>
    )
  }

@matthieu994 this is a question better suited for stackoverflow
but use initialRegion instead of region it should work
see
https://github.com/react-native-community/react-native-maps/blob/master/README.md

@ma

@matthieu994 did you find the solution, i am facing same problem


Fix producer stucks on creating ledger timeout

Motivation
The ledgerCreated flag is passed as ctx to the createLedger callback.
The callback already had the logic on handling ledgerCreated flag. But we set the flag to false
when timeout happens. It will trigger the following race condition:
a) The createComplete callback is triggered when timeout. But the pending add requests are not error'd out.
b) If the createComplete callback eventually completes, it will see ledgerCreated flag is set to true,
so it will cause checkAndCompleteLedgerOpTask returns false and exist too early without processing the
pending add requests.
This race condition only happens when creating ledger times out.
    public synchronized void createComplete(int rc, final LedgerHandle lh, Object ctx) {
        if (log.isDebugEnabled()) {
            log.debug("[{}] createComplete rc={} ledger={}", name, rc, lh != null ? lh.getId() : -1);
        }

        if (checkAndCompleteLedgerOpTask(rc, lh, ctx)) {
            return;
        }

Modification
The timeout logic shouldn't modify the ledgerCreated context. It should let the callback itself to process
the ledgerCreated context.

/pulsarbot run-failure-checks

Hi,
We are facing this issue in chaos monkey type of testing of Pulsar, with Pulsar version 2.5.2 .
It's great that this issue is already fixed. Will this fix be available in Pulsar 2.6.1 ? How about Pulsar 2.5.3 ?
Thank you!


feat!: have oldVal passed on notify

Continuation of #236
There's still some work with writing documentation and improving the typing. Just wanted to post it now that all the previous tests are passing, to make sure the API changes are what we want.

@ai - I've expanded on the work of @y-nk and taken the PR to a working, but not fully reviewable state. Just wanted to make sure that the resulting API changes are what you had in mind before I spend the time on polish.

Looks good.

Then I think it's ready for review 😄


Feature/ecc errors

Hi thanks for the note.  At the moment I'm not coupled enough to the code to become a maintainer.  Like you my use of nvml for work only is pretty limited and I am only doing it because the support in the official repo requires the library to be present.  My hope is that the Nvidia official code improves how it couples to the callers code and the need for this goes away.
So I am not advertising any changes other than maintaining the fork back to your original code.  Someone that is prepared to jump the hoops to review the forks is probably ready to take their own fork from any one of the three or four that exist.  I think thats sufficient, I think its the right thing to do ?
In the meantime I will close the PR.  Thanks for the message.


[New model] Kermut

Hi ProteinGym team!
We would like to submit our model, Kermut, to the supervised substitution benchmark leaderboards, where it achieves state-of-the-art performance across splits.
All code required to run the model has been added to proteingym/baselines/kermut and model details have been added to proteingym/constants.json. We have included a README with detailed instructions on how to access either raw or preprocessed data (recommended) and train/evaluate the model. Our data directory matches that of ProteinGym and should work seamlessly if scripts are called from the ProteinGym root directory. Aggregated results across assays can be found in kermut/results/summary, while the per-variant predictions can be accessed via the README.
To reproduce results, the preprocessed data (e.g., precomputed embeddings) can be downloaded and unzipped as described, after which either example_scripts/benchmark_single_dataset.sh or example_scripts/benchmark_all_datasets.sh can be run. We have a script to merge all individual results (src/results/merge_score_files.py), the output of which can be used directly with performance_DMS_supervised_benchmarks.py. Note that our environment has additional dependencies, all of which are listed in kermut/environment.yaml.
The results in results/summary are from the updated CV splits (see issue), while results for the old splits which can be directly compared to the current leaderboard can be found in results/summary_old.
Thank you for your efforts!
Peter

Code has been streamlined and refactored for easier use.


Format code block in comment

Closes #554.
Closes #1695.

Awesome! Does this work with lines starting with # (to hide them in rustdoc's rendered output)?

Nice!

@killercup This does not work with lines starting with #, they will not be formatted. I will create a follow up PR to fix it.

Cool. It might be a bit tricky to format the hidden lines while also making
the visible ones look correctly indented. Let me know if you need some
weird test cases!
Seiichi Uchida<EMAIL_ADDRESS>schrieb am Mi. 27. Dez. 2017 um
11:47:

@killercup https://github.com/killercup This does not work with lines
starting with #, they will not be formatted. I will create a follow up PR
to fix it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang-nursery/rustfmt/pull/2311#issuecomment-354094905,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABOX0ra36aOY9Z97ZdMGRaGAsq8ppz7ks5tEiA_gaJpZM4RL8Xv
.


WIP: Enable Chakra.ICU on ARM

This is the next major step in converting to ICU on all platforms. This change converts Chakra.ICU.Toolutil and Chakra.ICU.GenCCode to be built to run on the host architecture (usually x64) in order to generate icudtl_dat.obj, which is platform-agnostic and can be linked into an ARM-compatible DLL using the regular ARM linker. As a downside, it means that a clean build of Chakra.ICU takes about twice as long because ICU's common and i18n code are built twice, once for the target that gets linked to the resulting ChakraCore.dll, and once for the host that gets linked to GenCCode.exe. This is going to be a major problem for CI, however we can (hopefully) solve that problem later through caching
/cc @Microsoft/target-dev (I cant add target-dev as a reviewer for unknown reasons). I think this will also enable ARM64, however I haven't checked that yet.

Thanks to @Cellule for the idea of building the host-specific tools in a separate PowerShell script!

@MSLaguana suggested adding a flag to control building the data file, since for ARM in CI scenarios, we never actually use the data file (or anything, really, since those builds only build and don't run tests). I think I will update the build system with a new flag to control building the data file that we can turn off by default for certain CI scenarios.

Confirmation that the resulting binaries actually run on ARM:

/cc @Microsoft/target-dev (I cant add target-dev as a reviewer for unknown reasons).

I just added @Microsoft/target-dev as a Read-access contributor team to Microsoft/ChakraCore so you should be able to add that list as a reviewer now.


Update AppCfg/Gcloud/DevAppServer based command API

This is the inclusion of the major API changes as described in this comment : https://github.com/GoogleCloudPlatform/appengine-plugins-core/pull/579#issuecomment-380243084

@elharo, can you re-review, the old approval got dismissed.

merged.


[PE-D][Tester D] Left-click on Student Panel Card does not work as expected

The UG states that users can view a single student's details in a formatted and organised manner by left-clicking on a specific Student Panel Card within the Student List Panel. However, a single-left click does not yield this result, it requires two left clicks! A simple fix would be to specify that it is a double click :)
Your browser does not support the video tag.video:https://raw.githubusercontent.com/lim-jt/ped/main/files/8cd6f8d5-905d-43fd-8420-95c5819c8192.mov

Labels: severity.VeryLow type.FunctionalityBug
original: lim-jt/ped#1

Will update in user guide. Double-click is needed to view a single student's details.


DNS not working in homeassistant container

I'm running hassos inside a Virtualbox 6 Server.  (Tested on Fedora 30 and Windows 10)
arch | x86_64
dev | false
docker | true
hassio | true
os_name | Linux
python_version | 3.7.4
timezone | America/Curacao
version | 0.97.2
virtualenv | false
Description of problem:
hassos is set with an static IP (<IP_ADDRESS>),  resolv.conf:
#Generated by NetworkManager
  nameserver <IP_ADDRESS>
  nameserver <IP_ADDRESS>
hassos can resolve:
ping www.google.com
www.google.com is alive!
the supervisor is also working:
docker exec hassos_supervisor ping -c1 www.google.com
PING www.google.com (<IP_ADDRESS>) 56 data bytes
64 bytes from <IP_ADDRESS> seq=0 ttl=54 time=60.334 ms
``
--- www.google.com ping statistics ---
1 packets transmitted, 1 packets received, 0% packet loss
round-trip min/avg/max =60.334/60.334/60.334 ms
But, doing the same test on homeassistant:
docker exec homeassistant ping -c1 www.google.com
ping: bad address 'www.google.com'
Another test:
docker exec homeassistant nslookup www.google.com
nslookup: can't resolve '(null)' : Name does not resolve
nslookup: can't resolve 'www.google.com' : Try again
I can't even use another server:
docker exec homeassistant nslookup www.google.com <IP_ADDRESS>
Server:       <IP_ADDRESS>
Address 1: <IP_ADDRESS>
nslookup: can't resolve 'www.google.com' : Try again
Still, there is internet access:
docker exec homeassistant  ping -c1 <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56 data bytes
64 bytes from <IP_ADDRESS> seq=0 ttl=55 time=62.134 ms
``
--- www.google.com ping statistics ---
1 packets transmitted, 1 packets received, 0% packet loss
round-trip min/avg/max =62.134/62.134/62.134 ms
Additional information:
To reproduce the issue:
Download the latest vmdk image: hassos_ova-2.12.vmdk

Create a VM based using the image
Start the VM
Going to the home assistant page, create the user, etc. The Weather is not updating, there are errors trying to connect.

Update: Today I rebooted, for the 100th time.  And it's been auto-fixed.  No clue what was wrong.  Nothing really changed, except the auto-update of the supervisor on boot.


Add Slf4j version to pom to prevent incompatibilities (refs #35)

This PR specifies the version of Slf4j directly in the POM, so there should not be logging incompatibility issues anymore.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.isadykov seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


The Linux clang binary in wasi-sdk-20+threads release is fantastically cursed

If you run it, it segfaults:
$ ./wasi-sdk-19.0/bin/clang++ --version
clang version 15.0.7
Target: wasm32-unknown-wasi
Thread model: posix
InstalledDir: /home/whitequark/Projects/yowasp/nextpnr/./wasi-sdk-19.0/bin
$ ./wasi-sdk-20.0+threads/bin/clang++ --version
Segmentation fault

If you run it under gdb, gdb breaks:
$ gdb --quiet --args ./wasi-sdk-20.0+threads/bin/clang-16 --version
"0x7ffcfc1ce650s": not in executable format: file format not recognized
(gdb)

If you look at it, it looks really weird:
$ file -L ./wasi-sdk-19.0/bin/clang++
./wasi-sdk-19.0/bin/clang++: ELF 64-bit LSB pie executable, x86-64, version 1 (GNU/Linux), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 3.2.0, BuildID[sha1]=3b2817eb45d389450bc034cf206408503ae5a6df, stripped
$ file -L ./wasi-sdk-20.0+threads/bin/clang++
./wasi-sdk-20.0+threads/bin/clang++: ELF 64-bit LSB shared object, x86-64, version 1 (GNU/Linux), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, missing section headers

No executable code at all!

For comparison this is how a normal binary looks like

(Does the CI run the built artefacts at all before releasing them?)

Okay, I redownloaded the archive and now it's fine. I think the binary got truncated somehow and it looks like Linux tooling really doesn't handle that case well?

Phew, glad you figured out the issue!


I don't know how to save checkpoint every n steps anymore in new colab

which notebook? fyi we never have any parameter named save_every_n_steps

Perhaps i misunderstood this value of 1 here?
It only saves the CKPT after completing all steps, regardless of the total Epoch count.

maybe its this?

It would be useful to be able to save intermediate checkpoints on larger training sessions 👍
I usually Train LORA's with your notebooks, but I like using your Trainer for CKPT training lately.
Thanks for your hard work on it ! 💯


Statistics

There should be statistics for workernames (like there is for versions)

Sure that's possible, there are many things which could be added, only question is if they're useful or not.

Sure that's possible, there are many things which could be added, only question is if they're useful or not.

Workernames are useful. but of course now when i think of it {COMPUTERNAME} would be a problem with that


Current 2.0.0 Beta Release can't start with db-sync v3.1.1 working

After the start graphql just does 10 attempts of Applying PostgreSQL schema and Hasura metadata: Attempt 7 of 10, retying.. and exits. I suspect this is because db-sync is writing to PG at the moment.
We need either to add this part to the db-sync, or mention in requirements that graphql needs exclusive access for first start.

Have no idea why it's hapenning, I left only graphql app running and see the same attempts loop. We need some error trace in logs

It's not a requirement, the system is intended to be resilient to start order. You're actually on an incompatible version pair there, so the result is expected.
Good news is 2.0.0 is out! :tada:  Please stick to the strict pairings as per the release notes https://github.com/input-output-hk/cardano-graphql/releases/tag/2.0.0

TY! I still believe we need to improve logging to see the reason for the failed attempt

@rhyslbw I've just started a completely separate setup with coin-node inputoutput/cardano-node:1.18.0, db-sync inputoutput/cardano-db-sync:3.1.0, graphql inputoutput/cardano-graphql:2.0.0, hasura inputoutput/cardano-graphql-hasura:2.0.0
And I still see the same loop Applying PostgreSQL schema and Hasura metadata: Attempt 8 of 10, retying...
Please advice

Try the dev setup:

fresh clone of the repo
yarn mainnet:up

It started working with full root permissions (had used a user with restricted access), TY

The startup logging and error handling needs work. Glad you got up and running


Color fidelity feature seems not to be available in web version

Scheme colors fidelity is a major feature but it is not to available in the web version of MTB.
The "match color" option seems to be missing whereas it is essential when dealing with a corporate palette.

There are two web tools called 'Material Theme Builder'. If you want the same features as the Figma plugin, including the 'color match' option, just use the one at Material Theme Builder.
For more info, check out the official source: Material Color Utilities.

This is correct, for newer features, please use the Github version as we work on updating the Material.io version. Thank you!


Feature request: Manually control the compilation import-path

In a company-internal project I have the following setup:

The FooBar project shall publish some .proto files
Those protos depend on other protos coming from Maven dependencies. There are three dependencies in total:

com.google.api.grpc:proto-google-common-protos (a jar with both .class and .proto files)
my.company:stuff (a zip with .proto files)
my.company:things (a zip with .proto files)

There are transitive dependencies between them: stuff depends both on things and proto-google-common-protos

I want to do two things in the FooBar project:

I want to invoke protoc with the FooBar .proto files as an automated sanity check. That already works with the plugin.
I want to compile the generated .java files in order to run an additional integration test with the generated classes.

For 2. to work, I need to compile the .java files resulting from the FooBar .proto files. Those classes however import classes that would result from stuff.proto and things.proto because of the dependencies within the .proto files. Since the stuff and things dependencies only provide the .protos, not precompiled .class files like the google dependency does, I would have to compile things.proto and stuff.proto as well. I tried to configure an additional execution of the plugin to do that.
However, the plugin always adds all dependencies to as includes so that things.proto appears both as a source-to-compile and as an import for the compilation which protoc quite sensibly does not accept. I know that I can already add more imports, but as far I as can see there is no way to exclude imports.
I propose a toggleable mode which lets me explicitly control the import path for compilation. I understand that my case is more complicated than one would like to deal with once transitive dependencies enter the picture, so I would still be happy if that mode did not automatically resolve transitive dependency like the normal operation of the plugin does.

So effectively a way of preventing automatic dependency resolution? If so, I can definitely add that.

Just for my understanding, so you have FooBar and that gets compiled, but you are trying to also compile additional resources outside the current project as well which is the issue you are seeing?
Might not have time to look into this immediately as I have some stuff coming up for a couple of weeks where I won't have access to a computer but I can try and think up some ideas around this if that sounds okay?

I can already compile additional .proto files by having a second execution with a specific source directory (which I can fill with .proto files by using the maven-dependency-plugin before). The problem is that the protoc-invocation will fail when the same .proto file is both a source-to-compile and an imported file.
Please, don't feel rushed. You get to it when you get to it. And if you don't, then my team will have to work out a different solution themselves.

Are you able to provide a dummy example of how you'd structure this project? Just gives me something to make an integration test for so I make sure I don't miss anything important!

Ok so summarising, it sounds like the functionality for this suite of use cases needs:

The ability to control whether Maven dependencies are pulled in recursively, only directly, or not at all (thinking of adding an enum for this purpose). This would let you take complete control on the import path itself.
The ability to add Maven artifacts as dependencies that have to be compiled with the build (i.e. including a zip archive from a Maven repository that is also compiled to Java sources with your project). This would then be usable with the first part to allow you to compile the mycompany.stuff ZIP without needing to unpack it elsewhere first, and have explicit control over what you are including.

I think this would solve your use case, does this sound like what you are looking for, or is there anything else I am missing here?

Update on the progress for this.

GH-139 implements a mechanism to control whether dependencies are globally resolved recursively or only directly.
GH-153 implements the ability to add custom archive artifacts located on Maven repositories as compilable sources to the project. This also adds the ability to control recursive resolution of individual dependencies.

Update on the progress for this.

GH-139 implements a mechanism to control whether dependencies are globally resolved recursively or only directly, providing the ability to include or exclude those dependencies as you need.
GH-153 implements the ability to add custom archive artifacts located on Maven repositories as compilable sources to the project. This also adds the ability to control recursive resolution of individual dependencies, which I believe is what you're trying to work around with the example you gave in the description.
GH-154 adds the ability to ignore project dependencies in the import path entirely.
GH-155 adds the ability to add explicit import paths using Maven dependencies, providing some additional fine grained control over importing dependencies and controlling whether they are handled recursively or not.

All of these are currently on main. If you have time, please can you take a look at them and see if they cover what you are looking for? If this is what you need then I will proceed to set up the full test packs for those features (holding off doing that until I have confirmation that this does what you need).
Thanks for the patience. I'll look into releasing this as a new version once it is confirmed.

I've released this under 1.2.0-rc1 provisionally.

Going to assume that this is okay to close as completed. If there are any issues with 1.2.0, please can you raise a new issue and let me know.

But on the bright side: A slightly different configuration works for my use-case :-)

This causes #164 (fixed in #171), and a reproduction for the original issue as part of #172 which now tracks that defect.


Native Ads Video hidden onScroll Listview or Gridview

When i add it to listview and scroll down to outside of AdView it will hidden, then i scroll up to view it again it not show

Are you using NativeAdmobController?


Enhance Contact Form with Hover Effects and Responsive Design Adjustments

Pull Request Title:
Enhance Contact Form with Hover Effects and Responsive Design Adjustments
Description:
This PR includes updates to improve the user experience on the contact form through hover effects, size transitions, and responsive design changes.
Key Changes:

Hover Effect with Size Transition:

Form Input Fields: Introduced a hover effect on text input fields and the textarea. When hovered, fields scale slightly (scale(1.05)), providing users a visual focus cue.
Glow Effect: Added a soft green glow to input fields on hover using box-shadow: 0 0 5px 1px rgba(0, 255, 0, 0.5).
Submit Button Hover: The submit button now has a hover effect, growing in size (scale(1.05)) with a similar green glow for better interaction feedback.

Reduced Glow Intensity:

Lowered the intensity of the glow effect by adjusting the box-shadow spread to 5px and decreasing opacity to ensure a more subtle effect without being distracting.

Responsive Design Improvements:

Made hover effects smooth across devices by retaining responsive behavior for all form fields.
The form's layout stacks vertically on smaller screens for better usability.

Form Input Enhancements:

The border color of input fields changes on hover to a vibrant green (border-color: #00ff00).
Kept a smooth transition with transition: all 0.3s ease-in-out on both inputs and the submit button.

Retained Original CSS Structure:

Preserved existing styling, ensuring new effects integrate seamlessly with the previous layout.

fixes: #445

Hey @neeru24  your changes are not visible and your deployment preview has not been generated by netlify please pass all the checks by netlify.

Hey @neeru24  your changes are not visible and your deployment preview has not been generated by netlify please pass all the checks by netlify.

Okay

Hey @neeru24 your changes are not visible and your deployment preview has not been generated by netlify please pass all the checks by netlify.

@gauravsingh1281 I tried to do so, but I  do not know why preview is not getting generated and the file has no merge conflicts. If you could help me, then kindly let me know or please merge it as the file have no conflicts.

@neeru24 ?

I was trying to figure it out, but it was not getting resolved so I closed it.


Getting error in karate api - Cannot invoke "io.netty.handler.codec.http.cookie.Cookie.name()" because "c" is null,

we are using karate api framework to test out API but we are facing the below issue intermittently in our automation
ERROR: 14:02:56.430 Cannot invoke "io.netty.handler.codec.http.cookie.Cookie.name()" because "c" is null, http call failed after 1019 milliseconds for url: https://xyz/abc
14:02:56.431 classpath:
Tried in 1.4 & 1.3 version of karate-junit5 dependency getting same error
If we get any help it will be helpful for us to run our tests seamlessly.

@rgud-softwareag I know this may be hard, but can you replicate this problem following this process: https://github.com/karatelabs/karate/wiki/How-to-Submit-an-Issue
the other option I suggest is you could make a change to the code, test if it works in your environment and then contribute a PR. for a similar fix, most likely the same problem, refer: https://github.com/karatelabs/karate/pull/2057

Hi @ptrthomas
I cloned the karate-template code ,added our api call and updated pom.xml as per our project.
I faced the issue againt
How i can share it to you as it contains authorization details of our application?
Thanks,
Rithesh

@rgud-softwareag

How i can share it with you as it contains authorization details of our application?

if you read the instructions I linked to, you see that you can use mocks. yes I know it is hard, but as an open-source project that's the best we can do. if this is urgent and you need someone from Karate Labs to sign an NDA and investigate, you can use our support form and provide your business email id: https://www.karatelabs.io/support

closing as we cannot replicate. if your server is sending a cookie without a name it may be a serious issue with your server


Avoid Kedro fsspec requirements being mutually incompatible with pandas 1.1.0

Description
Kedro 16.4 enforces fsspec<0.7.0,>=0.5.1 & PANDAS = "pandas>=0.24, <1.0.4" in setup.py
However,

the pd upper version limit is not enforced, so newer pd versions can coexist after pip install kedro
The blocking issue (for kedro including higher pd versions) on parquet reads in pandas is resolved re: https://github.com/pandas-dev/pandas/issues/34467 in pd >1.0.5, so the key blocker on Kedro upgrading pandas versions no longer exists.

IF kedro allows pandas 1.1.0, then you are going to hit an incompatibility with fsspec, as pandas 1.1.0 requires fsspec>=0.7.4.
Context
You cannot run kedro and pandas>1.1.0 on the same environment.  Pandas needs fsspec>0.7.4.
There are meaningful improvements in newer pandas, so I would like to be able to run them together out-of-the-box.
The current reason to not allow higher pd versions as per setup.py in the kedro source code (https://github.com/pandas-dev/pandas/issues/34467) is no longer applicable, so i think it is time to make this change unless fsspec has deal-breaking problems in later versions.
Possible Implementation
Can you just update fsspec version requirements to fsspec<0.7.4,>=0.5.1?  I'm not aware of any major problems in the newer versions
Possible Alternatives
Raise warnings or errors if kedro co-exists with pd>=1.1.0

Happy to make pull request with my suggested implementation if desired.  Although it is only a 1-liner edit.

nb./ This issue is a superset of issue #488.  Modin compatibility will be fully resolved with an update to fsspec and pandas 1.1.0

@JMBurley would it also be possible to update to fsspec 0.8.0 if you create a PR or do you see any problems with the latest version?

@fjp I could update fsspec in kedro setup files.
However, kedro has a lot of I/O functionality that could intersect with fsspec in a lot of ways.  Unless quantumBlack is very confident in their hooks and tests I don't think I should be making that change (particularly as I don't understand fsspec very deeply).
On the other hand, I am a (minor) Pandas contributor and understand that library well enough to know that 1.1.0 is a good upgrade and will not break data pipelines if the fsspec dependency is correctly managed (gcsfs and similar have been stable for long enough that it shouldn't be an issue for kedro)

Hi guys, thanks a lot for the interest here. Unfortunately fsspec 0.7.* introduced a breaking change for us (namely on versioning), which was addressed in develop. We've unpinned pandas for now, but broader fsspec boundaries will be made available in the next major release (0.17.0).

Thanks @lorenabalan,  if I'm reading  develop: 3218055 correctly then it would make sense to allow broader pandas  boundaries as part of the same feature release in 0.17.0?
Would need to change the appropriate lines in requirements.txt and setup.py.  The reason quantum Black unpinned pandas (pandas-dev/pandas#34467) is solved, and the fsspec problem that would have been introduced by a pd update is now fixed.

Oops. FYI, I'm running into an incompatibility with s3fs 0.5.0 as it requires fsspec>=0.8.0.

Oops. FYI, I'm running into an incompatibility with s3fs 0.5.0 as it requires fsspec>=0.8.0.

Im having this same issue. Its keeping me from creating my environment. A lower version of s3fs is not really a good option for me.

Hey @JMBurley , I just noticed with Kedro 0.16.5, I get new pandas==1.1.2 and fsspec==0.6.3 after kedro install. Are you able to validate on your end that it's indeed the case and the original parquet problem is fixed? Feel free to close the issue once that's confirmed.

Hi @lorenabalan ,
If I'm reading your comment correctly kedro 0.16.5 has created the exact problem I was forewarning against, pandas>=1.1.0 necessitates fsspec**>=0.7.4**.
If kedro enforces pandas==1.1.2 and fsspec**==0.6.3**  then all of the s3 operations in pandas will fail (eg. pd.read_csv("s3://my-bucket/some-file-key.csv").
QuantumBlack can keep track of Pandas dependencies here: https://pandas.pydata.org/pandas-docs/stable/getting_started/install.html
PS. To clear up any confusion,  I only mentioned parquet as it was the blocker on kedro allowing higher pd versions.  It is not the key part of this issue, which is flagging up kedro compatibility issues if you upgrade pandas versions.

@JMBurley pandas==1.1.2 has fsspec in optional arguments, meaning that dependency resolution tools have no idea that it depends on fsspec. You can confirm that by installing pandas==1.1.2 and running pip show pandas:
Name: pandas
Version: 1.1.2
Summary: Powerful data structures for data analysis, time series, and statistics
Home-page: https://pandas.pydata.org
Author: None
Author-email: None
License: BSD
Location: .../python3.7/site-packages
Requires: pytz, python-dateutil, numpy  # <<< no fsspec here
Required-by:

Kedro has not created a problem, Kedro declares its dependencies and the corresponding acceptable version ranges using the standard mechanism - install_requires in setup.py. If pandas is not specifying the dependencies the way that is recognised by the dependency resolution tools, then Kedro has no control over it.
pandas==1.1.2 works within Kedro 0.16.5 without issues since Kedro pulls the data from the filesystem using fsspec directly and then pass the file-like object to pandas.read_* methods. If you, however, intend to read/write to non-local locations using pandas directly from within thee environment where Kedro is installed, that will not work. This is the reason we strongly recommend having isolated environment for each project. The solution would be to cap the upper bound version of pandas in src/requirements.txt (or src/requirements.in if already compiled) of your Kedro project.
Lastly, correct dependency version resolution is not the responsibility of Kedro library, but rather package management tools like pip, pip-compile, poetry, etc. Kedro uses pip-compile under the hood when installing project dependencies, however pandas doesn't expose enough information about its dependencies to pip-compile to resolve the versions correctly.

Hello again!
So what happened was we relaxed the pandas requirements, to be just >0.24.0. fsspec is an optional requirement for pandas, so it's not an impediment in fetching the latest pandas. fsspec is pulled according to Kedro core requirements,  <0.7.0.
The way Kedro interacts with fsspec means pandas doesn't have to, so there are actually no compatibility issues on that front. We read the data using fsspec directly and then pass a file-like object to the pandas.read_csv. The example you gave, with passing a path/string indeed won't work, but fortunately that is not how Kedro datasets work so it should be safe.
We hear you all on the fsspec bump, and as I said this has been implemented on develop and will be made available on 0.17.0. Hopefully this release will come sooner rather than later, thank to you all raising awareness/visibility on your issues! :)

@DmitriiDeriabinQB has beat me to it. 😂 ⏲

Thanks both,
Would I be correct in understanding the official position here as " all file loads should be handled by .yml in your catalog folder; it is anti-pattern to directly read files with pandas, therefore we don't view it as critical for kedro to facilitate pandas s3 functionality, nor maintain it if it is incidentally present in some kedro versions"?
I understand the position if so, although I think there are good reasons to prototype eda in a project using pandas s3 functionality rather than hopping in and out of the data catalog to create potentially disposable items.
Regardless, looking forward to v0.17.0!
PS. I agree that anyone who doesn't use virtual environments deserves the problems they encounter

fsspec maximum required version was updated in 151cbcd commit.
fsspec>=0.5.1, <0.9

The changes will be available in the upcoming Kedro 0.17.0 release.


syntax error: unexpected token

Can't use the compressor at all because of this error
after attempting to convert glb to gltf:
SyntaxError: Unexpected token {
 at createScript (vm.js:74:10)
 at Object.runInThisContext (vm.js:116:10)
 at Module._compile (module.js:537:28)
at Object.Module._extensions..js (module.js:584:10)
 at Module.load (module.js:507:32)
at tryModuleLoad (module.js:470:12)
at Function.Module._load (module.js:462:3)
at Module.require (module.js:517:17)
at require (internal/module.js:11:18)
at Object.<anonymous> (/usr/local/lib/node_modules/gltf-pipeline/node_modules/fs-extra/lib/mkdirs/index.js:3:44)
after attempting to compress glb:
$ npx gltf-pipeline -i file.glb -d --draco.compressionLevel 10 -o compressed.glb
Unexpected token {

@zaaakk I'm not seeing this error. What version of Node are you using?

Hello, has this problem been solved? I also have the same problem. My node version is 8.11.4

@zaaakk @lilleyse

@liuyu-struggle could you try the latest Node 12 release? https://nodejs.org/dist/latest-v12.x/


Added machine annotation service

Added Machine Annotation Service.
Some minor changes to the rest:

formatting
fixes in controller where we did not follow JSON:API
minor optimization to tests

https://naturalis.atlassian.net/browse/DD-539

Sonar fails because of condition coverage. This is due to changes in the HandleService which doesn't have tests and will be replaced soon. So I want to leave that be and don't fix the Sonar coverage


Local images not deleted after upload

Maybe my logic behind this is wrong but imager is not working like "I would like to".  So all my assets are setup with an S3 bucket, now I wanted to optimize all uploaded images and also upload to S3.
I saw there were some issues with this but for me it worked without a problem.
Here is my config:
return [
//    'optimizeType' => 'runtime', //  If set to 'runtime', all optimizations will be run immediately.
    'useCwebp' => true,
    'resizeFilter' => 'hermite',
    'storageConfig' => [
        'aws'  => [
            'accessKey' => getenv('S3_API_KEY'),
            'secretAccessKey' => getenv('S3_SECRET'),
            'region' => getenv('S3_REGION'),
            'bucket' => getenv('S3_BUCKET'),
            'folder' => '',
            'requestHeaders' => array(),
            'storageType' => 'standard',
            'cloudfrontInvalidateEnabled' => false,
            'cloudfrontDistributionId' => '',
        ],
    ],
    'storages' => ['aws'],
    'imagerUrl' => 'https://s3.amazonaws.com/' . getenv('S3_BUCKET') . '/',
    'cacheRemoteFiles' => false,
    'cacheEnabled' => false,
    'cacheDurationRemoteFiles' => 0,
    'cacheDuration' => 0,
    'cacheDurationExternalStorage' => 0,
    'optimizerConfig' => [
        'jpegoptim' => [
            'extensions' => ['jpg'],
            'path' => '/usr/bin/jpegoptim',
            'optionString' => '-s',
        ],
        'optipng' => [
            'extensions' => ['png'],
            'path' => '/usr/bin/optipng',
            'optionString' => '-o2',
        ],
        'gifsicle' => [
            'extensions' => ['gif'],
            'path' => '/usr/bin/gifsicle',
            'optionString' => '--optimize=3 --colors 256',
        ],
    ],
    'optimizers' => ['jpegoptim', 'gifsicle', 'optipng'],

];

What is the problem? The problem with this is that Imager is storing the images locally. I know it needs to download the images (because they are remote) before transforming them, but why doesn't it delete after the transformation. For this reason I tried to disable "cache":
    'cacheRemoteFiles' => false,
    'cacheEnabled' => false,
    'cacheDurationRemoteFiles' => 0,
    'cacheDuration' => 0,
    'cacheDurationExternalStorage' => 0,

How I imagined it working is:

Check if image transformation exists (remote or local)
If not get normal image (download if remote)
Transform image
Upload (remote) or Save (if local)
If remote clean the local copy.

I really didn't have time to dig to deep in the code but from what I see here:
https://github.com/aelvan/Imager-Craft/blob/b8f9af372dc9fb60d778cdc1a00a6b7026bf0986/src/transformers/CraftTransformer.php#L196-L198
Imager needs a local copy? Correct me if I am wrong.
Why am I asking this?
The whole (one of) idea of us using S3 was to offload the storage to AWS and not fill up the server. If imager works like this (or I didn't configure it right), we will fill up the local server and S3 will work ONLY as CDN.
Why is this a problem?
Because we deploy on Heroku and upload changes are not persistent and will be cleaned out, therefore making this useless.

Yes, this is the intended behavior, even if you choose to upload the transformed file to one or more external storages, the transform in the imagerSystemPath is still the "master" and is used for caching. The setting cacheRemoteFiles refers to the copy that Imager has to download from an external source to do the transform, not the transform itself.
One of the main motivations behind making Imager was to have something that was completely file-based, and not rely on looking up a transform in the database to know if it exists or not, which the native Craft transforms do. But if your main motivation is saving space, you should instead just use native transforms.


Unable to search single backslash

Laravel-mongodb Version: #.#.#
PHP Version: #.#.#
Database Driver & Version:

Description:
While using Contains operator to search entries containing single backslash \, it will return contents containing %. It looks like the issue caused by $regex = preg_replace('#(^|[^\\\])%#', '$1.*', preg_quote($value)); in the function compileWhereBasic.
Steps to reproduce

Pass the value as "%\%"
After being replaced by the regular expression, the $regex = ".*\\%"
It will return entries containing "%"

Expected behaviour
Tell us what should happen
Ideally, it should replace the query with ".\\." to perform the search that returns entries containing "\".
Actual behaviour
Tell us what happens instead
Actually, it replaces the query with ".*\\%" which will search all entries having "%"
Logs:
Insert log.txt here (if necessary)

Fixed by https://github.com/mongodb/laravel-mongodb/commit/ea89e8631350cd81c8d5bf977efb4c09e60d7807


slight fixes for rnaseq prepro

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/rnaseq branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Tidy, thanks Jon! I'm embarrassed to say I didn't notice the duplicate fail_trimmed_samples_mqc.tsv files when looking through it this morning.
Not as embarrassed as I am to have duplicated it ;-)


Massive spam issue even though domains are blacklisted and only admin or API can shorten

Technical details regarding my environment

YOURLS version: 1.7.2

Plugins enabled: 404 Redirect, Anti spam, Google Analytics, Force Lowercase, Change error messages, Mass remove links, Random backgrounds, Random keywords, SSL for SSL, Yourls Abusedesk, reCaptcha.

PHP version: 7.1
Running on a Ubuntu 17.10 with quite a bunch of subdomains - a lot running PHP, but none of them having been hacked before.

Bug description
My instance of YOURLs gets a lot of spam domains or malwareridden domains by many different IP's and countries even though the domains they shorten are banned.
Example:
It's not even possible to shorten big sites such as YouTube, yet still a link or two appears daily in my administrative panel with multiple clicks already on it.
HOW does this happen and how do I block it if even administrator-login, captcha and regeneration of the API key doesn't block it?..

Hi @ozh ,
this is to share probably the same concern as @olenoerby 's post.
This is my POC for yourls.org -> http://tlkm.my/
i closed the form, however there's still unidentified input in url created.

if we see the ip, it's an invalid ip format
1.1 4709a868646bacc9738100003797f08a d
so, there's somewhere that spammer can get to create the link without form.
Thank you,
Fahmi Paridin

@afahmiparidin : most likely this isn't added to the YOURLS DB using standard input, since in that case the IP would be determined by the system, yet here it appears the value is being submitted. I suggest you change your secret key (see https://github.com/YOURLS/YOURLS/wiki/PasswordlessAPI) and your server passwords (MySQL, FTP)

Dear @ozh, thanks for the reply.
For sure .. will do as per recommended. however, if i find out anything wrong, i'll let you know.

@afahmiparidin - Hi Fahmi!
I agree with ozh, it looks like someone got into your site,  It is hard (pretty much impossible) to fake the IP from outside of a system because it is used to return the reply.  So... it looks like your system was comprised from inside, rather than outside.
Beside the secret keys ozh told you about, and things like usernames, passwords, and YOURLS_COOKIEKEY (in config.php) there are more passwords you really need to change/strengthen. You may want to take a look at the tips on this page http://a.fil.net/s/admin/setup-config.php?step=0
Beyond MySQL and FTP passwords, you may have a password to admin your server from a webpage.
Likewise you may have a password for something like PHPMyAdmin with a ridiculously simple password (or no password from within the website). Your hosting company may have set this up without you even knowing it.  With access to a PHPMyAdmin account, for your server, I could change everything in your YOURLS database!  And never even get close to your website form!
Finally, when a cracker gains access, he might not want to lose it!  Most crackers will add hidden code to files you expect to be there.  The easy way to fix this is to copy all the core files onto your server again.  The standard files will not have the extra code and will therefore wipeout any added codes.  Just be aware of your custom files.  For example, config.php. Any file you made any change to manually, you will want to save before you upload the new files.
I hope this helps you Fahmi

@PopVeKind yeah, well that helps.

Finally, when a cracker gains access, he might not want to lose it! Most crackers will add hidden code to files you expect to be there. The easy way to fix this is to copy all the core files onto your server again. The standard files will not have the extra code and will therefore wipeout any added codes. Just be aware of your custom files. For example, config.php. Any file you made any change to manually, you will want to save before you upload the new files.

forgotten already about this tip. Thanks for reminding me.

Since this sounds like the result of a compromised server, I'm going to close it. We can reopen if it turns out that the entry point was YOURLS after all.


featureadd extra min-max runtime filter if target is the first column.

Proposed changes
suppose we have schema: L(a, b, c), R(d, e)
for join: L join R on L.a=R.d where R.e < 100, nereids generate RF(R.d->L.a)
since L.a is the first column, data in segments are ordered by L.a. If we create a min-max RF, scanner could use this min-max RF to prune scan-range.
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

opt in another way later


Go generate fixes

@vishh hooks/check_boilerplate.sh complains about the generated file not having the copyright. It's a generated file, so I'm thinking we should make the script ignore that file. Thoughts?

As said above, made hooks/check_boilerplate.sh ignore go generated files, which is only *extpoints/extpoints.go for now.

Excluding generated files SGTM. Why not delete and regenerate extpoints? With this PR will extpoints.go be regerated whenever that package is modified?

See my commit messages - go generate is not supposed to be run by the user, but by the developer(s).
When what package is modified, go-extpoints or heapster? At the moment, neither. It is up to whomever changes the go-extpoints input to re-run the tool. And if the tool ever changes, which it shouldn't do often, someone would eventually run the updated tool and notice.
I thought about making travis run go generate and check if it is up to date. I could bundle that with a go-extpoints in Godep/ so that it never breaks in the future. What do you think?
Side note: master currently has this problem. If go-extpoints breaks, heapster breaks, because we just go get it via master.

I meant chnages to heapster extpoints/interface.go which should result in
regenerating extpoints/ expoints.go. Vendoring go-extpoints makes sense.
On Mon, Aug 10, 2015 at 10:25 AM, Daniel Martí<EMAIL_ADDRESS>wrote:

See my commit messages - go generate is not supposed to be run by the
user, but by the developer(s).
When what package is modified, go-extpoints or heapster? At the moment,
neither. It is up to whomever changes the go-extpoints input to re-run
the tool. And if the tool ever changes, which it shouldn't do often,
someone would eventually run the updated tool and notice.
I thought about making travis run go generate and check if it is up to
date. I could bundle that with a go-extpoints in Godep/ so that it never
breaks in the future. What do you think?
Side note: master currently has this problem. If go-extpoints breaks,
heapster breaks, because we just go get it via master.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/heapster/pull/459#issuecomment-129537406
.

I'll vendor go-extpoints and add the thing to travis.

retest this please

retest this please

@vishh any idea why jenkins failed twice with the same error message?

Otherwise this is ready to be merged

retest this please

The new jenkins build failed, but an earlier one succeeded. Jenkins is not being very deterministic.

retest this please

green again!


Uncrafting Grinder new help button crashing game

I have replicated this error several times with different pieces of armor. This happens mostly with Bronze Armor as the progress bar goes all the way non-stop. The only way to stop is by removing the armor from the uncrafter. If I hit the ? button then the game crashes. Crash log attached.
Something else interesting is that when I remove the bronze armor while the progress bar is filling (after several cycles), the bar does not reset but stays wherever it was when i removed the item,
Crash Log

had another report of what looks like the same issue http://pastebin.com/v5eHrFQ6

Fixed in 1.9.4.  Keep the reports coming :) https://minecraft.curseforge.com/projects/cyclic/files/2358564


Draw red square around the leader character

Right now the party leader is drawn in inverse video (black on white),
but this does not work well for non-hero actors, because their colour
is not visible. Drawing them in color on white does not work either,
because for bright colours their letter is hard to discern.
On text frontends we may use cursors to highlight the leader,
but the insertion cursor in gtk has a fixed, bad shape (and the recent
patch for overtype mode was reverted and generally it's messy),
so in gtk we should probably draw a red square by hand over the textView
whenever it's redrawn.

This is implemented on master in both principal frontends: sdl and browser.


[4.0] Clean up diagnostics for @NSKeyedArchiveLegacy attributes

Explanation: Improve the Fix-It for the @NSKeyedArchiveLegacy attribute to provide the class name one should use for archive compatibility, and ban the implementation-detail @_staticInitializeObjCMetadata attribute.
Scope: UI improvement to a newly-added attribute.
Radar: rdar://problem/32118614
Risk: Extremely low; this tunes the diagnostic text of a new warning and bans the use of a new, hidden attribute.
Testing:  Compiler regression testing.

@swift-ci please test


Remove deprecated code for v0.16.x

Not much to remove which is I think a good sign that huggingface_hub is getting more mature :)

force to use kwargs in ModelHubMixin (instead of passing everything a positional arg)
not possible anymore to pass model_id as username/repo_name@revision in ModelHubMixin

(+2 type annotations fixes)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Not defined Models

Hello.
I have a problem with model definitions. I use sequelize.
What should I do to make the data for the model (#/definitions/{{modelName}}) pull up from the model itself? Because now it is empty.

Model example:
module.exports = function (sequelize, DataTypes) {
  var activities = sequelize.define("activities", {
    removed: {
      type: DataTypes.BOOLEAN,
      defaultValue: false
    },
    name: {
      type: DataTypes.STRING,
      allowNull: false,
      defaultValue: false
    },
    createdAt: {
      type: DataTypes.DATE
    },
    updatedAt: {
      type: DataTypes.DATE
    },
    agencyId: {
      type: DataTypes.INTEGER,
      references: {
        model: "companies",
        key: "id"
      }
    }
  });
  return activities;
};

I would be grateful for any help.

Hmmm this lib does not support sequelize, just leveraging on sails integration with waterline


Update to React 16: Remove obsolete React.createClass and use prop-types

Closing this as it's been fixed in another PR and merged


Linux fixes for dispatch-806 merge

Collection of small fixes to dispatch-806 merge to
allow code to compile/link/run on Linux.

everything makes perfect sense to me but for the commented hunk.

Indeed, seems I misremembered, this is fine :)


Added getters and setters for body offsets in PositionalLight

Allows for changing which way a light points, for example, while attached to a body in case the body needs to pointing another way. Current alternative is to extend the light class in question which is, last I checked, not good practice.

Note, it may be cleaner to just make the fields public rather than using getters and setters,


Remove region input from README.md

:gear: Issue
The #96 removed region variable. But the variable is still present in README.md.
:white_check_mark: Fix
This PR removes the region variable from README.md.

Thank you!


Jetty listening on <IP_ADDRESS>:3003

Hey folks,
it just occurred to me that ananas is running a Jetty 9.3 (jetty-9.3.z-SNAPSHOT) on Port 3003 listening on <IP_ADDRESS> on linux. I can not say if this version is potentially vulnerable to CVE 2016-4800 (https://www.cvedetails.com/cve/CVE-2016-4800/) but there are updates available. I would also suggest listening on localhost only if not explicitly defined by a user.
By the way, the "Get Started" link in your wiki (https://github.com/ananas-analytics/ananas-desktop/wiki) points to http://localhost:3000/docs/user-guide/getting-started.
Cheers
Florian

@shantycode Thanks for your report! Ananas Desktop uses a Sparkjava server as backend, you can run this server from CLI directly. It is intended to listen on localhost only when running with the UI. For the potentially vulnerable you mentioned, it is because we used an old version of Sparkjava, which depends on jetty 9.3.
I create a fix to

upgrade sparkjava server to the latest version 2.9.1 https://mvnrepository.com/artifact/com.sparkjava/spark-core/2.9.1, which depends on jetty server 9.4.18.v20190429
Explicitly listen to "localhost" only for ananas server

Fix commit here:  https://github.com/ananas-analytics/ananas-desktop/commit/a9e844dfe8a32c58da7b7d3627f995fbd9088cf9

fix in PR: https://github.com/ananas-analytics/ananas-desktop/pull/75


Resolve compilation issue on OSX.

Problem:
This PR is related to https://github.com/twitter/vireo/issues/8.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Use ember-cli-sass instead of pre-compiling scss

To work around an issue with including SASS in an Ember-CLI Add-On, the styles and building thereof, are now located under styles/ and must be built from the command line as so:

README.md#generating-styles
Basically, this isn't needed anymore.

Right. The style thing was a remnant of this being an old ember cli ~0.0.38 project.

Closing this completely as I'm moving away from SASS entirely. I find it encourages overly specific CSS output. See #96


"Attempt to decode JSON with unexpected mimetype" error.

Intermittently since 2022.8 upgrade of HASS, but more frequently past week or two, I cannot login to alarm.com via HASS and get the following error on Integrations page:
Retrying setup: 0, message='Attempt to decode JSON with unexpected mimetype: text/html; charset=utf-8', url=URL('https://www.alarm.com/web/api/video/camera
I have tried deleting and re-adding Integration but to no availl

I am having the same issue. Tried uninstalling and re-installing.

We recently moved the integration to a new GitHub repo not realizing that this would cut off existing users from upgrades.
The issue you're reporting is resolved, but you'll need to do some work to update:

Delete the integration from Home Assistant via the Integrations page.
Delete the integration from HACS.
Re-download the integration via HACS.
Restart Home Assistant.
Re-add the integration from the integrations page.

Sorry for the trouble. Please let me know how this works for you.

We recently moved the integration to a new GitHub repo not realizing that this would cut off existing users from upgrades.
The issue you're reporting is resolved, but you'll need to do some work to connect your HA server to the new repo:

Delete the integration from Home Assistant via the Integrations page.
Delete the integration from HACS.
Re-download the integration via HACS.
Restart Home Assistant.
Re-add the integration from the integrations page.

Sorry for the trouble. Please let me know how this works for you.

Thank you!  All good now!

This seems to have solved it.  THANK YOU!
From: ssambhi900 @.>
Sent: Wednesday, August 17, 2022 9:10 AM
To: pyalarmdotcom/alarmdotcom @.>
Cc: bfish2 @.>; Author @.>
Subject: Re: [pyalarmdotcom/alarmdotcom] "Attempt to decode JSON with unexpected mimetype" error. (Issue #168)
We recently moved the integration to a new GitHub repo not realizing that this would cut off existing users from upgrades.
The issue you're reporting is resolved, but you'll need to do some work to connect your HA server to the new repo:

Delete the integration from Home Assistant via the Integrations page.
Delete the integration from HACS.
Re-download the integration via HACS.
Restart Home Assistant.
Re-add the integration from the integrations page.

Sorry for the trouble. Please let me know how this works for you.
Thank you! All good now!
—
Reply to this email directly, view it on GitHub https://github.com/pyalarmdotcom/alarmdotcom/issues/168#issuecomment-1217990173 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ37CVXPMWOAIIJ45NXK3ZDVZTQC3ANCNFSM56ZGTIWA .
You are receiving this because you authored the thread.Message ID: @.***>


19041 Created constants (enum) for resource phrases

Issue #: bcgov/entity#19041
Description of changes:

app version = 5.8.9
created constants (enum) for common resource phrases
updated misc resource files to use constants

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

All, what do you think about this change -- worth doing or too abstract?


feat: alter dockerfile processing to use different requirements for dev

Reason for Change
https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/gh/chanzuckerberg/single-cell/678
Changes
introduces a new file, processing/requirements-dev.txt and has Dockerfile.processing install from this list of requirements if we're in dev, otherwise it'll install from processing/requirements.txt
Testing steps
going to look at the rdev build on this and see what it installs

In this rdev build, we pip install requirements.txt


ttx_diff: pass --keep-overlaps for static fontmake builds for now

Fixes #975 (until we also have gain the ability to remove overlaps)

I don't think we should mark the issue fixed?
On Wed, Oct 2, 2024, 5:55 PM Colin Rofls @.***> wrote:

@.**** approved this pull request.
makes sense!
—
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontc/pull/979#pullrequestreview-2344458152,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABRKXAFQ5OKHWDSOQSRCC4TZZSIXDAVCNFSM6AAAAABPHEB7USVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNBUGQ2TQMJVGI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Can't use defaultRepresentation

Hi! Thank you for making this wrapper, it's super useful!
I'm not sure about this, but I think there's an issue with how representations are handled when
{ defaultRepresentation: true }

is passed into loadFileParams for Component. The behavior is that the structure briefly flashes on screen (with the expected colorful defaultRepresentation) before disappearing, leaving just an empty Stage.
From briefly browsing around, I think what happens is that when the Component instance is created and this runs:
  useEffect(() => {
    if (component) {
      component.removeAllRepresentations();
      reprList.forEach((repr) =>
        component.addRepresentation(repr.type, repr.params)
      );
    }
  }, [component, reprList]);

whatever representation was created from the first flag passed into loadFileParams is cleared out and it causes an empty Stage. I don't know if that's how it works under the hood, but I suspect that that may be the case.
I haven't tested this, but I think that changing the above to:
  useEffect(() => {
    if (component && reprList.length > 0) {
      component.removeAllRepresentations();
      reprList.forEach((repr) =>
        component.addRepresentation(repr.type, repr.params)
      );
    }
  }, [component, reprList]);

would allow for the defaultRepresentation flag to work as long as a reprList prop weren't passed in to Component.
Let me know what you think!

Thank you for reporting an issue!
I took a look into the code and confirmed that the default representation set by the { defaultRepresentation: true } flag is overwritten by an empty reprList.
How about allowing reprList to be an uncontrolled prop?
That is, how about changing default prop for reprList of Component from [] to undefined and change the useEffect section to something like this?
useEffect(() => {
  if (component && reprList) {
    component.removeAllRepresentations();
    reprList.forEach((repr) =>
      component.addRepresentation(repr.type, repr.params)
    );
  }
}, [component, reprList]);

By setting reprList={undefined}, representation is left uncontrolled by react-ngl and thus { defaultRepresentation: true} flag of ngl will take effect.
By setting reprList={[]}, uses will still be able to clear representations.

yeah that looks good to me!


dependencies and fixes for testing

This PR updates dependencies and fixes tests that broke due to mocking. In particular, the FileHandler component itself was updated to determine what step it should be at on first render using the state it has. The testing was updated to reflect this state-based testing (as interaction testing at the unit level is not possible due to mocks).

Waiting on issues regarding tests

Fixes for tests being handled elsewhere.


chore: add modules to all apps

Description
add modules to all apps
Completes: AB#FILL_IN_YOUR_ISSUE_ID
Type of change
Remove options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code.
[ ] I have linked my DevOps task using the AB# tag.
[ ] My code is easy to read.
[ ] Documentation and comments is added where needed.
[ ] My code is covered by tests.
[ ] UX has reviewed relevant UI contributions.

Merging it for now, will probably refactor the modules and ...Options later on.


[Bug]: All integration tests for RPM, Debian and Windows are failing

Describe the bug
Note: This is an observation for OpenSearch only. OSD might be a different story
Recently so many changes went in related to security plugin as well as in general. We enabled integration testing to be triggered via distribution pipeline for all distributions. (Earlier it was just running for tarball)
To reproduce
All windows failed: https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/7136/pipeline
All debian failed: https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/7135/pipeline
All RPM failed: https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/7134/pipeline
Expected behavior
All tests should run. The infrastructure and test set up should be fixed.
Screenshots
If applicable, add screenshots to help explain your problem.
Host / Environment
No response
Additional context
No response
Relevant log output
No response

Will check soon, most likely the deb/rpm is due to sudo permission not added to the ci-runner images.

k-NN: Test with @junqiu-lei on tar for concurrent search, resolved https://github.com/opensearch-project/k-NN/pull/1419.

Resolve rpm:
2024-01-26 23:15:19 INFO     rpm installation requires sudo, script will exit if current user does not have sudo access
LANG=C.UTF-8
HOSTNAME=1c85fa7fa882
TERM=xterm
MAIL=/var/mail/root
PATH=/sbin:/bin:/usr/sbin:/usr/bin
LOGNAME=root
USER=root
HOME=/root
SHELL=/bin/bash
SUDO_COMMAND=/bin/env OPENSEARCH_INITIAL_ADMIN_PASSWORD=myStrongPassword123!
SUDO_USER=ci-runner
SUDO_UID=1000
SUDO_GID=1000
XDG_SESSION_ID=c1
XDG_RUNTIME_DIR=/run/user/0
DBUS_SESSION_BUS_ADDRESS=unix:path=/run/user/0/bus
OPENSEARCH_INITIAL_ADMIN_PASSWORD=myStrongPassword123!
No match for argument: opensearch
No packages marked for removal.
Dependencies resolved.
Nothing to do.
Complete!
Rocky Linux 8 - AppStream                                                                                7.1 MB/s |  10 MB     00:01
Rocky Linux 8 - BaseOS                                                                                4.6 MB/s | 5.1 MB     00:01
Rocky Linux 8 - Extras                                                                                29 kB/s |  14 kB     00:00
Extra Packages for Enterprise Linux 8 - x86_64                                                                                8.6 MB/s |  16 MB     00:01
packages for the GitHub CLI                                                                                15 kB/s | 2.6 kB     00:00
Dependencies resolved.
============================================================================================================================================================================================================================================================================================================================
 Package                                                                       Architecture                                                              Version                                                                      Repository                                                                       Size
============================================================================================================================================================================================================================================================================================================================
Installing:
 opensearch                                                                    x86_64                                                                    2.12.0-1                                                                     @commandline                                                                    660 M

Transaction Summary
============================================================================================================================================================================================================================================================================================================================
Install  1 Package

Total size: 660 M
Installed size: 931 M
Downloading Packages:
Running transaction check
Transaction check succeeded.
Running transaction test
Transaction test succeeded.

Removed:
  opensearch-2.12.0-1.x86_64

Complete!
2024-01-26 23:33:17 INFO     Recording component test results for geospatial at /home/ci-runner/opensearch-build/test-results/57995af2c3444c6d86f67ce3fbad526e/integ-test/geospatial/with-security
2024-01-26 23:33:17 INFO     Walking tree from /home/ci-runner/opensearch-build/test-results/57995af2c3444c6d86f67ce3fbad526e/integ-test/geospatial/with-security
2024-01-26 23:33:17 INFO     Recording component test results for geospatial at /home/ci-runner/opensearch-build/test-results/57995af2c3444c6d86f67ce3fbad526e/integ-test/geospatial/without-security
2024-01-26 23:33:17 INFO     Walking tree from /home/ci-runner/opensearch-build/test-results/57995af2c3444c6d86f67ce3fbad526e/integ-test/geospatial/without-security
2024-01-26 23:33:17 INFO     Skipping ml-commons
2024-01-26 23:33:17 INFO     Skipping neural-search
2024-01-26 23:33:17 INFO     Skipping notifications-core
2024-01-26 23:33:17 INFO     Skipping notifications
2024-01-26 23:33:17 INFO     Skipping opensearch-observability
2024-01-26 23:33:17 INFO     Skipping anomaly-detection
2024-01-26 23:33:17 INFO     Skipping alerting
2024-01-26 23:33:17 INFO     Skipping security-analytics
2024-01-26 23:33:17 INFO     Skipping index-management
2024-01-26 23:33:17 INFO     Skipping flow-framework
2024-01-26 23:33:17 INFO     Removing /tmp/tmpun19ylg0
2024-01-26 23:33:17 INFO     | geospatial           | with-security        | PASS  |
2024-01-26 23:33:17 INFO     | geospatial           | without-security     | PASS  |

Looks like debian is fixed: https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/7376/pipeline
RPM too: https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/7375/pipeline
Windows not yet: https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/7371/pipeline
Thanks @peterzhuamazon

More changes needed due to this:

https://github.com/opensearch-project/opensearch-dashboards-functional-test/pull/988

IntegTest should be fully fixed now on Windows.

Windows looks good too now: https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/7681/pipeline
Thanks @peterzhuamazon 🙌

See issues:
https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test-opensearch-dashboards/detail/integ-test-opensearch-dashboards/5530/pipeline/
x64, linux, osd-2.12.0-7254, os-2.12.0-9373, rpm
https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test-opensearch-dashboards/detail/integ-test-opensearch-dashboards/5527/pipeline/
arm64, linux, osd-2.12.0-7254, os-2.12.0-9373, tar
{"type":"log","@timestamp":"2024-02-09T18:57:36Z","tags":["fatal","root"],"pid":521,"message":"Error: Cannot find module '../../types'\nRequire stack:\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/plugins/securityAnalyticsDashboards/server/utils/constants.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/plugins/securityAnalyticsDashboards/server/clusters/createSecurityAnalyticsCluster.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/plugins/securityAnalyticsDashboards/server/plugin.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/plugins/securityAnalyticsDashboards/server/index.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/plugins/plugin.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/plugins/discovery/plugins_discovery.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/plugins/discovery/index.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/plugins/plugins_service.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/plugins/index.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/index.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/cli/serve/serve.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/cli/cli.js\n- /tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/cli/dist.js\n    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:1075:15)\n    at Module.Hook._require.Module.require (/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/node_modules/require-in-the-middle/index.js:61:29)\n    at require (node:internal/modules/cjs/helpers:110:18)\n    at Object.<anonymous> (/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/plugins/securityAnalyticsDashboards/server/utils/constants.ts:6:1)\n    at Module._compile (node:internal/modules/cjs/loader:1254:14)\n    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1308:10)\n    at Module.load (node:internal/modules/cjs/loader:1117:32)\n    at Function.Module._load (node:internal/modules/cjs/loader:958:12)\n    at Module.require (node:internal/modules/cjs/loader:1141:19)\n    at Module.Hook._require.Module.require (/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/node_modules/require-in-the-middle/index.js:80:39)\n    at Module.Hook._require.Module.require (/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/node_modules/require-in-the-middle/index.js:80:39)\n    at Module.Hook._require.Module.require (/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/node_modules/require-in-the-middle/index.js:80:39)\n    at Module.Hook._require.Module.require (/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/node_modules/require-in-the-middle/index.js:80:39)\n    at Module.Hook._require.Module.require (/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/node_modules/require-in-the-middle/index.js:80:39)\n    at require (node:internal/modules/cjs/helpers:110:18)\n    at Object.<anonymous> (/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/plugins/securityAnalyticsDashboards/server/clusters/createSecurityAnalyticsCluster.ts:6:1) {\n  code: 'MODULE_NOT_FOUND',\n  requireStack: [\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/plugins/securityAnalyticsDashboards/server/utils/constants.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/plugins/securityAnalyticsDashboards/server/clusters/createSecurityAnalyticsCluster.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/plugins/securityAnalyticsDashboards/server/plugin.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/plugins/securityAnalyticsDashboards/server/index.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/plugins/plugin.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/plugins/discovery/plugins_discovery.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/plugins/discovery/index.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/plugins/plugins_service.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/plugins/index.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/core/server/index.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/cli/serve/serve.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/cli/cli.js',\n    '/tmp/tmphnsq2vgw/local-test-cluster/opensearch-dashboards-2.12.0/src/cli/dist.js'\n  ]\n}"}

Should be solved by this PR:

https://github.com/opensearch-project/security-analytics-dashboards-plugin/pull/875

New issue:

https://github.com/opensearch-project/opensearch-build/issues/4115#issuecomment-1947502229

There seems to be a bug where Windows will run better if:
# OpenSearch:
http.host: <IP_ADDRESS>
network.host: <IP_ADDRESS>

# OpenSearch-Dashboards:
server.host: <IP_ADDRESS>

If not this then it will nearly 100% fail the manual run of Windows ftrepo directly.
Very weird.

Resolved now.


colorizing events

Expected Behavior
'kubecolor events' will produce the same colorized output as 'kubecolor describe'
Current Behavior
'kubecolor events' produces plain output
Output of 'kubecolor describe'

Output of 'kubecolor events'

Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Operating System: Windows 10
kubectl version: 1.26
kubecolor version: 0.2.2
shell environment: Powershell 7.4.0

Did not know kubectl events was a thing. I've always used kubectl get events. This should definitely be fixed!
Thanks for reporting this, and extra thanks for providing screenshots @mattcockrell !

Thanks for being so active. kubectl events is nice because you can add a --for parameter to target a specific resource. kubectl events --for pod/podname is what I was using when I noticed the missing color.

Hi @applejag
I can work on that.
It should be easy since there is already coloring functionality for kubectl get events.