React Native: ReferenceError: Property 'MessageChannel' doesn't exist

In React Native, i get a crash here: https://github.com/nbd-wtf/nostr-tools/blob/master/helpers.ts#L5 when do relay.subscribe, i get event btw.
Looks like class MessageChannel is not part of React Native.

Lol I saw that. I think that function should be replaced with return new Promise(resolve => setTimeout(resolve)) for basically the same effect.
But you are still going to have the problems in #80. Try downgrading to 1.x and see if you can get it to work at all.

Well recreated the class that mimics the behavior of MessageChannel in React Native, leave it here if helps someone.
class MessageChannel {
  constructor() {
    this.port1 = new MessagePort();
    this.port2 = new MessagePort();
    this.port1.setOtherPort(this.port2);
    this.port2.setOtherPort(this.port1);
  }
}

class MessagePort {
  constructor() {
    this.otherPort = null;
    this.listeners = new Map();
  }

  setOtherPort(otherPort) {
    this.otherPort = otherPort;
  }

  addEventListener(event, listener) {
    if (!this.listeners.has(event)) {
      this.listeners.set(event, []);
    }
    this.listeners.get(event).push(listener);
  }

  removeEventListener(event, listener) {
    const eventListeners = this.listeners.get(event);
    if (eventListeners) {
      const index = eventListeners.indexOf(listener);
      if (index !== -1) {
        eventListeners.splice(index, 1);
      }
    }
  }

  postMessage(data) {
    this.otherPort.dispatchEvent('message', { data });
  }

  start() {
    // No-op in React Native
  }

  dispatchEvent(event, data) {
    const eventListeners = this.listeners.get(event);
    if (eventListeners) {
      eventListeners.forEach(listener => listener(data));
    }
  }
}

global.MessageChannel = MessageChannel;

global.MessageChannel is now compatible with yieldThread function in helper.ts
The crash gone and i get all events now, will be testing... Thanks @alexgleason.

Used @damiandizeo 's workaround and it works!, can't evaluate the stability yet, will update if it breaks elsewhere! But thank you for posting! ❤

Well recreated the class that mimics the behavior of MessageChannel in React Native, leave it here if helps someone.
class MessageChannel {
  constructor() {
    this.port1 = new MessagePort();
    this.port2 = new MessagePort();
    this.port1.setOtherPort(this.port2);
    this.port2.setOtherPort(this.port1);
  }
}

class MessagePort {
  constructor() {
    this.otherPort = null;
    this.listeners = new Map();
  }

  setOtherPort(otherPort) {
    this.otherPort = otherPort;
  }

  addEventListener(event, listener) {
    if (!this.listeners.has(event)) {
      this.listeners.set(event, []);
    }
    this.listeners.get(event).push(listener);
  }

  removeEventListener(event, listener) {
    const eventListeners = this.listeners.get(event);
    if (eventListeners) {
      const index = eventListeners.indexOf(listener);
      if (index !== -1) {
        eventListeners.splice(index, 1);
      }
    }
  }

  postMessage(data) {
    this.otherPort.dispatchEvent('message', { data });
  }

  start() {
    // No-op in React Native
  }

  dispatchEvent(event, data) {
    const eventListeners = this.listeners.get(event);
    if (eventListeners) {
      eventListeners.forEach(listener => listener(data));
    }
  }
}

global.MessageChannel = MessageChannel;

global.MessageChannel is now compatible with yieldThread function in helper.ts
The crash gone and i get all events now, will be testing... Thanks @alexgleason.

This is awesome, thanks for posting this solution

The solution of @damiandizeo worked for me, but isn't TypeScript typed, so I had to put a bunch of @ts-ignore in.
Alternatively, I was also able to solve the issue by installing MessageChannel polyfill:
npm i message-port-polyfill

and importing it before nostr-tools:
import 'message-port-polyfill'


Export TrOCR to ONNX

I was trying to export my fine-tuned TrOCR model to ONNX using following command. I didn't get any errors, but in onnx folder only encoder model is saved.
!python -m transformers.onnx --model=model_path --feature=vision2seq-lm onnx/ --atol 1e-2

So, regarding this, I have 2 questions.

How to save decoder_model.onnx, so that I can use this inference script.
If it is not possible to export the decoder model to ONNX, how can I perform inference using encoder_model.onnx? According to my understanding, model.generate() takes time to generate output, while the decode method doesn't consume as much time compared to the generate method. Is there any way to use encoder_model.onnx with the existing decoder model in order to optimize response time?

p = processor(image, return_tensors="pt").pixel_values
generated_ids = model.generate(
    p,
    do_sample=True,
    top_k=5,
    top_p=0.1,
    num_beams=4,
    num_return_sequences=1,
    output_scores=True,
    use_cache=True,
    return_dict_in_generate=True
)

generated_text = processor.batch_decode(generated_ids.sequences, skip_special_tokens=True)[0]

Please correct me if this approach to optimize response time is wrong.
Thanks.

Hi @VallabhMahajan1
I tried to reproduce the issue using this command:
python -m transformers.onnx --model=microsoft/trocr-small-printed --feature=vision2seq-lm onnx/ --atol 1e-2
This generates the following files:
./onnx
├── config.json
├── decoder_model.onnx
├── encoder_model.onnx
├── generation_config.json
├── preprocessor_config.json
├── sentencepiece.bpe.model
├── special_tokens_map.json
├── tokenizer.json
└── tokenizer_config.json

As you can see, there is an ONNX file for the decoder present. Which TrOCR model from the hub are you finetuning?

Solution has been provided. Closing due to inactivity.


Faulty emit for union type of arrays

TypeScript Version:  2.0 RTM
Code

const  foo = [] as any | [''];

Expected behavior:
The emit should be the following:
var foo = [] | [''];

Actual behavior:
The actual emit is
var foo = [];

This was actually an accident, I meant to write const  foo = <getArray()> as any || ['']; where the left-hand array could be null. I had accidentally left out a pipe.
There should have been a runtime error at least.

The as operator expects a type as the right hand operand, and any | [''] is indeed a type. You need to add parentheses to get the expected behavior: ([] as any) | [''].

@ahejlsberg, it's the emit that's the problem.

I see what's going on. Thanks.


Support config system tag info

Background
In the actual implementation process, customers need project dimension management.
Expect
easeagent support config project.id to set the field system.

fixed at https://github.com/megaease/easeagent/pull/75


endpoint

Following this post http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html , why do we not use * as the endpoint for describing the API rather than using openapi.json or openapi.yaml? We can then support the OPTIONS method per resource with a documentation fragment, as well as for the entireity of the server

Just a note: the current HTTP 1.1 spec is RFC's 7230ff, in this case RFC 7231, section 4.3.7.
So you are suggesting to deliver the/an OpenAPI document (the definitions for all supported paths, likely) when doing an OPTIONS request on * (and fragments thereof for OPTIONS requests on specific paths). I guess this is possible (likely just when a sensible Accept: header is set), but I'm not sure how useful it would be in the general case, and whether it would interfere with other uses of OPTIONS.

There are two issues with doing that. As @ePaul mentioned, some APIs use OPTIONS as part of their API. Not going into whether that's a good decision or not, removing support to that seems like an unnecessary limitation.
The other issue is fragments - we've toyed with the idea before (and it's not necessarily a spec concern) but it could be an issue with references. It's true that while serving the fragments, references can be dereferenced, but that takes away some obvious benefits.

So ignoring the fragment case of using OPTIONS per resource which I can agree might be a sore spot for people who are currently using the OPTIONS method in their existing APIs, the OPTIONS * request is still a viable choice?

I think hijacking OPTIONS and tying them to a specific use case would be a very bad idea and create more problems than solutions.


408 - Fix: Allow reconnection if SSL fails because of encrypt/decrypt errors

👍  Merge at will.

Rebased and merged at 23f4572e2fd52ed3f567b16bd392eefb66cf7608.


Increment progress by one instead of setting progress to the specified value

Why there is no .increment() or .update() parameterless method that would increment progress by 1 by default? This would be really handy, especially if ProgressBar object is passed to function handler (e.g. to copy_function in shutil.move function as I currently do).
I'd like to be able to update progress by one using bar.update() or bar.increment() instead of calling ugly bar.update(bar.value + 1) (or possibly bar.__next__()).

Sorry for the slow response, completely forgot about your question.
By default the bar.__next__() method actually does exactly what you're looking for: https://github.com/WoLpH/python-progressbar/blob/develop/progressbar/bar.py#L430-L440
    def __next__(self):
        try:
            value = next(self._iterable)
            if self.start_time is None:
                self.start()
            else:
                self.update(self.value + 1)
            return value
        except StopIteration:
            self.finish()
            raise

For Python 2 compatibility (and your convenience in this case) you can also use bar.next()

Calling __next__() explicitly doesn't look good (I think that functions with leading and trailing underscores shouldn't be called explicitly in well designed application), but if calling next() does the same, it's OK for me. :-)

I agree :) The shorthand method is definitely the recommended option of the two


Support async iterator for calendar events

Add an async iterator support to page through calendar events
Issue #23

Codecov Report

Merging #27 (4ccb116) into main (97d06bc) will increase coverage by 0.45%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
+ Coverage   94.19%   94.65%   +0.45%
==========================================
  Files           5        5
  Lines         224      243      +19
==========================================
+ Hits          211      230      +19
  Misses         13       13

Impacted Files
Coverage Δ

gcal_sync/api.py
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Incorrect export data limitation

In the limitations section it says "Export data isn't available for users outside of your organization who are using a dashboard that an internal user has shared with them." I have tested this limitation and I do not think it a limitation any more. An internal user can export tiles to Excel and it also works for external users sharing using AAD B2B.

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

ID: c87c556b-b1da-0b24-606b-89c371c6bae8
Version Independent ID: 20141574-acfa-f5f8-b6cf-6b32b3c55b4f
Content: Export data from a Power BI visualization - Power BI
Content Source: powerbi-docs/visuals/power-bi-visualization-export-data.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @lanndermsft. I've assigned this issue to the doc author for review.


Implement more detail in the Get-SbQueue output

Based on the suggestion: https://github.com/tommagumma/PSServiceBus/issues/27#issuecomment-587413153 & https://github.com/tommagumma/PSServiceBus/issues/27#issuecomment-587416962
The task is up for grabs!

I got it! ✋

My test says this is a keeper!


Brew doctor throws error

Running $ brew doctor gives the following error:
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: Unbrewed dylibs were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected dylibs:
  /usr/local/lib/libcdt.5.dylib
  /usr/local/lib/libcgraph.6.dylib
  /usr/local/lib/libgvc.6.dylib
  /usr/local/lib/libgvpr.2.dylib
  /usr/local/lib/libpathplan.4.dylib
  /usr/local/lib/libxdot.4.dylib

Warning: Unbrewed header files were found in /usr/local/include.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected header files:
  /usr/local/include/graphviz/arith.h
  /usr/local/include/graphviz/cdt.h
  /usr/local/include/graphviz/cgraph.h
  /usr/local/include/graphviz/color.h
  /usr/local/include/graphviz/geom.h
  /usr/local/include/graphviz/graphviz_version.h
  /usr/local/include/graphviz/gvc.h
  /usr/local/include/graphviz/gvcext.h
  /usr/local/include/graphviz/gvcjob.h
  /usr/local/include/graphviz/gvcommon.h
  /usr/local/include/graphviz/gvconfig.h
  /usr/local/include/graphviz/gvplugin.h
  /usr/local/include/graphviz/gvplugin_device.h
  /usr/local/include/graphviz/gvplugin_layout.h
  /usr/local/include/graphviz/gvplugin_loadimage.h
  /usr/local/include/graphviz/gvplugin_render.h
  /usr/local/include/graphviz/gvplugin_textlayout.h
  /usr/local/include/graphviz/gvpr.h
  /usr/local/include/graphviz/pack.h
  /usr/local/include/graphviz/pathgeom.h
  /usr/local/include/graphviz/pathplan.h
  /usr/local/include/graphviz/textspan.h
  /usr/local/include/graphviz/types.h
  /usr/local/include/graphviz/usershape.h
  /usr/local/include/graphviz/xdot.h

Warning: Unbrewed .pc files were found in /usr/local/lib/pkgconfig.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected .pc files:
  /usr/local/lib/pkgconfig/libcdt.pc
  /usr/local/lib/pkgconfig/libcgraph.pc
  /usr/local/lib/pkgconfig/libgvc.pc
  /usr/local/lib/pkgconfig/libgvpr.pc
  /usr/local/lib/pkgconfig/libpathplan.pc
  /usr/local/lib/pkgconfig/libxdot.pc

Warning: You have unlinked kegs in your Cellar
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  binutilsfb
  dwarfutils
  graphviz
  jemallocfb
  node012
  ocaml
  php56
Error: /usr/local/Homebrew/Library/Taps/rcmdnk/homebrew-file/brew-file.rb:1: syntax error, unexpected <<
<<<<<<< 993416d995bac57048343ecae5526796fcdb485c
  ^
Please report this bug:
  https://git.io/brew-troubleshooting
/usr/local/Homebrew/Library/Homebrew/formulary.rb:25:in `module_eval'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:25:in `load_formula'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:42:in `load_formula_from_path'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:97:in `load_file'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:88:in `klass'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:84:in `get_formula'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:182:in `get_formula'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:222:in `factory'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:253:in `from_keg'
/usr/local/Homebrew/Library/Homebrew/formulary.rb:236:in `from_rack'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1357:in `block in installed'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1355:in `map'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1355:in `installed'
/usr/local/Homebrew/Library/Homebrew/diagnostic.rb:837:in `check_missing_deps'
/usr/local/Homebrew/Library/Homebrew/cmd/doctor.rb:43:in `block in doctor'
/usr/local/Homebrew/Library/Homebrew/cmd/doctor.rb:35:in `each'
/usr/local/Homebrew/Library/Homebrew/cmd/doctor.rb:35:in `doctor'
/usr/local/Homebrew/Library/Homebrew/brew.rb:94:in `<main>'

Ok, solved it. Package brew-file was the culprit.


Uncaught TypeError: t.toLowerCase is not a function

While passing a nested JS object literal, Fuse keep throwing this error .
Uncaught TypeError: t.toLowerCase is not a function.

Did anyone else face this issue ?

The issue was not with Fuse but in my code I was passing a Regex instead of string.


Set Require Only App-Extension-Safe API to Yes

Fix error about linking against dylib not safe for use in application extensions.

Thanks!

:+1:


IoTHubOwner connection unstable

I want to establish a stable connection between my Rpi4 device to IoTHub Owner (not IOTHub device) using python sdkv2.
previously, in Azure IoTHub sdk1, the class (IotHubMessaging) from iothub_service_client was used to establish a connection between IoTHub Owner and the device which provide a stable connection.
from iothub_service_client import IoTHubMessaging, IoTHubMessage, IoTHubError
But in the newer IotHub sdkv2, this class not more available.
I tried IoTHubRegistryManager from sdkv2 which is built over AMQP to establish the same connection, but the commutation is not stable and disconnected after 10 - 15 mins.
registry_manager = IoTHubRegistryManager.from_connection_string(connection_str)
Is there any stable replacement for the IoTHubMessaging class in sdkv2 that can I use?

Hi @SaifAlmaliki, thanks for bringing up this issue. A few questions:

Based on my understanding of your issue, it is C2D messages which fail after 10-15 minutes. Is this understanding correct? Are there other operations on IoTHubRegistryManager which fail for you besides C2D?
Can you elaborate on what happens when the failure occurs? Is there an error message logged? Does it fail silently? Does it hang?

Thanks!

@SaifAlmaliki
A fix for this issue has been added as part of our release of azure-iot-hub version 2.6.0, available now via pip
Please let us know if this has addressed your issue


'added new login page'

Hey @adigunsherif I've made a new login page with custom templates and some littile bit of bootstrap tweaks. can you please review it and merge it. My code will be li'l bit messy. ..

Greatly appreciated for this taking this initiative. Would come for a discussion if you wanna have.

Yeah sure.
Is the pull request merged??

Hey @adigunsherif i have made some changes to my master. can you review it

hey @adigunsherif I've made the changes .. can you review it and merge it

Sorry for the late reply. Will check and merge before the end of the day.
On Sat, Oct 3, 2020, 3:49 PM Aman Kumar<EMAIL_ADDRESS>wrote:

hey @adigunsherif https://github.com/adigunsherif I've made the changes
.. can you review it and merge it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/adigunsherif/Django-School-Management-System/pull/19#issuecomment-703115363,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHL6TAA3Y32A4AUAJVDLM3SI42YJANCNFSM4SAV66TA
.

Great job @aman2457 . I have checked and confirmed this on my local setup. I have few comments before approving these changes.

The background color on the login page is far from the overall color of the application itself. Can this be made as simple as the application is, like using  a grey background or something close or no background.
Since the application won't allow registration by external users, it makes sense not to have "create your account"
Since the "Forgot Username / Password?" is not yet linked, please in your update, add the "password change" to the todo.txt file so we can keep track of this.
I'm available to merge this night if you can work on these asap.
Great job man

Hey @adigunsherif right now its 1:00am night. Can i work it early in the morning like after 8-10 hours. Will that be okay??

No problem. At you convenience
On Sat, Oct 3, 2020, 7:48 PM Aman Kumar<EMAIL_ADDRESS>wrote:

Great job @aman2457 https://github.com/aman2457 . I have checked and
confirmed this on my local setup. I have few comments before approving
these changes.

The background color on the login page is far from the overall
color of the application itself. Can this be made as simple as the
application is, like using a grey background or something close or no
background.
Since the application won't allow registration by external users,
it makes sense not to have "create your account"
Since the "Forgot Username / Password?" is not yet linked, please
in your update, add the "password change" to the todo.txt file so we can
keep track of this.
I'm available to merge this night if you can work on these asap.
Great job man

Hey @adigunsherif https://github.com/adigunsherif right now its 1:00am
night. Can i work it early in the morning like after 8-10 hours. Will that
be okay??
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/adigunsherif/Django-School-Management-System/pull/19#issuecomment-703148246,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHL6TEOWAKHEEAZDD53J7TSI5WWNANCNFSM4SAV66TA
.

@adigunsherif updated..

Thank you for your effort. Will merge ASAP
On Sun, Oct 4, 2020, 8:16 AM Aman Kumar<EMAIL_ADDRESS>wrote:

[image: Screenshot from 2020-10-04 12-45-17]
https://user-images.githubusercontent.com/54279054/95009412-90624e00-063f-11eb-8496-5221c6eac488.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/adigunsherif/Django-School-Management-System/pull/19#issuecomment-703214261,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHL6TA6ZEFQJVW3WCK3S5LSJAONDANCNFSM4SAV66TA
.

Thank you for your effort. Will merge ASAP
…
On Sun, Oct 4, 2020, 8:16 AM Aman Kumar @.***> wrote: [image: Screenshot from 2020-10-04 12-45-17] https://user-images.githubusercontent.com/54279054/95009412-90624e00-063f-11eb-8496-5221c6eac488.png — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#19 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHHL6TA6ZEFQJVW3WCK3S5LSJAONDANCNFSM4SAV66TA .

yeah @adigunsherif it would be great. and after merging close this PR also


Markdown Tables not working in Outlook Desktop

Using the following Markdown,
`

Request Details

Article Description
Cell

What do you want to do?
Cell

Article Number
Cell

GL Code for Purchase Orders
Cell

Vendor Code
Cell

Account Assignment Group
Cell

Article Pricing Group
Cell

Comments
Cell

`

Results in the following in Outlook on the Web:

And the following on Outlook desktop, version 16.0.13127.20266

Something is very clearly not working.

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

ID: e22eb42c-6ab3-854c-c229-569f1ca6d2ea
Version Independent ID: b86cc920-b40d-3a2c-c88b-a36e3692fbbc
Content: Use markdown to format Power Automate approvals - Power Automate
Content Source: articles/approvals-markdown-support.md
Service: flow
GitHub Login: @gcorvera
Microsoft Alias: gcorvera

@HTEredrock  Thanks for reporting this issue. We'll investigate this.

@dblyth-msft Can you please evaluate whether this is a docs issue? Let me know if there's anything needed to be updated in docs to help with this.

This is a known bug in Outlook when the requestor feature is used ("Requested by"). This should be fixed by the end of the month or so.

From: Dean Haas<EMAIL_ADDRESS>Sent: Wednesday, September 16, 2020 9:01 PM
To: MicrosoftDocs/power-automate-docs<EMAIL_ADDRESS>Cc: David Blyth<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/power-automate-docs] Markdown Tables not working in Outlook Desktop (#318)
@dblyth-msfthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdblyth-msft&data=02|01|dblyth%40microsoft.com|15feac30f0034c7e340508d85abe5d88|72f988bf86f141af91ab2d7cd011db47|1|0|637359120940333099&sdata=gTEzo21vUoEon0NQERAtxiixS4Rb08v4ifq0m8hyl8A%3D&reserved=0 Can you please evaluate whether this is a docs issue? Let me know if there's anything needed to be updated in docs to help with this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fpower-automate-docs%2Fissues%2F318%23issuecomment-693794349&data=02|01|dblyth%40microsoft.com|15feac30f0034c7e340508d85abe5d88|72f988bf86f141af91ab2d7cd011db47|1|0|637359120940333099&sdata=rqnPq870dCIsfMCgpgn0HJLK8tAuZ8eT6RlyXjBU7fs%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAISOUHA7TPBDTCPNVSFKFRDSGGCZ3ANCNFSM4QM5L32A&data=02|01|dblyth%40microsoft.com|15feac30f0034c7e340508d85abe5d88|72f988bf86f141af91ab2d7cd011db47|1|0|637359120940343059&sdata=8XxBl%2BQbPaeoTiWXB3ecQNCPCxOL2WNPDRNXxVpFGzQ%3D&reserved=0.

Closing this out as a known issue. Thanks everyone.

Hi,
Any updates on this issue? I am still experiencing the same table issue on Outlook Desktop.

Hi,
Any updates on this issue? I am still experiencing the same table issue on Outlook Desktop.


Exec tachikoma update<PHONE_NUMBER>5807

:hamster::hamster::hamster:   Strategy david
Powered by David Update as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage decreased (-2.2%) to 75.87% when pulling edf549229ace34e3eca805fb565ca00cb2525e06 on pazjacket:tachikoma/update-20161113165807 into 561aeec19d8836578f0838fe4cbff883050bb66d on festivals-tech:master.


Add tests that check generated output

The current tests only check if the result returned via a --check call is right. In other words, they only assure that valid files pass (with error code 0) and invalid files fail (with error code 1).
To make sure, that tsv files are generated correctly, we need to prepare some expected tsv output files, call the tool in the test, and compare outputs. This can either be done via tempfiles or by adding the option to write to stdout.

A minimal conversion test was added in #21


Added prettier linter workflow

Fixes Issue
Closes #828

Changes proposed

Added the prettier linter to ensure proper and uniform linting of the codebase.
For the linter to run successfully please open the settings of the repo go to Actions -> General -> workflow permissions and enable read and write permissions.

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

Also fixed the deployment issue while making the PR.

Hey, I see you modified 51 files? It's a lot too to merge..also your branch is giving conflicts..

The changes made by me are only in linting.yml. The rest are the changes by the prettier linter, and is automated.

@MonalikaPatnaik Updated PR in #842


Add a Gitter chat badge to README.md

crystal/crystal now has a Chat Room on Gitter
@ctate has just created a chat room. You can visit it here: https://gitter.im/crystal/crystal.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Thank you!


Listen on <IP_ADDRESS> in collector

As of v0.104.0, the collector listens on localhost and not <IP_ADDRESS>. This breaks containerized environments like cloudrun
https://github.com/open-telemetry/opentelemetry-collector/releases/tag/v0.104.0

I can add this to all configs -- just lmk if it looks good

Updated!
I see that the dockerfile is pinned, and I totally overlooked that when debugging what could have gone wrong. But I'd guess people will be following this template as a guide, and copying some parts but not all. So this backwards compatible change would make sense to include!

Thanks for the PR @ottaviohartman


fix if condition in Test_ThrottleCacheRateFail

I think Test_ThrottleCacheRateFail expects throttle.OK(key) return fail when query exceeds cache rate limiting.

Mmmmm, testing your PR and I get test failures:
pallotron@Pallotron.local:(test_pull_request_17) ~/projects/dhcplb/lib
$ go test
--- FAIL: Test_ThrottleSingleConnectionFail (1.09s)
	throttle_test.go:121: Throttling didn't work for single connection!
--- FAIL: Test_ThrottleCacheRateFail (1.07s)
	throttle_test.go:169: Throttling didn't work for cache rate limiting
FAIL
exit status 1
FAIL	_/Users/pallotron/projects/dhcplb/lib	5.950s
[Exit code 1 @ 12:22:57]

I will take a closer look later...

Thank you for your reply.
I think this is another issue. I have not changed "Test_ThrottleSingleConnectionFail".
I got test failure, when I run on mac, though I did'n get when I run on ubuntu vm.
First.
This pull request is for changing the success condition of "Test_ThrottleCacheRateFail".
I think "Test_ThrottleCacheRateFail" is for cheking cache rate limiting function of Throttle is working or not.
And this test expects throttle.OK(key) will return the false, when throttle.OK(key) query exceeds the const cacheRate = 64.
So I think this test should success when throttle.OK return false.

  if ok, _ := throttle.OK(key); ok {

  if ok, _ := throttle.OK(key); !ok {

Second.
I think the reason of test failure is value of sleeptime and loopCount.
throttle.OK(key) call the limiter.Allow() and check whether query exceeds limit or not.
limiter use the token bucket.
Below is comment from the "golang.org/x/time/rate".
// A Limiter controls how frequently events are allowed to happen.
// It implements a "token bucket" of size b, initially full and refilled
// at rate r tokens per second.
I will take "Test_ThrottleSingleConnectionFail" as an exmaple and explain why this fail.
const connRate = 64 // per sec
So token bucket first has 64 token and refille at rate 64 tokens per second.
loopCount := connRate * 2
sleepTime := time.Duration((1000 / connRate) / 2)
So this loop call throttle.OK(key) 128 times and it takes around 1s.
If this loop takes more than 1s, throttle.OK(key) don't return false.
If this loop takes less than 1s, throttle.OK(key) return false.
So I don't think this loopCount and sleepTime is Adequate for test.
I think you should change
loopCount := connRate * 3
(or change sleepTime := time.Duration((1000 / connRate) / 3))

Yeah you are right that sleeping is never a good idea in unit tests :)

So I tried this and it works ok on my laptop: https://gist.github.com/pallotron/1918f061023cec53e609aa9ca859680c
So I'd say amend your PR and I will merge it :)
Thank you very much for catching this.

Thank you for reviewing and testing my PR.

 I see no changes...

you have to apply changes as per ttps://gist.github.com/pallotron/1918f061023cec53e609aa9ca859680c

Sorry I made mistake.
I'm inexperienced in the github, I will fix.
I'm really sorry for this inconvenience.

don't worry :) everything is fine, take your time :)

thank you. I have updated.


Add a new function called thousandthToNumber

Description

Resolves #(issue number)

Lodash[ACTION] #100 -> https://lodash.com/docs/4.17.4#(method)
What does your PR belong to?

[ ] Website
[x] Snippets
[ ] General / Things regarding the repository (like CI Integration)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (non-breaking improvement of a snippet)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Lodash Backlog

[x] I added the prefix [UPDATE: method.md] or [ADD: method.md]
[ ] I have referenced the method to the lodash backlog.

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have checked that the changes are working properly
[x] I have checked that there isn't any PR doing the same
[x] I have read the CONTRIBUTING document.

Thanks for taking the time to contribute! However, we already have this snippet in our codebase under toDecimalMark.


Can not use husky hooks

Version
3.0.0-beta.6
Steps to reproduce
vue create project
cd project
yarn add husky
Test hooks

It could be nice to document that @vue/cli-service is using yorkie for some lint, and a guide to add custom hooks

Yeah, yorkie does the same thing with husky. Use "gitHooks" instead of "husky", and use dash delimited hook names.

I did use yorkie at the end (note that the next version of husky does kinda the same as yorkie - moving from scripts to husky, pre-commit instead of precommit)
The thing is, I had to check in my .git/hooks/pre-commit, then yarn why yorkie to find out.
It would be cool to have a line in the documentation :)

Yeah, the docs is still WIP, we will have lots of things to add when we reach RC.

can there be an update to move Vue-CLI to support Husky? i noticed Yorkie has not been updated since late 2018

@yyx990803
I agree with the comment above.
P.s. Husky is supporting a project in a subdirectory, but Yorkie doesn't.

@yyx990803
I agree with the comment above.
P.s. Husky is supporting a project in a subdirectory, but Yorkie doesn't.

Agreed. Also missing subdirectory project support.


Installing RBenv and Ruby on Mac with Apple M1 Machine

I saw this answered in another post but the answer did not help me unfortunately. I am trying to install on my terminal
a command-line tool called RBenv, for Ruby using the following command :
RUBY_CFLAGS="-Wno-error=implicit-function-declaration" rbenv install 2.6.6

However I am getting the following error message:
BUILD FAILED (macOS 12.4 using ruby-build 20220713)

Inspect or clean up the working tree at /var/folders/n6/1yg6j3z56r5_dqkq48d2gy0h0000gn/T/ruby-build.20220721174039.33183.vsvD3U
Results logged to /var/folders/n6/1yg6j3z56r5_dqkq48d2gy0h0000gn/T/ruby-build.20220721174039.33183.log

Last 10 log lines:
clang  -Iinclude -arch arm64 -O3 -Wall -D_REENTRANT -DZLIB -DZLIB_SHARED -DNDEBUG -I/Users/anasosa/.rbenv/versions/2.6.6/include  -MMD -MF test/wpackettest.d.tmp -MT test/wpackettest.o -c -o test/wpackettest.o test/wpackettest.c
clang  -Iinclude -arch arm64 -O3 -Wall -D_REENTRANT -DZLIB -DZLIB_SHARED -DNDEBUG -I/Users/anasosa/.rbenv/versions/2.6.6/include  -MMD -MF test/x509_check_cert_pkey_test.d.tmp -MT test/x509_check_cert_pkey_test.o -c -o test/x509_check_cert_pkey_test.o test/x509_check_cert_pkey_test.c
test/v3ext.c:201:24: error: implicitly declaring library function 'memcmp' with type 'int (const void *, const void *, unsigned long)' [-Werror,-Wimplicit-function-declaration]
        if (!TEST_true(memcmp(ip1->data, ip2->data, ip1->length) <= 0))
                       ^
test/v3ext.c:201:24: note: include the header <string.h> or explicitly provide a declaration for 'memcmp'
1 error generated.
make[1]: *** [test/v3ext.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [all] Error 2
xxxxx@xxx-MacBook-Pro ~ %

Any idea how to fix this? I read this is a common issue however I am not sure what to do since I tried everything that I saw online.
ps: This is my first post, thank you for your patience :)

As the name suggests RUBY_CFLAGS is used for building Ruby, not for OpenSSL. You could use another env var, if any.
For OpenSSL's side, this issue is a duplicate of #18720.

Duplicate of https://github.com/openssl/openssl/issues/18720


Document fixing tslint with update to @types/vscode

With the update to @types/vscode, the extension lint settings, which I pulled from another extension, causes the following error every time vscode is imported.
Module 'vscode' is not listed as dependency in package.json
I am not a JS developer normally, so I had no clue how to fix this, and the release notes showed no way on how to do this either. Looking around, it looks like adding "no-implicit-dependencies": [true, "dev"], to my tslist settings, but I realized I already had that, and that doesn't work. What is the right way to solve this issue, and can a solution be added to either the release notes or documentation on building extensions?

I think this would be an issue with the tslint extension you have. If running npx tslint locally doesn't yield any error, that's probably due to extension not resolving the config correctly.
Open an issue for https://marketplace.visualstudio.com/items?itemName=eg2.tslint or https://marketplace.visualstudio.com/items?itemName=ms-vscode.vscode-typescript-tslint-plugin, whichever you are using. (Also, try the deprecated one if the new one doesn't work).

@octref  Its not caused by an extension, actually running npm lint directly from the command line causes the error. Its more of a documentation error for VS Code, because it updated how it provides itself to extensions, but not common issues that could occur, not this one.

From other thread I saw "no-implicit-dependencies": [true, "dev"] was working. Is it not working for you? Then you should probably report an issue for tslint. Some global extension runtime such as vscode, chrome doesn't have direct dependencies as they are provided by the tool's runtime.

same here for me.
"no-implicit-dependencies": [true, "dev"] is not working.


Make DialogType public

This PR makes DialogType public, because it's necessary for generic purposes. Without it I have to wrap all calls with own enum type which makes program wasty. I didn't find any downside if it's becomes public.

Looks good. Thanks!


UDPEndPointImplLwIP: Support LWIP_TCPIP_CORE_LOCKING=0

Wrap calls to LwIP APIs in tcpip_api_call(), as required. When LWIP_TCPIP_CORE_LOCKING is enabled, this internally becomes LOCK_TCPIP_CORE/UNLOCK_TCPIP_CORE and when it isn't, it posts a message to the TCPIP task to run the function.
https://github.com/project-chip/connectedhomeip/issues/28590
cc @shubhamdp

 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.

@bzbarsky-apple ok, it's all green now, PTAL.
i realized that there is a TCP endpoint and it needs similar treatment. i'm not going to do it now (we don't use it in our project) but i moved the assertion there from the UDP endpoint and added a TODO.


Using the video tag works normally on Chrome live preview, doesn't work on regular Google Chrome

I created a website using Bracket's live preview feature which uses Google Chrome for rendering and stuff.  I used a sample .mp4 and used video.js for the video player.  There is also a fall back .webm file.  It loads and works normally when using live preview.  When I tested this out on Opera, Firefox and Safari, they didn't load the video.  I then tested this on Chrome outside of the live preview, and that didn't work either.  These two screenshots are Live preview, and Chrome without live preview.

Bumping this for help.  Does the video only work if it's on a live server?  Because the way I'm trying to do it is just simply drag and drop the HTML file into the tab place on the browsers.

I figured out that I used single quotes instead of double quotes in my HTML tag.


Get column names when the query has no results

I'm getting the column names using Object.keys(recordset[0]), to show the results in a table. But when the query has no results, and the recordset is empty, this code, obviously, is not going to work.
There is any way to get the column names when the query has no results?

You can use recordset.columns. More info in docs.

Oh, sorry that I missed this into the documentation...
Thanks @patriksimek.


VSO 1715076: Update getting-biztalk-server-assistance.md

Replaced broken/redirected links

Docs Build status updates of commit 401b09e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/getting-biztalk-server-assistance.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


github-user improvements & footer updates

This pull request does the following:

Adds more information to the footer and restyles it to match the new website styling
Refactors the github-user partial to have better error handling / reporting, more arguments, and lazy loading.
Adds a githubUser shortcode

me when loc


Clarify how gorouter handles W3C tracing headers

Documentation states:
* If the `traceparent` and `tracestate` HTTP headers are present in the request, the Gorouter forwards them unmodified.

But that is not what is happening. Gorouter creates a new SPAN ID and updates the tracestate with the tenant, as it is tested in its integration tests:
https://github.com/cloudfoundry/gorouter/blob/9ee77f4fdebd038c1978ecd5982aa2d3ae88b503/integration/w3c_tracing_test.go#L158

Thanks @jriguera - would it be correct and helpful to instead say this?:

If the traceparent and tracestate HTTP headers are present in the request, the Gorouter updates the tracestate value to a new SPAN ID in the HTTP headers before forwarding the request.

And if so (or either way) how is the new SPAN ID value derived?
Thanks again.

The new spanid is generated by the gorouter, it is a 8 byte array which represents an operation (in this case would be "reverse-proxy") in the chain of request calls (eg. one microservice calling other in the same paas). The gorouter adds the tenantid to the tracestate (the other W3C tracing header) representing itself processing the request. In my opinion is normal that the gorouter generates a new spanid and adds the tenant to tracestate because it is an actor in the communication, which can fail (eg. the route is gone) and it adds latency to the chain of requests.
More info: https://www.w3.org/TR/trace-context/#mutating-the-traceparent-field

In the first case (the documentation where this issue was open) what the sentence says is wrong. The Gorouter does not do that, it creates a new span.
In the second case:

If B3 headers are provided and they are in a proper v4 UUID format, GoRouter uses them for X-Vcap-Request-Id and W3C headers. If they are not provided, GoRouter generates v4 UUID and use it for B3 headers, X-Vcap-Request-Id and W3C headers.

It is not wrong, but it does not make it clear that it generates a new span based on that.

Thanks again @jriguera -- how would you rewrite to clarify that the Gorouter creates a new span?  Something like this (in the second case, linked page):

If B3 headers are provided and are in a proper v4 UUID format, GoRouter uses them for the X-Vcap-Request-Id and W3C headers. If they are not provided, GoRouter generates a new X-B3-Spanid and includes it in new B3, X-Vcap-Request-Id, and W3C headers.

That may not be correct and even if it is, it may not be the clearest way to explain what the Gorouter does in such cases.  What do you think would be the most helpful explanation?

Maybe something like

If Zipkin (B3) headers are provided and valid (they are in a proper v4 UUID format), GoRouter uses them to generate X-Vcap-Request-Id and W3C headers. If they are not provided, GoRouter generates a new Span ID to use it for X-Vcap-Request-Id, Zipkin and W3C headers.
When Zipkin is enabled, If W3C headers are provided but no Zipkin (B3) headers are found, GoRouter generates a new Span ID for Zipkin headers which will differ from W3C headers, resulting on different span ids for Zipkin and W3C. Applications would need to agree on which span id (Zipkin vs W3C) use for their tracing data.

I am not sure if makes things clear or not, but the reason of the second paragraph is due to some issues we have because of the behavior of generating headers in this direction Zipkin -> W3C but not in W3C -> Zipkin resulting on different SpanIDs at application level depending on if the app uses Zipkin or W3C as source for their spans.

Thanks @jriguera for continued help -- I changed the section HTTP headers for W3C tracing to just point to the section Unique trace identification for explanation-- seems like it makes the points you make above.  Or if not, please PR for any corrections.  Thanks again!


Rethink how tracing is enabled in helm chart

Since we added -enable-feature=tracing to turn on tracing, the helm chart config to enable tracing now must be as follows:
args:
- -enable-feature=tracing
- -otlp-grpc-server-listen-address=:9202
tracing:
  enabled: true

This is redundant and verbose. The following would be convenient:
tracing:
  enabled: true

(Although what happens when tracing becomes GA and is enabled by default?)

Just trying to test this and can't get it installed with or without disabled tracing.
When tracing is on it says it requires cert-manager because of the transitive dep, understandable but even when tracing.enabled is false it is still trying to create cert-manager related CRDs.
We don't have cert-manager and can't use it because it conflicts with our own PKI.
Is it not possible for promscale to depend on https://github.com/open-telemetry/opentelemetry-collector which doesn't have the cert-manager dep instead of the operator?
        tracing:
          enabled: false
        args: []
#         - "-otlp-grpc-server-listen-address=:9202"

Above fails trying to create Certificate CRD

@zcourts can create a separate issue for your question, and can you give more details on what you're doing? Are you using tobs to install promscale?

@zcourts it looks like you are using tobs to install Promscale with tracing enabled which by default installs otel-operator which internally is dependent on cert-manager.  Can you create an issue against tobs github repo? We can discuss more on this on the issue created. :)

Yes, using tobs to install promscale. See https://github.com/timescale/tobs/issues/197 that I just created.

I have 2 step proposal which should handle tracing enablement and clarify some confusions around naming.
Change section name
Since under tracing we are enabling OpenTelemetry Line Protocol (OTLP) support, I would suggest we change:
tracing:
  enabled: true

to
openTelemetry:
  enabled: true
  port: 9202

This would allow having just one switch for enabling openTelemetry support (openTelemetry.enabled) and the section would be analogous to prometheus section presented in https://github.com/timescale/promscale/pull/924.
Replace args with extraArgs
Remove args and replace it with extraArgs section in values.yaml. This would allow setting arguments based on other switches (like openTelemetry.enabled) and should not confuse users about args that are set automatically.
In Deployment template we would then do something like the following:
{{- if or (.Values.openTelemetry.enabled) (.Values.extraArgs ) }}
         args:
         {{- if .Values.openTelemetry.enabled }}
         - --enable-feature=tracing
         - --otlp-grpc-server-listen-address=:9202
         {{- end }}
         {{- with .Values.extraArgs }}
{{ toYaml . | indent 10 }}
         {{- end }}
{{- end }}

In the future, we possibly would need to also add enableFeatures section in values.yaml to handle setting features in a more controlled way. But that is out of scope for this issue.
Result
With both those proposals enablement of tracing/otlp would be handled by a single openTelemetry.enabled.

I agree with @paulfantom's suggestion on both counts

Done in https://github.com/timescale/promscale/pull/935


Reconsider if we need a parameterized Response<> type in the http package.

We currently have an HttpResponse type in the http.models package, should we have two distinct but very similar types?

Duplicate #38075


Fix fake xDS server bug that caused unnecessary round trips in NACK tests

Before this fix, whenever the xDS server received a NACK from the client, it would resend the resources, causing the client to NACK again, which basically resulted in an infinite loop.  This wasn't actually causing any tests to fail, because the right status would still be reported to gRPC, but it was causing unnecessary round trips and noise in the logs.

The "Artifact Build Linux" failure seems to be an infrastructure issue.


no matching pull request metadata found

Extension version: 0.4.0
VSCode Version: 1.31.1
OS: MacOS - 10.14.3

Steps to Reproduce:

Install github pull request vscode extension
Sign in using github enterprise account, browser opens with the vscode and also links to vscode, but no PR's are seen
Below is the log from the output window, tried multiple times with uninstall, still not working
[Info] Looking for git repository
[Info] Git repository found, initializing review manager and pr tree view.
[Info] Found GitHub remote
[Info] https://api.github.com: GraphQL support detected
[Info] {
"data": {
"viewer": {
"login": "e",
"__typename": "User"
}
},
"loading": false,
"networkStatus": 7,
"stale": false
}
[Info] Review> no matching pull request metadata found for current branch hotfix/feb-26

By default the extension is looking for PRs for upstream and origin remotes - do your remotes match this? (git remote -v) This can be configured with the githubPullRequests.remotes setting

Just added git remote add upstream https://github.com/abcd/bcdg.git and it started to show, very strange,  had never done this in the past. The tool has been flaky and fragile, hope to see more stability, but the tool has been a great help - a clear winner with no easy alternative


Code cleanup for synthetic_spatial_datagen

De-verbosify most of the comments in here, as well as the function names.
Testing: de-verbosify most of the comments

Looking back, why did I write so much lol?

<3


Not working with Webpack

Can this work with Webpack?
I'm using this THREE.js boilerplate.
https://github.com/ethanny2/threejs-es6-webpack-barebones-boilerplate

Have you tried using import?

See https://github.com/RonenNess/partykals/issues/5
Either include as another script tag in your HTML, or add the required wrapper needed for import. Someone already did it you can fork them.
Thanks


Subscribe from a Denormalizer (for integration with Legacy Systems is very useful)

We just avoid this, and in turn, use a domain service with idempotency enabled


Update docker-compose.yml

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

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

Testing Instructions

@Bohratomteorisi , please read the header on this file and the README.
Changes must to be made to the source files located in compose-builder folder.
We would not accept this as a default setting anyways, @bnevis-i correct?.
@Bohratomteorisi , you are welcome to add this to your local copy of the compose file.

We would not accept this as a default setting anyways, @bnevis-i correct?.

Correct.  @Bohratomteorisi  you should fork the repo, modify add-security.yml to add this line, and generate your own set of docker compose files.


Memory issues

The instance I'm running Security Monkey becomes unresponsive every few days.  Looking in /var/log/messages I saw oom-killer messages every ~8 hours.  Running sar reports I was able to record regular system memory usage, and saw a classic memory leak chart.  It looks like every ~2 hours the memory usage jumps up.  Ultimately this get's to near 100% and the oom-killer steps in and kills monkey.  I suspect that sometimes so much memory is being used that not even oom-killer can run, which is why the system is unresponsive (that's my assumption).
I further assume that perhaps the monkey process is getting so large because it is running every hour, and maybe one watcher is not finished by the time the next starts.  I have bumped up the interval of the watcher, but I'm unsure yet if this will resolve the issue and wondering if others have experienced this.  If this is the issue, it would be good to include some logic in Security Monkey to not starts new scans of an AWS account until the previous scan has finished.

@0xdabbad00 I experienced a similar issue. I bumped up my instance to a t2.medium and added some swap space. That seems to have mitigated the OOM problem and the systen hasn't locked up ever since.

@0xdabbad00 Not sure which instance type you are using, but in our quickstart guide for AWS, we do mention launching instances with at least 1.7GB RAM here.

I'm running Security Monkey by itself on a t2.large (8GB of RAM) with the database running on an RDS.  Due to the memory usage suddenly increasing by a large amount every 2 hours and never dropping until oomkiller comes in, this appears to be a memory leak.  I'm wondering how I can go about debugging what is causing these memory issues.

Wow, that is odd....
I guess the main questions are: which watchers are you running, and how big is the AWS account in question (number of roles, security groups, etc.)?

I'm not sure what you mean by which watchers.  It is only monitoring AWS (a few environments, one main one that I'd say is "medium" sized). I'm running the 0.9.2 release, installed two weeks ago.  I increased the schedule interval to 4 hours, with no improvement.  I turned on "debug" logging, but nothing relevant.  oom-killer is still whacking the monkey scheduler process periodically.
I don't know how to debug this.  Is there a way to say "don't slurp updates anymore" and "don't check anything anymore"?  And then start doing "only get updates on EC2" and then "only get updates on EC2 and S3", etc.  Maybe then I could start narrowing down where the memory leak is?
I also don't know how to upgrade to any newer code. I had installed by starting with git clone --depth 1 --branch v0.9.2 https://github.com/Netflix/security_monkey.git

I've tried using python's memory_profiler library with no luck on getting it to output anything, so any help there would also be appreciated.

Is there a way to say "don't slurp updates anymore"
In the UI, go to Settings->Watcher Configuration (where you probably set the interval to 4 hours) and you can enable/disable individual watchers.
v0.9.2 is latest stable release, newest functionality is on the develop branch.

I added a return to the top of the run_change_reporter function in security_monkey/scheduler.py and this appears to have stopped the memory leak from exhibiting itself, so that at least narrows down where it is.  I'll continue commenting out code. :( I don't know of a better way of debugging this.

I also don't know of a better way to debug ;)
Here is what I think should be done:

Set the log level to DEBUG (modify ALL lines in your config file: https://github.com/Netflix/security_monkey/blob/develop/env-config/config.py#L17-L54)
Disable the scheduler completely so NO watchers are running:sudo supervisorctl
stop securitymonkeyscheduler

Then, you will need to run the monkey find_changes -m <TECHNOLOGY_HERE> command. Do this one command at a time and keep note of which ones fail and which don't. If it does fail, keep note of where it's failing since the debug logs are very verbose. To get started, try these technologies first:

s3
securitygroup
iamrole
route53
vpc

Once you have that information, then we can do additional work to find the source of the issue.
You are the first user I'm aware of that has encountered this issue, so debugging will be a bit more involved.

So I had not updated Security Monkey, and didn't have time to get more involved with investigating this.  I did however disable everything related to one service that we are using very heavily (more heavily than anyone else is likely to encounter) and I am no longer seeing oom reapings.  I am closing this issue.

We experience this issue about every other day. We run security-monkey-scheduler on a c3.xlarge, colocated with some other services running as docker containers.
We analyze a lot of AWS accounts (about 20), that get fairly high usage across them all.
I'd be happy to help diagnose this, because it is most definitely affecting others. Just let me know what I can provide to be helpful.

@karlgrz : If you are having difficulty with one instance, you can have multiple instances/containers where they are dedicated to specific accounts and technologies.
We're actively working on event-driven models vs. polling, but it will be some time before that is complete.

@mikegrima good point, but that's annoying, too, heh.
I'll keep an eye on the event-driven models development.


tablet: add sentinel entry for tablet tool enum

Required by https://github.com/swaywm/sway/pull/5472; could be worked around but this seems cleanest.
This might be a breaking change if a compositor had a switch based off this enum, and is compiling with -Wall.

In general I'm not a fan of these because it forces us to add a dummy case for it in all switch statements...

(I can definitely see how this helps with the Sway PR though)

Fair enough, I've rewritten the sway side to not require this (at the expense of some nasty bits, but that was unavoidable either way).


AggregateType in RepositoryFactory

Currently:
$aggregateType = AggregateType::fromAggregateRootClass($config['aggregate_type']); see https://github.com/prooph/event-sourcing/blob/master/src/Container/Aggregate/AggregateRepositoryFactory.php#L85
Maybe it should be AggregateType::fromString($config['aggregate_type']); or maybe like this:
if (class_exists($config['aggregate_type'])){
    $aggregateType = AggregateType::fromAggregateRootClass($config['aggregate_type']);
} else {
    $aggregateType = AggregateType::fromString($config['aggregate_type']);
}

Maybe this can be considered a BC? I don't know.
Thoughts?
@codeliner @basz @lunetics

Also note this:
https://github.com/prooph/event-sourcing/blob/master/src/EventStoreIntegration/AggregateRootDecorator.php#L51-L60
Maybe there is a way to get a mapping implemented as well?

@prolic with the class_exists check it should not be a BC break I guess
Mapping is good idea. Every now and then it pops up in the chat. Everybody is looking for a way to avoid class names in the database be it message classes or aggregate types. We should think about a mapping mechanism provided by prooph that uses FQCNs only as a fallback and maybe remove FQCNs with the next majors entirely.

:+1: Will work on it this weekend.

Great idea
Am 13.06.2017 5:13 nachm. schrieb "Sascha-Oliver Prolic" <
<EMAIL_ADDRESS>
👍 Will work on it this weekend.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prooph/event-sourcing/issues/64#issuecomment-308149183,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJI-DsmwgZ6hsNnwdPb9_G1WZ5Jufm5ks5sDqb_gaJpZM4N4QCv
.


Add support to append dependencies to an existing release when using the identify command

We decided to make this the normal behavior in ProGet v2022.23+ so this parameter is no longer needed.


getInitialProps is called multiple times using Express/Fastify

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

Expected Behavior
A page's getInitialProps should only be called once per page load.
Current Behavior
Sometimes getInitialProps is called multiple times in one page load.
Steps to Reproduce (for bugs)

Minimal example app (almost identical to the next example app "custom-server-express"): https://github.com/approots/next-express-issue
To reproduce:

Run npm run dev and check the terminal console for multiple outputs of the console.log statement in the getInitialProps function of pages/index.js ("getInitialProps index").
Or click the link to "post 2", refresh the page, then load the index page again (localhost:3000/), and check the terminal and see multiple console.logs of "getInitialProps index". This also happens in production.

Context
This means that API calls are being duplicated.
Your Environment

Tech
Version

next
^5.1.0

node
8.9.4

OS
Windows

browser
Chrome

express
^4.16.3

getInitialProps is only ran once per render. The only thing that I could see is that sometimes HMR kicks in and reloads the page really fast while booting up. But that's not unexpected.
Here's the code: https://github.com/zeit/next.js/blob/canary/server/render.js#L79
Note that the correct way to test per-render is to do curl http://localhost:3000, since then there are no other factors.

@expandrew I'm pretty sure your issue is something else. Please create a new and complete issue (including example repository). Thanks.

@approots I think that your issue is in middlewares. So when I load page a, middleware for a gets called, but then this middleware server.get('*', (req, res)  also gets called because it catches all URLs.  So your page is being caught by two middlewares.
I think that returning from first middleware that matches your route pattern, will solve your issue.


Add an option to create dot model in different directions

This commit adds an optional argument to functions plot() and model_to_dot() specifying the direction of the dot object

https://github.com/deepersystems/consolenet

That's reasonable; please fix the merge conflicts and we'll merge it. Sorry for the delay.

I believe it is done!

PEP8 issue: https://travis-ci.org/fchollet/keras/jobs/219284824

Another PEP8 issue: https://travis-ci.org/fchollet/keras/jobs/219492699

Any update? In particular with respect to the failing PEP8 test? Docstring update?

@fchollet I forgot to fix the code. I will inform you when it is done, hopefully by the afternoon. Thanks for reminding me!

LGTM, thanks

@fchollet Done!

LGTM


fix: test deployemtn

What does it do?
Describe the technical changes you did.
Why is it needed?
Describe the issue you are solving.
How to test it?
Provide information about the environment and the path to verify the behaviour.
Related issue(s)/PR(s)
Let us know if this is related to any issue/pull request

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 30.22%. Comparing base (30603c0) to head (f82e230).
Report is 7 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #170   +/-   ##
=======================================
  Coverage   30.22%   30.22%
=======================================
  Files           4        4
  Lines       13967    13967
  Branches     4975     4975
=======================================
  Hits         4222     4222
  Misses       7709     7709
  Partials     2036     2036

Flag
Coverage Δ

unit
30.22% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


devilutionx: 1.4.1 -> 1.5.0

Description of changes

Things done

todo:

test
package simpleini because appears that devilutionx will support a system provided simpleini now

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Hey @aanderse ! Thank you for the PR.
Unfortunately it crashes when I start going through the menu.
So first the intro videos play. Then after a first Enter in the main menu the sounds disappears, and after a second Enter it completely freezes.
I can reproduce it on two different PCs, under i3 + xfce4, on both shareware and full GOG version of Diablo.
There is no error message, and nothing interesting if run with --verbose flag.
Can you reproduce it as well?

I get this repeated many times followed by a crash:
ERROR: Missing font kerning: fonts\24-00.bin
ERROR: Error
Failed to open file:
fonts\grayui.trn

Couldn't open /nix/store/szmaj1vv8snmlqzks30c1mn0ph1q31wa-devilutionx-1.5.0/bin/assets/fonts/grayui.trn
ERROR: Error
Failed to open file:
ui_art\diablo.pal

I thought the mpq we build contains these resources... :thinking:

@karolchmist @aanderse any updates on this PR? I was thinking about open an issue on this today...

Unfortunately I won't be able to return to this PR for at least another few weeks so please feel free to open an issue.

@icodeforyou-dot-net yes it would be great!
I tried to figure it out, but I've failed since I have zero C++ skills :disappointed:

Yeah I might create an issue today or tomorrow. I want this mission critical piece of software up and running again on the newest release to run it over Zerotier.

I've been trying to run it too, but had freezes too. However it looks like the freeze happens after a certain amount of time, and not when trying to enter some menu. If I span Enter when running the game I can get to the menu to load a singleplayer game, and then it freezes.

I get this repeated many times followed by a crash:
ERROR: Missing font kerning: fonts\24-00.bin
ERROR: Error
Failed to open file:
fonts\grayui.trn

Couldn't open /nix/store/szmaj1vv8snmlqzks30c1mn0ph1q31wa-devilutionx-1.5.0/bin/assets/fonts/grayui.trn
ERROR: Error
Failed to open file:
ui_art\diablo.pal

I thought the mpq we build contains these resources... 🤔

https://github.com/diasurgical/devilutionX/blob/master/docs/installing.md
fonts.mpq is a file provided by upstream, and is only required for certain translations. So if you don't have it it's normal to get errors related to fonts.
I've tried to compare the logs from running devilutionx --verbose on 1.4.1 and 1.5.0, but they are the same, with just the order of some lines beings changed and differents directories in the nix store being referred, which is normal. So nothing in the logs, and it looks like this is a nix issue ... I'm clueless

So devilutionx 1.5.1 was released last week, and I tried it on this branch. The progress is, now it crashes before the main menu, with an error message :tada:
VERBOSE: Unhandled SDL event: SDL_AUDIODEVICEADDED 1
ERROR: Missing file: fonts\24-00.pcx
ERROR: Error loading font: fonts\24-00.clx
[1]    50663 segmentation fault (core dumped)  /nix/store/m19m8adj3sv9sm7r2ca89znjxknwhis8-devilutionx-1.5.1/bin/devilutionx

So, I tried to test the build locally with 1.5.1 (basically updating the version and the rev and hash fields of all custom dependencies). Once in the menu, the game freezes on the second press of a button (so I can go down twice, or up twice or inside a menu entry, etc) but no error or report is displayed.
I don't know if this helps but the appimage version of the game, run with appimage-run, works and does not freeze.

oof... if someone gets motivated we should consider asking upstream for some help
any takers?


Bug Report - Latest release broke CDN Javascript File

Bug Report

What

It appears that the latest release broke the file hosted here:
https://polyfill.io/v3/polyfill.min.js?flags=always&features=Intl.~locale.en-US
Details
These errors are being reported by the console.
polyfill.min.js?flags=always&features=Intl.~locale.en-US:3 Uncaught TypeError: Setter must be a function
    at n (polyfill.min.js?flags=always&features=Intl.~locale.en-US:3)
    at Function.te (polyfill.min.js?flags=always&features=Intl.~locale.en-US:4)
    at polyfill.min.js?flags=always&features=Intl.~locale.en-US:4
    at Call (polyfill.min.js?flags=always&features=Intl.~locale.en-US:3)
    at Array.r (polyfill.min.js?flags=always&features=Intl.~locale.en-US:3)
    at B (polyfill.min.js?flags=always&features=Intl.~locale.en-US:4)
    at polyfill.min.js?flags=always&features=Intl.~locale.en-US:4
    at polyfill.min.js?flags=always&features=Intl.~locale.en-US:4
    at polyfill.min.js?flags=always&features=Intl.~locale.en-US:4
    at polyfill.min.js?flags=always&features=Intl.~locale.en-US:8

TypeError: Cannot set property 'prototype' of undefined
    at o (polyfill.min.js?flags=always&features=Intl.~locale.en-US:3)
    at Function.ce (polyfill.min.js?flags=always&features=Intl.~locale.en-US:4)
    at Se (libs?v=-BKd589K0voQel7CAPPt4XJME0uCJTvtAaQ2PdZrccA1:10)
    at c (libs?v=-BKd589K0voQel7CAPPt4XJME0uCJTvtAaQ2PdZrccA1:10)
    at Ne (libs?v=-BKd589K0voQel7CAPPt4XJME0uCJTvtAaQ2PdZrccA1:10)
    at Function.e.extend (libs?v=-BKd589K0voQel7CAPPt4XJME0uCJTvtAaQ2PdZrccA1:10)
    at Function.e.<computed> [as component] (libs?v=-BKd589K0voQel7CAPPt4XJME0uCJTvtAaQ2PdZrccA1:10)
    at project.min.65b418b4.js:1

My organization is using this file on a production system (bad idea, I know). I'm hoping reporting this may get the version rolled back while we remove the link to the CDN and host our own file.

@ibitts to fix this issue you can include gated in the request like this:
https://polyfill.io/v3/polyfill.min.js?flags=always,gated&features=Intl.~locale.en-US
It is recommended to used gated whenever using always, if you don't include gated then we will override native implementations of lots of feature such as Object.defineProperty (which is the polyfill with the bug you just found).
A fix for the bug in Object.defineProperty has been implemented over in the library --> https://github.com/Financial-Times/polyfill-library/pull/950


[SPARK-38943][PYTHON] EWM support ignore_na

What changes were proposed in this pull request?
EWM support ignore_na
Why are the changes needed?
ignore_na is supported in pandas.
after adding this param, EWM can deal with dataset containing NaN/Null.
Does this PR introduce any user-facing change?
Yes, a new param added
How was this patch tested?
added testsuites

build link: https://github.com/zhengruifeng/spark/actions/runs/2188979445
all tests passed

build link: https://github.com/zhengruifeng/spark/actions/runs/2189810762

Merged to master.

Thanks @HyukjinKwon for reviewing!


Clarify package name in pip

I see this in pypi - https://pypi.org/project/azure-iothub-device-client/
Is that the same as this project? This projects document https://pypi.org/project/azure-iothub-device-client/
points to
pip install azure-iot-device

The correct pypi link for our Device Client Library is
https://pypi.org/project/azure-iot-device/
And the correct command is pip install azure-iot-device
The correct link for IoTHub Service Library the pypi link is
https://pypi.org/project/azure-iot-hub/
And the command is pip install azure-iot-hub

@bearrito, thank you for your contribution to our open-sourced project! Please help us improve by filling out this 2-minute customer satisfaction survey


Evaluation data

Hi!
Amazing work. Is it possible to get the evaluation data to quickly compute the scores? If you could upload the (prompt, audio) as a zip, it would be a great help. Thanks.
Best,
Soujanya

Hi Soujanya,
Thanks for your interest. I have just uploaded the evaluation data (the text prompt and reference audio filename) eval_data.pkl with the relevant data loading script. The corresponding reference audio, panns embedding and vggish embedding are also uploaded to GoogleDrive. Please refer to the Github ReadMe page for more detail.
Thanks,
Yuhang

Hi Yuhang,
Many thanks. May I know which ones are the evaluation samples? If I could quikcly load them and run the evaluation script for some models, that would be awesome. Thank you!
Best,
Soujanya

HI Yuhang,
Here is our new model. If you could include results of this model in your paper, it would be great https://arxiv.org/abs/2412.21037
Thanks!


Main branch CI build failing, snyk workflow out of memory

Expected Behavior

When PRs are merged, all checks pass.
Actual Behavior

The Snyk sync workflow is failing when a PR is merged. Latest example https://github.com/backstage/backstage/blob/3e2d945826585ac0306e1c00af5bef218e24bc59/.github/workflows/ci.yml#L24
This is due to Javascript heap our of memory.
Possible fix
Maybe increase memory limit in GitHub actions workflow like this?

Reopening as its still an issue

Needs even more memory? :sweat_smile:

Are we sure this is a memory issue? It does look like a compilation issue from the stacktrace
/home/runner/work/backstage/backstage/node_modules/ts-node/src/index.ts:859
    return new TSError(diagnosticText, diagnosticCodes, diagnostics);
           ^
TSError: ⨯ Unable to compile TypeScript:
Error: scripts/snyk-github-issue-sync.ts(21,28): error TS2307: Cannot find module '../snyk.json' or its corresponding type declarations.
Error: scripts/snyk-github-issue-sync.ts(150,8): error TS7031: Binding element 'projectName' implicitly has an 'any' type.
Error: scripts/snyk-github-issue-sync.ts(150,21): error TS7031: Binding element 'displayTargetFile' implicitly has an 'any' type.
Error: scripts/snyk-github-issue-sync.ts(150,40): error TS7031: Binding element 'vulnerabilities' implicitly has an 'any' type.

    at createTSError (/home/runner/work/backstage/backstage/node_modules/ts-node/src/index.ts:85[9](https://github.com/backstage/backstage/actions/runs/3219153852/jobs/5264168236#step:7:10):12)
    at reportTSError (/home/runner/work/backstage/backstage/node_modules/ts-node/src/index.ts:863:19)
    at getOutput (/home/runner/work/backstage/backstage/node_modules/ts-node/src/index.ts:[10](https://github.com/backstage/backstage/actions/runs/3219153852/jobs/5264168236#step:7:11)77:36)
    at Object.compile (/home/runner/work/backstage/backstage/node_modules/ts-node/src/index.ts:1433:41)
    at Module.m._compile (/home/runner/work/backstage/backstage/node_modules/ts-node/src/index.ts:1617:30)
    at Module._extensions..js (internal/modules/cjs/loader.js:[11](https://github.com/backstage/backstage/actions/runs/3219153852/jobs/5264168236#step:7:12)14:10)
    at Object.require.extensions.<computed> [as .ts] (/home/runner/work/backstage/backstage/node_modules/ts-node/src/index.ts:1621:[12](https://github.com/backstage/backstage/actions/runs/3219153852/jobs/5264168236#step:7:13))
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:75:12) {
  diagnosticCodes: [ [23](https://github.com/backstage/backstage/actions/runs/3219153852/jobs/5264168236#step:7:24)07, 7031, 7031, 7031 ]
}

from https://github.com/backstage/backstage/actions/runs/3311019667/jobs/5465950950#step:7:31

I tried raising it to 6gb without issues, probably time to open an issue for the snyk workflow job

I'll open a ticket on their site for support 🙏 🤞

Update on this is that it seems like the Snyk CLI doesn't support Yarn 3 yet


Is it possible to load GLTF model ?

Thanks again for this great library.
Is AR.js can show GTLF model ? If yes which version ? There are a breaking change between GLTF 1.0 and GLTF 2.0. The loader is GLTF2Loader instead of GLTFLoader.
Another question is AR.js can directly load a STL model ?

Good point. What mesh object format can be used?

If you're using aframe, it's really easy to just include the gltf model as an asset :)

This is an example that doesn't work with gltf :
<a-scene  embedded artoolkit='sourceType: webcam; sourceWidth: 240; sourceHeight: 180;' antialias="true" outline>
                <a-gltf-model src="models/Unity2Skfb.gltf"></a-gltf-model>
 </a-scene>

I can see this message in the console.
Cannot read property 'slice' of undefined
I think i have a problem of gltf version. So my question is simple which gltf can i use 1 ou 2 ?
Which format of model is preconized for AR.js and do you know a good converter. We use CAD Assistant from openCascade which seems prettu good. But the model generated from this soft doesn't work at all. So maybe it's a problem from converter.
Thanks for your help.

Have you remembered to also include your buffer_combined.bin file =) ? Otherwise try maybe uploading your gltf so I can see what it contains? I used blender to export to gltf which worked without any issues at all, like this:
<a-assets> 	<a-asset-item id="turret" src="./assets/models/turret.gltf"></a-asset-item> </a-assets> <a-entity gltf-model="#turret" rotation="-90 0 0"></a-entity>

Hi, I was working around this afternoon, i don't have an answer but here is the a-frame's doc about gltf:

I've not try yet gltf but i'll do soon, surely using that way.
I did link you the doc for 0.6.0 and atm the basic example is using 0.5.0 (but i've changed and anything goes well (in fact better, i'll open an issue / resolve just after for the "why?"))
"The loader is GLTF2Loader instead of GLTFLoader."
It's remind me the names of Threejs loaders
(e.g. https://github.com/mrdoob/three.js/issues/10806 )
About the formats, (i'm still doing theorical answer i did not try) a-frame support:
OBJ / MTL:
https://aframe.io/docs/0.6.0/components/obj-model.html
COLLADA:
https://aframe.io/docs/0.6.0/components/collada-model.html
+GLTF...
Threejs support a bunch! can't link all just go here
On the right go far down u'll find that list of link:
BabylonLoader
ColladaLoader
GLTF2Loader
MTLLoader
OBJLoader
OBJLoader2
WWOBJLoader2
PCDLoader
PDBLoader
SVGLoader
TGALoader
and more (JSON from blender exporter is missing in the list above but this is the way i used to do, i'll do gltf soon i think, this sound good)
Hope it helps :)

If you want to use gtlf 2.0 I think you need to include aframe-extras: https://github.com/donmccurdy/aframe-extras/tree/master/src/loaders

Thanxs everybody for yours messages. It's work's very well with this code
        <a-scene  embedded artoolkit='sourceType: webcam; sourceWidth: 240; sourceHeight: 180;' antialias="true" outline>   <a-assets>     <a-asset-item id="ironman" src="./models/IronMan.gltf"></a-asset-item>   </a-assets>   <a-marker type='pattern' url='../../data/data/patt.hiro' size='1'>     <a-entity scale="0.08 0.08 0.08">       <a-entity mmd='outline: true' afterglow:2.0;>         <a-entity scale="0.03 0.03 0.03" gltf-model="#ironman"></a-entity>       </a-entity>     </a-entity>   </a-marker>   <a-entity camera></a-entity> </a-scene>


Added OnClick event for MudScrollToTop (#6517)

Description
Issue #6517
MudScrollToTop it is an invisible wrapper for all inner components is does not allow 'clicking' them. So i think that better way to handle clicking is to add event handler for MudScrollToTop;
Before
https://github.com/MudBlazor/MudBlazor/assets/25841638/d702fc1d-09a1-49c9-9276-516abfab5067
After
https://github.com/MudBlazor/MudBlazor/assets/25841638/6b5e5c00-5809-4aec-b2e6-7523f3c9bf3d
How Has This Been Tested?
Added new unit and visual tests for MudScrollToTop component.
Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Thank you! @i-n-n-k-e-e-p-e-r


Split Tests into unit tests and stability tests

The "unittesting" is taking by far to long to work effectively with the current codebase.
I suggest that the unittests are split into

unit tests (which should complete under 45? seconds at max)
stability tests (which can run a few minutes)

https://semaphoreci.com/community/tutorials/how-to-split-junit-tests-in-a-continuous-integration-environment

http://maven.apache.org/surefire/maven-surefire-plugin/examples/inclusion-exclusion.html


[AIRFLOW-1196] Template trigger_dag_id field of TriggerDagRunOperator

Add 'trigger_dag_id' field to the list of templated fields for
TriggerDagRunOperator to allow for more flexible triggering of
DAGs based on the local context.
Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1196

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Adds trigger_dag_id to the (previously empty) list of templated fields on the TriggerDagRunOperator to allow more dynamic triggering of external dags. Supercedes #2290 initiated under a deprecated github account of mine.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

test_templated_trigger_dagrun tests that a dag id passed as a DagRun param gets properly templated into the trigger_dag_id argument.
Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2663 into master will decrease coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2663      +/-   ##
==========================================
- Coverage   71.66%   71.58%   -0.09%
==========================================
  Files         153      153
  Lines       11746    11746
==========================================
- Hits         8418     8408      -10
- Misses       3328     3338      +10

Impacted Files
Coverage Δ

airflow/operators/dagrun_operator.py
96.66% <100%> (ø)
:arrow_up:

airflow/task_runner/bash_task_runner.py
93.33% <0%> (-6.67%)
:arrow_down:

airflow/utils/helpers.py
53.44% <0%> (-2.88%)
:arrow_down:

airflow/jobs.py
78.62% <0%> (-0.46%)
:arrow_down:

airflow/models.py
87.15% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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


Make terminal chat code blocks readonly again

This was a feature but people on the team are confused by it and it's inconsistent. Let's remove it since run/insert are prominent and more stable now.
cc @meganrogge

For those that were confused, this was an accessibility request if I recall correctly - so screen reader users could edit the result before actioning it

Verified this. Would it make sense to show a little hover with the text 'readonly' like it happens I believe on the left side of a diff editor

@aiday-mar could create an issue for Rob if you think so

I am not sure, perhaps it it not necessary. What do you think @roblourens ?


[Bug][Go-Git] Libgit2 has a better performance then go-git

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
I'm using the developer setup of DevLake and while collecting the data for the same project I noticed that by using libgit2 it
takes 3 minutes and with go-git, it takes ~43 minutes this is an outrageous performance penalty.
What do you expect to happen
There shouldn't be this outrageous difference. Because it slowest the process by almost 20 times.slowed
How to reproduce
Go to the .env (env.example) file and change the value of this variable USE_GO_GIT_IN_GIT_EXTRACTOR=false
Collect the data for the same project and see the time duration when you use go-git and libgit2
Anything else
No response
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

You're right, and we have to delay switching to go-git.
Since DevLake reads the repo, if go-git can be thread safe when reading, we can test then and see if we can switch to go-git.

Even though go-git may provide some features that libgit2 doesn't, I agree with @d4x1 that it's better to keep it on testing and see if the time difference can be improved in go-git.


Use Projects to manage ToDo and Workflow?

I set up a project a while back, but am not sure how issues get automatically dropped in and flow through the project.
@Septem151 , do you know how to manage this?

In my opinion, it's probably best to continue using the ToDo issue you've made in #23 and not use the Projects feature. I've never used it before so I don't know how to automate it. Up to you.

OK, going to delete the project there and then leave it alone.


Add Guest OS support note for TRIM

There are public Blogs out incorrectly calling out W2K8R2 TRIM support for OS Disks. Azure OS Product Group has confirmed TRIM support starts with W2k12.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@sebdau : Thanks for your contribution to the Azure documentation! The author, @robinsh, has been notified to review your proposed change.

Looks good to me. Thanks very much for contributing. @sebdau

#sign-off Go ahead and merge this one.


Wish for a sample of a non-fullscreen Flutter view displayed from a full SwiftUI app

Use case
Adding Flutter into an existing app, such as one built with SwiftUI, is an important part of the journey for experimenting with Flutter and using flutter to reduce code duplication across your existing apps.
Proposal
The case of embedding some shared non-fullscreen UI into your app is one case we want to show an example of, presumably through one of the samples in  https://github.com/flutter/samples/tree/main/add_to_app
The proposal here is to show some simple scrolling SwiftUI based list. And one of the views scrolling in and out of view is a Flutter widget.
A sample like this really drives home that you can put Flutter inside of your existing SwiftUI. And it would complement other samples that are full-screen embeddings of Flutter inside a SwiftUI app.

cc @hellohuanlin as we were chatting about this. No action needs to be taken right now.


change source url from goolgemap to openstreetmap

change source url from goolgemap to openstreetmap

why @qy-yang ? this project name immediately points to google maps, and the code generation is tightly related to the gmaps APIs
you cany just change the URL and hope it works :( maybe that'd be a different project?

Will close since this is outside the scope of this Google-Maps-based project.

P.S. @qy-yang, there is another project, folium, that does Python visualization if you're looking for a library built on OpenStreetMap (and don't need Google-Maps-specific functionality).


EM() as the default solver?

Why do we use EM() as the default solver? From the example of StochasticDiffEq.jl docs this method seems to be prone to errors. The same can be said for Euler() as the deterministic solver.

Indeed Euler-Maruyama can be inadequate especially for large step sizes, but it is still so commonly used that one could argue it is still a reasonable default solver. Users would potentially be less familiar with the other SDE solvers. Which one would you use?
For ODEs I argee that one could replace Euler by something like RK4() or DP5().

DP5() is dangerous as it isn't useful for damped systems. I would use the default solvers that SciML uses which would be Tsit5() and SRIW1() I think.

Hmm, Tsit5() caused some problems when used with DynamicalSystems.jl (see this issue). But I agree that it's best to go with established defaults if we don't stick with the most basic defaults.


nerdctl build should accept --output <DIR> as an alias of --output type=local,dest=<DIR>

Description
nerdctl build should accept --output <DIR> as an alias of --output type=local,dest=<DIR>, as in docker build.
Steps to reproduce the issue
nerdctl build --output . .
Describe the results you received and expected
Received: error: invalid value .
Expected: Should work as --output type=local,dest=$(pwd)
What version of nerdctl are you using?
nerdctl version 0.23.0-26-g67c0f59
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
No response
Host information
No response

I think this can be a good first issue?  cc @AkihiroSuda

Assign me plz


bump async_zip to latest

This PR handles the upgrading of async_zip to the latest version. They did some work around supporting multiple async runtimes which we really don't benefit from here because we are just using tokio.
Anyway, this gets us unstuck from async_zip 0.7

pinging @silverjam because he 👍🏻  the original dependabot commit.
note for semver, this did change some type signatures for functions, but can be considered a patch release for esthri because none of those functions are public.


feature/array exercises and tests

There is an array exercise and 2 tests for each example the lesson/issue covers. Instead of using a and b or array1 and array2 for a few of the examples, I decided to get a little more creative for the argument names.
In addition, I saw two small typos in the README files, so I went ahead and fixed them.
Closes issue #10

These exercises look great @rlmoser99, very nice work 🏆


Second wave of speaker invitations for FTC SF OSS day

https://docs.google.com/document/d/1RXZL26hUzb6EtRDLb_YiGX85yUNmZh5wYYDkkFuR9uU/edit

Sent out new invitations yesterday. Going to close this and file other issues for followup work


SW-1537 Schema changes for v2 quantity tracking

Update the data model in preparation for changes to how we track seed quantities:

The accession_quantity_history table will, as its name suggests, hold a record
of changes to the remaining quantities of accessions, whether the new quantities
are observed (entered by the user) or computed.
The accessions table has columns to hold the latest observed quantity.
The withdrawals table has columns to hold the estimated seed count and weight,
if they can be computed from the user-specified withdrawal quantity.

This does not include any code changes to read or write the new table and columns,
nor to migrate existing v1-style data; those will come separately.

Current dependencies on/for this PR:

main

PR #586   👈

PR #587

PR #588

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: 🔄 Waiting for checks to pass
This pull request is currently waiting for CI and mergeability checks to pass before being merged.
This comment was auto-generated by Graphite.

Job Reference: QpIP1QCR1Nvluk7Y6jyS


Page title and breadcrumb not showing up

I have pulled the latest code from https://github.com/akveo/ng2-admin and tried run locally. everything is working fine except page title and breadcrumb. They are not showing and data as it's previous version.
Can anybody help me?

I'm suffering this issue too.
My temporary work-around is calling this.appState.notifyDataChanged('menu.activeLink', { title: this.title }); in the ngOnInit() of my pages for when they are switched to.

@ami0011 Thanks for reportings, fixed now.


Saml missing NameIDFormat

Closes #1499

:+1: with a comment on constant selection.

I'm going off the behavior specified here: http://saml2int.org/profile/current/#section92
it says:
In the absence of a ... <md:NameIDFormat> element in the Service Provider's metadata, the Format of
the <saml2:NameID> SHOULD be set to urn:oasis:names:tc:SAML:2.0:nameid-format:transient.

Excellent. Thanks for the reference. Is there a const available in opensaml we could use? Otherwise looks great

found the constant, going to wait for travis to finish then merge.

Just thought of something; I realize we must stick to the spec and that's perfectly fine, but is there a use case/need to externalize the default nameid format into a property, that is set to the spec expected value? I am thinking of cases where metadata is not able to provide this setting for whatever reason, and yet the SP sort of expects it. That is a bad implementation, but we could technically allow for that kind of thing.
Not sure if it's any use a all yet, but in general, I prefer things to be configurable when the effort is negligent.

If we do support that I think it should be defined per service and not globally for the entire server.
There is already a workaround for that use case by duplicating the service metadata and adding in the corresponding NameIDType to a local copy of the metadata.

That's fine with me. I think the registry model is flexible enough now to account for options settings.
Shall we log this as an issue for now and see how use cases pan out?

Because it's a deviation from spec I don't think we necessarily need to log it as an issue yet, I would wait for the actual use case to present itself through usage.

Sounds good.


Use requests.Session

In pipreqs project, it using this library.
Currently, yarg creates new HTTPS connection every time, and pipreqs queries many times to pypi so it consumes many time.
I fixed to use requests.Session and reusing connection reduces running time to about half.

Coverage increased (+0.05%) to 96.59% when pulling cf67924fd477dc384654ea91a677609d7d8cb06b on littmus:master into 87fc80ebe841e60699d80ac5779c66f4e8404149 on kura:master.


Cross-platform support for the MutationObserver API

Specific Demand
There is a web API called MutationObserver that provides this. As a stopgap, I can use this directly via web-sys, but longer-term it would be much nicer if Dioxus provided this sort of API across all platforms it supports.

Hmm.. if I understand correctly, the mutation observer is for tracking changes in the DOM. What's your use case for this?
As far as I can tell, there is little need for observing mutations in a declarative framework such as Dioxus, where the framework controls the entire DOM.

Yeah, I added this simply because I was reading about it and the IntersectionObserver together, but on reflection I'm not sure what use this would be either. I'll close this.


fix(google): fix few logging resources rfunc call

I don't know if my changes make sense or not. I faced this issue while migrating google resources.


Support crossorigin="anonymous"

Scripts loaded using little-loader can't be debugged using window.onerror because of CORS (you just get the standard 'Script error.' error). It would be nice to be able to set the crossorigin policy on the <script /> tag so that we can debug scripts that are hosted on a third party domain (such as a CDN)

Supported via generic setup(script) callback.


Require MySQL 8.0, Postgres 12, MSSQL 2012

MySQL 5.7 support and testing is dropped
MySQL tests now execute only against 8.1, previously it was 5.7 and 8.0
Postgres 10 and 11 support ist dropped
Postgres tests now execute against 16
MSSQL 2008 support is dropped
MSSQL tests now run against locked 2022 version

Fixes: https://github.com/go-gitea/gitea/issues/25657
Ref: https://endoflife.date/mysql
Ref: https://endoflife.date/postgresql
Ref: https://endoflife.date/mssqlserver
:warning: BREAKING :warning:
Support for MySQL 5.7, Postgres 10 and 11, and MSSQL 2008 is dropped. You are encouraged to upgrade to supported versions.

After this, the required checks config on github needs to be updated from test-mysql5 and test-mysql8 to just test-mysql. The PR will also not go green without this change.

I mean, we could also be so nice and block this PR until the mentioned versions are actually EOL.
But I'm also fine with the current approach.

I don't think that people who use MySQL 5.7 also use Gitea nightly in production 😆
I think it's fine, the release will definitely happen after all of the mentioned versions are EOL.

By the time the 1.22 branch releases they will be EOL. I imagine anyone running unstable builds will likely not run unsupported databases :)

@silverwind ✨ same though at the same moment ✨

I don't think that people who use MySQL 5.7 also use Gitea nightly in production 😆

Good point.

Thanks for keeping this current 😀
Could you add a note that should someone need support for an outdated database that they can contact us for an "extended support contract" (or some other name?). As then we can then test for those older databases and add fixes into this codebase too. I know there are some people who are using the outdated mssql version that are paying MS for them to backport fixes, so they may need to have that support in Gitea too.

Feel free to edit the OP to your liking :)

Yes, if we say it required mysql 8.0, we should test the minial version 8.0 but not 8.1 like we did with Golang.

@techknowlogick

This PR doesn't change the behaviour of what we are doing with tests (w/ psql we're using latest, and w/ mqsl we were using 8.x).
I agree we should test both min/max db versions, but that can be done in a different PR.

Testing the minimal version is good enough and the right approach, like before.
But I guess nobody would really follow it if this gets merged, until some bugs like #22809 occurs (pgsql 10 works but pgsql 9 doesn't -- newer version works, older version fails)
This is my last comment for this problem, because I have explained enough and I do not want to bother people.

ref: #27429


Encoding from existing JPG to JPG significantly increases file size

I've just noticed that using the encode method increases the file size significantly if you try to encode a JPG to JPG.
Example:
$image = $this->image->make($originalJpgImage);
$imageJpgEncoded = (string)$image->encode('jpg',100);

Results with GD:

Original: 42 KB
Encoded: 112 KB

Results with Imagick:

Original: 42 KB
Encoded: 61 KB

Setting the quality 80 gets me decently close to the original file size, but I fail to understand what is causing such an increase. Could someone elaborate on why this is happening? And is this normal behavior?

The quality parameter functions as an absolute value for the compression quality/image size and is not based on the existing compression/quality and is also not related to the original value.
If you encode with 100 as the quality value as shown in the example, you will always achieve the maximum possible quality value (and the associated file size), even if the input file was saved with a quality value of 0.