Incompatibility with file-icons plugin

Hi!
I'm using file-icons plugin along with foldername-tabs and I have an issue when a file name is too long.
With file-icons only:

With foldername-tabs only:

With both plugins:

As you can see, when I use both plugins at the same time, the tab name disappears as if both the icon and the text couldn't fit inside the tab.
I'm aware there can be a lot of incompatibilities like this one with all the plugins available on Atom and you can't handle all of them but if there was a way to fix this one, it would be great :)

I'm also seeing this bug, it looks like the title is being pushed to the next line below the icon but is not visible.

I can't find a good css solution to rectify this, but here is a workaround that works:
Add the following to your Atom stylesheet (Edit > Stylesheet...):
/* fix foldername-tabs package incompatibility - https://github.com/paulpflug/foldername-tabs/issues/15 */
.foldername-tabs {
  width: 0%;
}

I fix it like this :
.foldername-tabs {
  width: 1rem;
}

I yolo'ed:
.tab-bar .tab .title {
    display: flex;
    align-items: center;
}

div.foldername-tabs {
    top: 0; /* Adjust for theme */
    width: 100%;
    text-align:left; /* Optional */
}

div.foldername-tabs > span.folder {
    opacity: 0.6;
}

with pleasing results in One Dark theme:


Deleted bot remains in recents and some files still on disk

Describe the bug
When deleting a bot, it remains in the recents list until Composer is restarted.  Additionally, the sln and directory still remain on disk.
Version
Version:  1.4.0-nightly.235590.f4de79
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0
Browser

[x] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Create a bot
Delete it via Project Settings
Note it still appears in Recents
Check disk for remain files.

Expected behavior
Should be deleted in entirety and recents refreshed.

Closing this as a dupe of #6808


doesnt start

Hello.
I built with Visual Studio 2022 and exes show me

When I run under debugger I see this

When I press "Skip" and look at call stack I see

I.e. where is no elements with seted index.
I change
https://github.com/yuphin/Lumen/blob/0fbc12756a3118bce1dae47991635727eec6cf8d/src/Framework/VulkanBase.cpp#L22
to
std::vector<VkFence> _in_flight_fences(MAX_FRAMES_IN_FLIGHT + MAX_FRAMES_IN_FLIGHT);
and restart I have

and call stack is

What does the validation message say?

now works ok


Add metadata columns to the RDBMS contrib

The goal of this feature is to allow metadata column to exists for
specific tables created for Redshift.
Given the scenario where we would always have to have a created_tz
column at the end of every table generated by that contrib we could do
the following:
class UpdateTableTask(redshift.S3CopyToTable):

    def metadata_columns(self):
        return [('created_tz', 'TIMESTAMP')]

    def metadata_queries(self):
        query =  'UPDATE {0} ' \
                 'SET created_tz = CURRENT_TIMESTAMP ' \
                 'WHERE created_tz IS NULL'.format(self.table)
        return [query]

Adding a layer of abstraction over this feature, you could easily
add many default behavior for specific tables for versioning the table
and more.
This feature is opt-in by default since we don't want this break other
people's pipeline after integrating this.
Do we think that this would be a feature that you guys would be interested in merging? I'm unsure if this becomes too specific for our use-case, but it should help some other people to have that type of behavior if need be.
If that's something that you guys are interested in merging, then I'll spend some time in writing a proper testing suite!
Let me know!

This seems like a cool addition; however, i'm not completely sold on it yet. It really feels like a specific wrapper around post_copy, but it's executed along side it.

You would be right to say that it's a wrapper around post_copy because in our original implementation, we were using post_copy instead of post_copy_metacolumns.
I would agree that they are pretty much the same but we use post_copy as task-specifics post copy queries. For instance, we happen to use post_copy to deduplicate possible duplicated output.
Alongside this, we also want to be able to run generic updates. For instance, created_tz timestamp. It made sense to decouple them, in our mind, to differentiate task-specific post_copy and generic updates to table with post_copy_metacolumns.
I've created a gist that may better demonstrate how we use it: https://gist.github.com/cabouffard/73cfc798bef9ec09fc91807b99a6c5a9
That said, having this separated in two different methods, allow us to not have to override the content of post_copy  if we want to add generic metadata_columns behavior.

The end goal you're trying to achieve is to easily modify a set series of metadata columns after performing a Redshift Copy. This feels like a finer-grained version of post_copy.

Totally agreed with this statement!

I feel like I haven't sold on your on this one, is that the case @dlstadther ? :)

Hey @cabouffard
After thinking on this all morning, I think my hesitations here are more-so S3CopyToTable structural issues than feature addition problems.
So, let's continue with getting your metadata feature pushed through. I believe I would like like to see the Mixin moved to rdbms (assuming it's also applicable to Postgres (which i believe is the case)).
Can you also mesh some of your gist together in a docstring for the Mixin? I believe another hesitation I had (prior to your gist) was lack of implementation clarity.
I very much appreciate all your recent PRs! Sorry for the delay addressing them to completion!

@dlstadther No worries mate!
I've added a whole bunch of text, some extra bit of documentation and changed some code-part with the suggestions you've provided.
Is there anything else that you would like for me to do? I'm happy to hear any suggestions!
Thanks! :)

@dlstadther Hey, what's the latest on this one? :)

Sorry for my delayed response here @cabouffard . I pretty sure i'm all good here, but let me review real quick once more this afternoon :)

@dlstadther Hey ya, we're all green and I've added your suggestions / fixed typos! :)


Check for increase in alpha/beta diversity from baseline

Alpha Diversity:
We should add a hypothesis (or potentially a parameter in the baseline hypothesis that already exists) to check for an increase in alpha diversity.
This could also be housed (as a parameter to check for increase) under a check for the difference in alpha diversity at a single time point (week vs. control).
Beta Diversity:
Is distance to donor significantly lower (not just different) than it was at baseline (week 0, i.e. pre-FMT)?
wilcoxon_srt with baseline_group = 0, and alternative of greater/lesser depending on ordering (should be documented very explicitly - thanks @ebolyen).

Yeah! We should do this with an alternative_hypothesis parameter, which can be two-sided, greater, or lesser.

Solved by #19


Fail compilation if the JIT code contains a FAST_FAIL helper

this is a much more conservative approach than current.
Mark the function as "un-JITable" if it contains a FAST_FAIL helper, which is emitted by the .NET JIT in certain situations.
Will fix a number of crashes.
Also re-enable OPT-2 for all scenarios

This effectively makes n-body and many of the other benchmarks not compile.


Support Mac

Some people, like Eileen and Jonas (Linkedin) have requested a mac version

now available https://quba-viewer.org/quba_viewer-0.1.0.dmg


Introduction of cemerick/url as project dependency breaks speclj

Hi,
I have a project with existing speclj tests to which I just tried to add cemerick/url as a dependency. The tests still pass when I run them using lein spec but when I try to run them from within my editor, I get the dreaded "ClassNotFoundException". Seems the class that cannot be found is    speclj.platform.SpecFailure.
Any ideas why this might be? Googling this revealed something that seemed superficially similar but we are on version 3.3 and it looks like that got fixed already.

Seems that upgrading to 3.3.2 (from 3.3.0) fixed this issue for me. Hope google indexing this will save someone some time one day :-)


Chart support for custom tolerations and affinity

Fixes #1181

Thanks @octplane !

My pleasure!

Would it be possible to do a minor release of the Chart with this fix or is this too much work for you?
Thanks!

I have cut 7.4.0 https://github.com/appscode/voyager/releases/tag/7.4.0

Thanks!


Error: invalid DSN: invalid DSN port: strconv.Atoi: parsing "": invalid syntax

This might be related to this closed issue: Failed to connect: HTTP response 500
I am getting the error
Error: invalid DSN: invalid DSN port: strconv.Atoi: parsing "": invalid syntax

I am using the below DSN format to connect to databricks via go.
<EMAIL_ADDRESS>
and then I am passing the DSN to the below function.
sql.Open("databricks", dsn)

One observation, things were working fine when I was using
github.com/databricks/databricks-sql-go v0.1.4
github.com/apache/thrift v0.12.0 // indirect

thrift version depends on databricks-sql-go, so there is definitely something wrong with the latest version of databricks-sql-go
I am sure this is something caused after I did an upgrade to github.com/databricks/databricks-sql-go v0.2.0

Closing this issue. Seems to be DSN string format issue.


Add Qiskit native QPY ParameterExpression serialization

Summary
With the release of symengine 0.13.0 we discovered a version dependence on the payload format used for serializing symengine expressions. This was worked around in #13251 but this is not a sustainable solution and only works for symengine 0.11.0 and 0.13.0 (there was no 0.12.0). While there was always the option to use sympy to serialize the underlying symbolic expression (there is a use_symengine flag on qpy.dumps you can set to False to do this) the sympy serialzation has several tradeoffs most importantly is much higher runtime overhead. To solve the issue moving forward a qiskit native representation of the parameter expression object is necessary for serialization.
This commit bumps the QPY format version to 13 and adds a new serialization format for ParameterExpression objects. This new format is a serialization of the API calls made to ParameterExpression that resulted in the creation of the underlying object. To facilitate this the ParameterExpression class is expanded to store an internal "replay" record of the API calls used to construct the ParameterExpression object. This internal list is what gets serialized by QPY and then on deserialization the "replay" is replayed to reconstruct the expression object. This is a different approach to the previous QPY representations of the ParameterExpression objects which instead represented the internal state stored in the ParameterExpression object with the symbolic expression from symengine (or a sympy copy of the expression). Doing this directly in Qiskit isn't viable though because symengine's internal expression tree is not exposed to Python directly. There isn't any method (private or public) to walk the expression tree to construct a serialization format based off of it. Converting symengine to a sympy expression and then using sympy's API to walk the expression tree is a possibility but that would tie us to sympy which would be problematic for #13267 and #13131, have significant runtime overhead, and it would be just easier to rely on sympy's native serialization tools.
The tradeoff with this approach is that it does increase the memory overhead of the ParameterExpression class because for each element in the expression we have to store a record of it. Depending on the depth of the expression tree this also could be a lot larger than symengine's internal representation as we store the raw api calls made to create the ParameterExpression but symengine is likely simplifying it's internal representation as it builds it out. But I personally think this tradeoff is worthwhile as it ties the serialization format to the Qiskit objects and APIs instead of relying on a 3rd party library. This also gives us the flexibility of changing the internal symbolic expression library internally in the future if we decide to stop using symengine at any point.
Details and comments
Fixes #13252
TODO:

[x] Add QPY v13 format documentation
[x] Add release notes
[x] Expand test coverage. We need to ensure we have thorough coverage of all supported expression operations which we weren't previously exercising through QPY.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

271 of 295 (91.86%)  changed or added relevant lines in 5 files are covered.
5 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.006%) to 88.658%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/qpy/binary_io/value.py
184
208
88.46%

Files with Coverage Reduction
New Missed Lines
%

crates/accelerate/src/unitary_synthesis.rs
1
92.24%

qiskit/qpy/binary_io/value.py
2
87.14%

crates/qasm2/src/lex.rs
2
93.48%

Totals

Change from base Build<PHONE_NUMBER>2:
0.006%

Covered Lines:
74822

Relevant Lines:
84394

💛  - Coveralls

The state of this is it's stuck on the overly complex expression example test I added in: https://github.com/Qiskit/qiskit/pull/13356/commits/2b352d705ab30f2106f5668ec07a3fbe0bbfcd41 I'm thinking it might be worth approaching the deserialization in a slightly different approach instead of doing nested expressions recursively like it currently is. If anyone has bandwidth and can come up with a fix before me for that test, feel free to just push to the branch.

Ok, I fixed the issue from last week with: https://github.com/Qiskit/qiskit/pull/13356/commits/a903387f435294cc5a85a2e39d119a83e4864578 which switches to a stack based approach instead of a recursive one. I think this should be ready now.

Other than that, I was wondering if we should also communicate somehow that use_symengine isn't useful anymore for QPY versions over 13?

I added a sentence to the qpy.dump function's docstring for the use_symengine argument to say it doesn't do anything in qpy 13. Were you thinking we needed an upgrade release note or something to documente it further?
We can drop the argument in 2.0 potentially since 2.0 will have a minimum compatibility version of qpy format version 13 so nothing >=2.0 emits can support the use_symengine flag. But my worry with that is it is a potentially breaking api change for users that might want to support dumping qpy across code run with multiple qiskit versions. So maybe it's better to just document it as not doing anything in 2.0 and remove it in 3.0. Either way that's something we can debate post 1.3, since we probably should not deprecate it here, and only save that for a future release.


feat: adds plan id to apply notifications

Makes changes to how apply notifications are called in context.py to include plan_id. Also modifies the Airflow plan_request_id to be the same as plan_id so that apply notifications can include links to the Airflow job. All of this will help our users find logs when they want to see what SQL queries SQL Mesh runs or track down an error.
Currently when a SQL Mesh apply fails with Airflow acting as scheduler, the stack trace sent to the notification integration doesn't include any information about why the apply failed. Getting the actual stack trace out of airflow and into the notification would be very complicated, so a good work around is just to include a link to the DAG so users can quickly explore the logs for their run.

Looks like the test cases for notify() and notify_user() need to be updated or the logic needs to be changed to still be backwards compatible.


Package proposal: blobrunner.vm

Package Name
blobrunner
Tool Name
blobrunner
Package type
ZIP_EXE
Is the tool a console application?
false
Tool's version number
0.0.5
Category
Utilities
Tool's authors
OALabs
Tool's description
BlobRunner is a simple tool to quickly debug shellcode extracted during malware analysis.
Download URL
https://github.com/OALabs/BlobRunner/releases/download/v0.0.5/blobrunner.zip
Download SHA256 Hash
369ed39086e40fe9ae5404b52cafe0a9b747abb11f2d33d73e5a51097d0ae2a4
Dependencies
No response
Why is this tool a good addition?
This quote from BlobRunner's README says it all:

BlobRunner is a simple tool to quickly debug shellcode extracted during malware analysis. BlobRunner allocates memory for the target file and jumps to the base (or offset) of the allocated memory. This allows an analyst to quickly debug into extracted artifacts with minimal overhead and effort.

See #826 for 64-bit version.


writeRefAxis is midly broken

Description
DVGeometry.writeRefAxes appears to have broken midly due to a change in OrderedDict(). It should be reg tested but currently is not
Steps to reproduce issue

Call writeRefAxes in the normal manner

Current behavior
Raises the following error:
Traceback (most recent call last):
  File "runFFDExample.py", line 50, in <module>
    DVGeo.writeRefAxes('local_refaxes.dat')
  File "/home/ben/packages/pygeo/pygeo/DVGeometry.py", line 2066, in writeRefAxes
    print(self.points.keys()[0])
TypeError: 'odict_keys' object is not subscriptable

Expected behavior
Should not do that

Just needs to be cast to a list, I'll take care of it in my docs push


Create a named rare-spawn for each biome.

Need to generate a list of random names with mobs (that will actually spawn per biome) and make rare-spawns with them. Could make a rare spawn item per mob with some kind of name "Sally's Secret Crystal" or something. If only the name is changed the item can still function as a normal item, but would have the NBT that could be used for quests.


ARROW-5376: [C++] Workaround for gcc 5.4.0 bug

@kszucs Can you try to build this on the buildbots?

@pitrou it automatically gets triggered


Proposal: support return values in the draft function

https://github.com/reduxjs/redux-toolkit/pull/3074

If support return values in the draft function, this would mean that Mutative would need to determine if the value is a draft, and do a deep traversal of non-draft return values, and such an unpredictable return value would waste a considerable amount of performance.
But the community wants mutative to support it. The good thing is that Mutative makes a lot of performance improvements, and it doesn't lose any performance as long as we suggest trying to return the draft itself as much as possible in usage scenarios like Redux's reducer. Even if the returned value is not a draft, it doesn't have significant performance loss and we will have performance benchmarks to track it.

Suggestion from https://github.com/reduxjs/redux-toolkit/pull/3074#issuecomment-1379334527

Since the user may refer to the draft in the return of a completely new object, I will add a new option strictReturn , which we will suggest the user enable strictReturn in development mode and disable strictReturn in production mode. This will ensure safe returns and also keep good performance in the production build.


Consider making the license GPL v3

Is your feature request related to a problem? Please describe.
Right now, Jagex or any private server are legally allowed to use the RuneLite HD plugin without:

Giving credit
Making any modifications open source

In addition, spin-off OSRS HD plugins can be closed source and legally sold in bad faith.
The HD plugin is one of the best things to happen to OSRS. The GPL v3 copyleft license helps preserve the plugin from unsavoury actors.
Describe the solution you'd like

Move the license to GPL v3

TL;DR differences:
Same between GPL and BSD:

Commercial use allowed
Modification allowed
Distribution allowed
Private use allowed
No Liability
No Warranty

Additional in GPL:

Forks must State changes
Forks must Disclose source
Forks must use Same license

Describe alternatives you've considered

Move license to any other copyleft license

OSRS HD has already shown it's superiority as a product, and as an allowed HD plugin already has the largest marketshare. Most arguments for using BSD to gain corporate adoption are lack luster for this reason, meaning OSRS HD doesn't have anything to lose for going GPL, only things to gain.

I don't think this is going to happen. It would be far too complex to change the license at this point and would lead to a mixed-license model which is just plain confusing.


[BUG] Polyfill created by fetch tracking interferes with maxmind device tracking

When app insights automatic fetch tracking is enabled, maxmind's device tracking no longer works. Is there a way to manually trace fetch dependencies back to the server? Can I manually set the 'traceparent' on the fetch request? If so where do I get the 'traceparent' from?
Steps to Reproduce

OS/Browser: Win11 /Chrome (129)
SDK Version [e.g. 22]: 3.3.3
How you initialized the SDK: const insights: InsightsClient = new ApplicationInsights({
   config: {
     connectionString: connectionString,
     disableFetchTracking: false,
     disableExceptionTracking: false,
     enableUnhandledPromiseRejectionTracking: true,
     enableCorsCorrelation: true,
     enableRequestHeaderTracking: true,
     enableResponseHeaderTracking: true,
     enableAjaxErrorStatusText: true,
     enableAutoRouteTracking: true,
     autoTrackPageVisitTime: true,
     autoExceptionInstrumented: true,
     distributedTracingMode: DistributedTracingModes.W3C,
     ...config,
   },
 });

Expected behavior
app insights SDK should not interfere with device fingerprinting libraries
Additional context
Error produced, but only if 'disableFetchTracking' is not true: "has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled."

Ok, so it sounds like this maxminds (device fingerprinting) is adding some additional header to validate that the content of the full request has not changed (since sending).
Assuming this is the case the issue is (most likely) that we are adding either the traceparent and/or Request-Id header to the request in response to the distributedTracingMode, we don't have an explicit "None" option for this, however, looking at the code it is currently an "explicit" check for matching the 3 supported values (so in theory a hack to this (assuming we don't explicit reset it somewhere else), would be to set this value to something invalid (like -1 or 99, 1000, etc), and this (should) stop the header being set but still report that the "dependency" occurred.
However, to answer

Can I manually set the 'traceparent' on the fetch request

Yes, you can set the header to anything you like and we will not override it. As long as you keep the same traceid and create your own spanid for the request and set the header directly it should work.

Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource.

Are you sending the request via a proxy that is not returning the correct OPTIONS response? As the Asure Monitor (Application Insights) server should be handling (returning) the correct response based on the request sent. It may be that the request is having some additional headers added to the outbound "telemetry send" request (the sending of the events to Azure Monitor) and the fingerprinting is adding some additional headers which the Azure Monitor (Application Insights) does not accept -- you will need to configure the fingerprinting mechanism to NOT add the additional headers to these requests.
Alternatives.

Can you "initialize" our SDK after the maxmind has patched the fetch request, so that any of our changes will be fingerprinted as part of the request

@MSNev Ok, so in your 2nd paragraph, your hack is to basically disable distributed tracing by setting it to a non-"DistributedTracingModes" number?

Yes, you can set the header to anything you like and we will not override it. As long as you keep the same traceid and create your own spanid for the request and set the header directly it should work.

Are there any examples of this?

Workaround for now:
Create another fetch which copies a "globalFetch" variable from window.fetch as soon as the app starts, then create a custom polyfill at the top of the stack which  checks if the request is for maxmind's API, if so strip any headers that are attached by app insights.

Ok, now I'm understanding a little more there is a MUCH easier way (which needs more documentation)
We have two possible configuration items that help with this correlationHeaderExcludedDomains (string[]) and correlationHeaderExcludePatterns (RegEx[]) configs
The actual implementation that checks for these configurations (and a few others) is located here https://github.com/microsoft/ApplicationInsights-JS/blob/ed8632ae113cc7d013be825af05e1204b74a895f/shared/AppInsightsCommon/src/Util.ts#L37-L96
You can simply use this to tell us to not add the correlation headers (traceparent and / or Request-Id) to these specific domains, my previous comments where (limited (not sure why I didn't remember these at the time) to just always disabling)

@MSNev Ok, so in your 2nd paragraph, your hack is to basically disable distributed tracing by setting it to a non-"DistributedTracingModes" number?

Yes that is what I proposed, but there are no examples -- because it's not a "supported" sequence as someone (including myself) may (or may not) change that undocumented feature at some point in the future.

Ahh, this works much more simply - thanks
  correlationHeaderDomains: [
    "https://d-ipv6.mmapiws.com"
  ]


Fix the Injector creation in HadoopTask

Description
As a part of the Guice refactoring, the injectors are built differently, having the ability to have or not have modules required for a class. HadoopTask uses the injector to create an instance of ExtensionsLoader which contains the ExtensionsConfig class. However, the modules in the injector are insufficient to supply the properties passed to the druid server to the ExtensionsConfig (POJO), due to which irrespective of the Druid's config, it will get instantiated with the default properties.
This PR fixes the injector so that the ExtensionsConfig is instantiated properly.
Testing
To be added

Key changed/added classes in this PR

HadoopTask

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

@LakshSingla, thanks for finding and fixing this regression. LGTM (non-binding).

Thanks for the fix!
The line you're adjusting is a regression in 24.0.0. From looking at the code, it seems that the impact here is the index_hadoop task won't properly realize which extensions are configured (since it isn't reading the server properties files) and therefore won't ship those jars to the Hadoop cluster. I think it would cause problems for jobs that use extensions such as sketches. That's a big enough issue that we should do a 24.0.1 release. So, I created a milestone and added this to it.
About testing: how did test your fix here, and do you know why this wasn't caught as part of the 24.0.0 release testing? We'll want to make sure to test this case for 24.0.1 and future releases, ideally with an automated test.

@gianm, thanks for your analysis. You raise an important issue around testing. It appears we have no tests for the case of the Hadoop task with extensions and that is concerning. It means I was able to break this area with no feedback that something was amiss.
The workaround testing done here is to borrow some code from StatusResource and log the set of loaded extensions. Since we have no IT that runs this task, testing was done ad-hoc by hand on a local machine. Absolutely not ideal. But, perhaps good enough considering that extension loading is binary: they are either loaded or not.
Moving forward, we do have an IT for this task which you identified: ITHdfsToHdfsParallelIndexTest. Due to memory limitations on Travis, that is not run as part of the Travis build. It is run elsewhere. But, that test does not also ensure that extensions are loaded.
There are other HDFS-related tests in the HDFS_DEEP_STORAGE IT test group. None of them are run.
We need to find a way to run these Hadoop tasks so that we get code coverage. We're discussing moving the build to a new build system. Perhaps that system can provide sufficient resources that we can run the HDFS (and Hadoop) containers in our ITs and so we can test this area.

Regarding:

About testing: how did test your fix here

Manual testing has been done to identify the cause of the original problem: submitted ingestion jobs weren't picking up the location of the custom location where druid deps were present. Testing on a local setup revealed that the ExtensionsConfig wasn't getting populated correctly irrespective of whatever was supplied in the config file. After discussion with @paul-rogers, it seemed like that the injector was at fault, and didn't contain the properties passed in the config to properly instantiate the ExtensionsConfig.
Important note, the fix has been verified through manual logs/debug points, and the fix hasn't been verified on the original failing ingestion job yet.

Went ahead and added a log line containing the ExtensionsConfig in the constructor of the task post the approvals.

Went ahead and added a log line containing the ExtensionsConfig in the constructor of the task post the approvals.

Please don't add it there: tasks are serialized and deserialized at various times where they aren't actually run, so we don't want the constructor to log anything. Log messages should go to the run method.

@gianm, thanks for that info. As it turns out, the code that primes the injector, and preps the extensions is static: it will get loaded in any process that links to this code. Should this stuff be done in run() instead of static?

Cool. So we can leave well-enough alone for now. We can fix the static injectors later, but only after we ensure we have tests in this area so we don't break things a second time.

Cool. So we can leave well-enough alone for now. We can fix the static injectors later, but only after we ensure we have tests in this area so we don't break things a second time.

That sounds like a good approach to me.

Build failing on the code coverage. IMO it shouldn't be required because the added method only adds a log line. Any workarounds here?

I think It's ok. I am going to merge this PR since you have verified the change manually.


[Workspace Feeds] Export Row Not Updating on Card Details Page After Selection

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @joekaufmanexpensify
Slack conversation (hyperlinked to channel name): #quality
Action Performed:
Prerequisite:
Members and company cards added in workspace

Go to staging.new.expensify.com
Choose the workspace from settings
Click company cards >Card details
Click Quickbooks Online Credit card export > Expensify card liability account

Expected Result:
Selected option updated in export row
Actual Result:
Selected option not updated in export row
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence
https://github.com/user-attachments/assets/8ed3e31a-210c-4c34-be23-ab30bb3d4352

View all open jobs on GitHub

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

BZ team member, Callstack will work on this issue, please wait for them to comment before moving the issue forward.

Hi, I'm Nicolay from Callstack - expert contributor group - and I would like to work on this issue.

Added to the project

@allgandalf I already reviewed the PR, let me post checklist

BugZero Checklist:

[x] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[x] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production
[x] 2b. Reported on staging (deploy blocker)
[ ] 2c. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[x] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment: I think we don't need to comment on the previous PR because @narefyev91 also be author of this feature

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[x] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Prerequisite
Members and company cards added in workspace
Test:

Choose the workspace from settings
Click company cards >Card details
Click Quickbooks Online Credit card export
Select an option and verify that the new value is updated

Do we agree 👍 or 👎

@stephanieelliott Could you help to assign me to this issue? because I already reviewed the PR


ModuleNotFoundError: No module named 'swag.utils'

Hi,
Thanks for releasing the code. I was trying to run the training script train.py but got the error ModuleNotFoundError: No module named 'swag.utils'. This is in the file experiments/eigenvalues/train.py line 13 - import swag.utils as training_utils. I could not find swag in your codebase or in the setup.py, and after installing a package swag, I got that error above.

Hi,
Good question, the swag package you need can be found here: https://github.com/wjmaddox/swa_gaussian. I will be sure to update the package requirements accordingly.


Not compatible with react native 0.71.1 : android compilation error

Create a new react native 0.71.1 with react native cli.
npx react-native init test
add datadog dependency
npm install @datadog/mobile-react-native
And run
npm run start npm run android
Describe what happened
The android build process failed with the following errors :
Error: Command failed: ./gradlew app:installDebug -PreactNativeDevServerPort=8081 e<EMAIL_ADDRESS>(183, 9): No value passed for parameter 'sessionReplayEnabled'
e<EMAIL_ADDRESS>(233, 21): Val cannot be reassigned
e<EMAIL_ADDRESS>(238, 21): Val cannot be reassigned
e<EMAIL_ADDRESS>(244, 21): Val cannot be reassigned
Steps to reproduce the issue:
Describe what you expected:
The compilation errors to be fixed
Additional context
"@datadog/mobile-react-native": "^1.2.0"

Hi @agjini, thanks for reporting this!
It's indeed a problem we did not anticipate with RN 0.71, it added the sonatype snapshots repo in the list of gradle repositories, which pulls our own dev snapshot builds.
A PR has been opened here: https://github.com/DataDog/dd-sdk-reactnative/pull/382, it will be fixed early next week in the next release.
In the meantime, you can solve the issue by patching the library with the code from the PR and running (cd android && ./gradlew clean) before building again.
Thanks a lot for your patience, I will let you know when this is fixed!

Thanks for your reactivity,
I'll be patient.

Hi,
This issue should be fixed in the new 1.2.1 release.
I'm going to mark the issue as closed, please feel free to reopen the issue if you encounter any trouble fixing it :)

still facing this issue on :
react-native:0.71.2
@datadog/mobile-react-native: 1.3.1

Hi @sourabhdadapure, thanks for reaching out!
Unfortunately we're not able to reproduce this on our end. Do you make sure to run (cd android && ./gradlew clean) before building?
If the issue persists, could you share your android/build.gradle and android/app/build.gradle files as well as the stacktrace of the error you are seeing?
Thanks a lot!

Hi @louiszawadzki, sorry for the late response. This was happening bc we  upgraded from 0.68 to 0.71 and I was able to fix this by adding thisimplementation("com.facebook.react:react-android") to my build.gradle


#Selector error

New Issue Checklist

I have read the guidelines for contributing and I understand:

[x ] My issue is happening in the latest version of SideMenu.
[ x] My issue was not solved in the README.
[ ] My issue can not be answered on stackoverflow.com.
[x ] My issue is not a request for new functionality that I am unwilling to build and contribute with a pull request.
[ ] My issue is reproducible in the demo project.

Issue Description

Hello,
I pod installed the repo to my project, however I am getting this error.

I found this https://stackoverflow.com/questions/45696737/argument-of-selector-refers-to-instance-method-selectimage-that-is-not-ex in stack overflow. I can add these to the code, but since the example app builds i feel my issue is else where.
I am slightly new to iOS and swift, so sorry if this is a nooby question.
Edit: I should add I am on newest version of Xcode.

It is unclear why you are getting that error as that method is prefixed with @objc. Try cleaning your project and building again. If that doesn't work, you will need to troubleshoot further to figure out what is causing the issue as it works fine in my projects and report back.

@jonkykong I didnt see it in the code, I added it to mine. Thanks

@mcelhennyi It's in the latest code, so make sure you're up to date.

@jonkykong I added the code to my podfile as in the readme, that should give me the most up to date correct?

run pod update

says it installed 3.1 (was 2.3.2) ...awesome thanks man!


Several Fixes

Remove registry::add<Comp>(apx::entity, const Comp&) as we have registry::add<Comp>(apx::entity, Comp&&), and fix the latter to use std::forward.
When looping over the callbacks, use auto& to avoid unnecessary copies.
Fix a typo in registry::view<T>.
Make naming of variables in apx::handle consistent with the rest of the library, and use template keyword to allow for compiling with gcc. I'll be doing more testing on other platforms in the future.

Note to self: before landing, add more tests to cover other templated functions.


:construction_worker: (config): ut + sanitizers - use gcc 11 as experimental

This allows workflow to continue on error

goal is to move to gcc@12 directly, closing the issue for now


Adding go.mod and tagging release?

Can we do this please?

Please submit a PR so I can merge your commit.

Please submit a PR so I can merge your commit.


.RPL Files: paths are not relative?

Hi
Is there anything that can be done to make the Reaper Project list relative to a master Folder? I would like to use the same Project list on more than one computer and it's a challenge to always go into the RPL files to replace the proper Paths.
Thanks For you time.

I'd like to bump this. Anybody?

Project list paths can already be relative to the REAPER process' working directory (by editing the .RPL in a text editor). On Windows the working directory is set to the path in the lastcwd setting in REAPER.ini. On mac it is / (root) unless launched from terminal.
Not sure if this helps though. I suppose the idea here is to have project list optionally load and save projects relative to the RPL file's path?

Thank you very much CFillion
I'm not sure I understand the answer though. Maybe I did not ask the right question:
My goal is to:
Put an RPL file in a master folder containing several Reaper projects.
/Volume/ComputerA/.../MasterFolder/RPLFile.RPL
This File would point to projects that are located in:
/Volume/ComputerA/.../MasterFolder/
Then be able to copy this master folder to any computer and have Reaper being able to open this RPL file
/Volume/ComputerB/.../MasterFolder/RPLFile.RPL
As long as the projects are located in:
/Volume/ComputerB/.../MasterFolder
I hope my question makes more sense?

Ok I guess this is an advance but not sure it is quite what I need?
  I don't quite understand what the CD command does in the Terminal.  I thought that was a simple "Change Directory" command?

Yes cd is change directory. It changes the directory from which programs (here it's REAPER) are run next. If you set the directory to /Volumes/ComputerX/blabla/MasterFolder and have a line "ChildFolder/For/Project.RPP" in the .RPL SWS will attempt to load /Volumes/ComputerX/blabla/MasterFolder/ChildFolder/For/Project.RPP.
(This is pretty much a temporary solution short of rewriting that SWS feature to allow relative loads the proper way.)

Hello CFillion.  Any hope about this feature to be implementing properly?

I believe THIS is what needs to be modified:
https://github.com/reaper-oss/sws/blob/master/Projects/ProjectMgr.cpp

+1 from me;
relative paths would be great, but I also understand that absolute paths are less error-prone, as open projects could sit in totally different directories.
in my use case the RPL file sits in a different folder from the projects, so I would even need **....**xyz\project1.rpp "logic".
that would increase complexity to check which part of the paths is shared across all projects which should go into the list. and only if there is a common part, make the paths relative to the (saving location of) RPL file :-)
will also add two more related "issues"/feature requests about

associating RPL extension with reaper => open RPL right from browser with reaper ... don't know whether this is even possible
"open project list" using last folder, like "open project" and not always some user-profile-folder


Minor: fix streams quickstart in Jenkinsfile

Running commands like "cd" or setting variables inside sh steps do not survive to following steps in Jenkinsfile. Instead we're supposed to use things like dir("somedir") { ... } and injecting variables with env directive or directly into the command string (like I've done here).

Here's the evidence that it's working now https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-9285/1/execution/node/134/log/ :)


Getting started

Could you add a "getting started" guide? Just a brief (but complete) step-by-step guide from nothing to getting the program doing its thing.
I wouldn't have the first idea how to set this program up. I looks like it's intended for a somewhat novice userbase. Now I'm not going to demand you release a clearly runnable executable or even a setup program, like they do over at Upscayl, but it would be helpful to get some guidelines on how to get started with this piece of software.
For starters, requirements.txt looks like ghibberish to me. Not sure what the intention is, but I guess that's an example of what newcomers will likely have trouble with.

Hi my friend,
I have added a small guide on how to install everything you need, I hope it helps.
https://github.com/Djdefrag/QualityScaler?tab=readme-ov-file#make-it-work-by-yourself-

pip : The term 'pip' is not recognized as the name of a cmdlet, function, script file,
or operable program. Check the spelling of the name, or if a path was included, verify
that the path is correct and try again.
At line:1 char:1

pip install -r requirements
I get this error when i want to install, please help

Hi my friend,
have you installed Python?
are you on Windows?
are you using VSCode as editor?

There you go, all that is perfect material for a getting started guide.

Hi my friend,
have you installed Python?
are you on Windows?
are you using VSCode as editor?

YEAH I HAVE

@megarox4u  Hi my friend, can you share a screenshot of your terminal with the error please?

I too am getting the pip command error, I'm running the latest python package and have the directory open in vscode...
Any ideas?

Hi, I am also getting the pip command error...

Both the latest python and vscode are installed, any ideas?

Hi, I am also getting the pip command error...  Both the latest python and vscode are installed, any ideas?

Hi my friend,
have you also installed VsCode extension for python?


Add terraform remote backend functionality

Create a .tf file with the remote backend info
Fix issue with key file not being able to be read

Solved in https://github.com/StijnZanders/limber/pull/47


Module not working in MS teams desktop\web but working fine in mobile (dotnetcore)

Github issues for C# /JS / Java/ Python should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Sample information

Sample type: [samples]
Sample language: [dotnetcore]
Sample name:  [54.teams-task-module]

Describe the bug
Module not working in MS teams desktop\web but working fine in mobile.
on  \Bots\TeamsConversationBot.cs on OnTeamsTaskModuleFetchAsync after call Task.FromResult(taskInfo.ToTaskModuleResponse());
not retrieve date on desktop\web
To Reproduce
Steps to reproduce the behavior:

Go to 'bot chat in teams'
Click on 'Click to Youtube' on the Desktop\web.
See a blank view on the Module.

Expected behavior
Empty page on the module.
Screenshots

Dear @ramfattah,
all same but only the difference on manifest Version  my manifest v is { "manifestVersion": "1.11",}
I will change the version and come back.

the issue has been resolved after changing the manifestVersion to 1.5


Sync AlertDialog close behaviour with documentation

Documentation states that AlertDialog default config for both closeOnEscape and closeOnOutsideClick is true.
However the defaults currently are false.
https://www.bits-ui.com/docs/components/alert-dialog
Let me know if this is expected and the documentation needs to be updated instead of code.

In this case the documentation/component are half accurate! closeOnEscape should default to true, but closeOnOutsideClick should default to `false. The idea is that alert dialog's expect some form of response from the user and should be differentiated from regular dialogs!

Is this something you want to tackle or do you want me to take it from here @zomchak-code?


Feat: Enable proxy-authorization in admin client

TL;DR
Part of an effort to integrate Flyte with GCP Identity Aware Proxy, see https://github.com/flyteorg/flyte/issues/3965.
Allows the flyteidl admin client to sent a "proxy-authorization" header with every request to flyteadmin. Tokens for this header are created using an external command which is configured via the new proxyCommand config entry.
Same logic as was introduced in flytekit in https://github.com/flyteorg/flytekit/pull/1787.
I replicated the existing logic in MaterializeCredentials and NewAuthInterceptor for the second token. The external command is called a single time per flytectl call.
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
See https://github.com/flyteorg/flyte/issues/3965
Tracking Issue
Closes https://github.com/flyteorg/flyte/issues/3965
Follow-up issue
NA

I moved this PR to the monorepo: https://github.com/flyteorg/flyte/pull/4189.


Support DataModule on_save_checkpoint and on_load_checkpoint hooks

🚀 Feature
LightningModules have the ability to augment the checkpoint dict with custom information relevant to the module. The motivation for this issue is to provide the same functionality for DataModules, as they too might have custom information they need to read from or write to the checkpoint dict. Importantly, there will only be one checkpoint saved during training. This checkpoint will comprise the default state generated by the trainer, the lightning module custom data (optional), and the data module custom data (optional)
Ckpt = Trainer State + Lightning module custom data (optional) + Data module custom data (optional)

cc @nateraw
Motivation
Custom data we may want to include in the checkpoint:

Hyperparams
NLP: Path to a vocabulary file file (or even the vocabulary if it's small enough)
Image Text Recognition: Storing the alphabet character set of the dataset
Serialized dataloader state
....

These are all pieces of state that could live in advanced data modules. Importantly, this state isn't static: it can consist of arguments passed into the datamodule, or state generated during training
Pitch

Offer the on_{save/load}_checkpoint functions for the DataModule
Give the DataModule (if it exists) a chance to add things to the checkpoint before we save the checkpoint in a callback. This is an additional line here

if self.trainer.datamodule is not None:
    self.trainer.datamodule.on_save_checkpoint(checkpoint)

Give the DataModule (if it exists) a chance to load things from the checkpoint whenever the trainer loads a checkpoint. This is an additional line here

if self.trainer.datamodule is not None:
    self.trainer.datamodule.on_load_checkpoint(checkpoint)

Sharing code:
We can refactor the existing LightningModule hooks here into a separate CheckpointHook mixin:
Then both the LightningModule and DataModule can inherit from this mixin (similar to what was done for the DataHook), meaning both the LightningModule and DataModule will share the function declarations.
By default, the data module will have an empty stub for these function definitions (as is the current implementation for the LightningModule)
Alternatives
Users are forced to load state from their checkpoint by including their datamodule inside their lightning module
The lightning module ends up this looks like this:

class MyLightningModule(pl.LightningModule):
    def __init__(self, args):
        self.datamodule = MyLightningDataModule(args)
        ...
    def on_save_checkpoint(self, checkpoint: Dict[str, Any]) -> None:
        checkpoint["stuff"] = get_state_from_datamodule(self.datamodule)
    def on_load_checkpoint(self, checkpoint: Dict[str, Any]) -> None:
        self.datamodule.stuff = checkpoint["stuff"]

But this forces the datamodule to be instantiated within the lightning module. the datamodule is not an independent component anymore. According to the docs, this is not a recommended usage of data modules: https://pytorch-lightning.readthedocs.io/en/latest/datamodules.html#using-a-datamodule
Additional context
https://forums.pytorchlightning.ai/t/do-datamodules-have-on-checkpoint-load-save-hooks/180

Assigned @ananthsub as he mentioned he wouldn't mind taking a stab at implementing this. Assigned myself to keep an eye on it.
@PyTorchLightning/core-contributors lets support Ananth as he submits contribution for this. Please leave your thoughts on the proposed solution here if you have any 😄 .


(bug) Do not re-rescue errors raised before config is loaded

This fixes a bug introduced in #1917 that caused errors raised before
config is loaded to be re-raised and rescued by Bolt::CLI#parse and
sent to the outputter. This resulted in a NoMethodError being raised,
as the outputter relies on the config being loaded when it is
instantiated.
This seperates the logic that was in the #parse method into three
separate methods. The #parse_command method handles parsing and
validating the command, the #load_config method loads the project and
config, and the #finalize_setup method finishes configuring Bolt and
issuing any warnings from the previous steps.
Splitting the logic into separate methods allows us to handle errors
raised before and after config is loaded differently.
!bug

Do not re-rescue errors raised before config is loaded
Errors raised before Bolt's configuration was loaded were raising a
second, ambiguous NoMethodError. Bolt now correctly handles errors
raised before configuration is loaded and will no longer trigger
additional errors.

CLA signed by all contributors.

I updated the PR comment to match the commit message. This LGTM!


Moc labs review patch 5

Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Submitted by ParamRamki on behalf of MOC Labs Review


weave is kinda slow

I consider using kontena, but i read an article that says that you loose 94% of the bandwidth by using weave, wich is used by kontena:
http://www.generictestdomain.net/docker/weave/networking/stupidity/2015/04/05/weave-is-kinda-slow/
if i loose 94% of my bandwidth by using kontena it doesn't seem to make much sense to me using kontena.
did you make any performance tests with kontena ?
do you consider switching from weave to flannel (and add vpn-support to flannel)?

Yes, we have benchmarked weave and in our tests it is only 15-20% slower. If you have really high-performance networking needs this might not be acceptable.. but usually it's good enough.
Flannel is faster if you use vxlan backend but there are currently too many tradeoffs for us. One is that Flannel requires etcd to boot, we start etcd only after the overlay network is up (etcd is protected by weave/overlay network).


deleting eks cluster through kubectl delete ekscc does not delete the cluster

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Add catalog - https://github.com/rmweir/system-charts
Branch - add-eks-controller; commit id: c6651c4
Deploy rancher-eks-controller from this catalog.
Deploy an eks cluster using kubectl apply -f cluster.yaml where is something like cluster.yaml

apiVersion: ke.cattle.io/v1
kind: EKSClusterConfig
metadata:
  name: sowmya-2
spec:
  kubernetesVersion: "1.14"
  privateAccess: true
  publicAccess: true
  cloudCredential: ""
  displayName: "sowmya-2"
  region: "us-east-2"
  loggingTypes: []
  tags:
    "test" : "one"
  nodeGroups:
  - nodegroupName: "sowmya-2-ng"
    instanceType: "t3.medium"
    desiredSize: 1
    maxSize: 1
    minSize: 1

create a couple of similar EKS clusters.
delete a cluster using kubectl using kubectl delete ekscc <name>

Actual Result:
The cluster is not deleted. command line hung for sometime.
Expected Result:
The cluster should get deleted
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head - master-3244-head
Installation option (single install/HA): HA

Verified on master-head commit id: 6d6199121

Login as a standard User. User is able to delete the cluster from Rancher UI
Cluster is deleted in the EKS console.
kubectl delete ekscc c-jdmjm -n cattle-global-data deleted the cluster from rancher UI and the EKS console.


Fixed GlobeAwareDefaultPawn not being georeferenced during play

Fixes: https://github.com/CesiumGS/cesium-unreal/issues/421
The GlobeAwareDefaultPawn needs to re-add itself to the CesiumGeoreference during BeginPlay in order to be subscribed for georeference update notifications. This is true even if it had previously added itself during editor mode.

Thanks for the pull request @nithinp7!

:heavy_check_mark: Signed CLA found.

Reviewers, don't forget to make sure that:

[ ] Cesium for Unreal Samples works.

@kring This fixes another 1.2.0 regression, so hopefully we can get it into the hotfix release we were talking about.


net/http: requests sometimes sent even when context is already canceled

We are observing a intermittent error with the ctxhttp package when calling it with already cancelled contexts.
What version of Go are you using (go version)?
Error has been observed on multiple Go versions (1.6, 1.7, 1.9)
Here's some more details from the golang 1.9 build environment (Travis CI based) where this happened today:
$ go version
go version go1.9.7 linux/amd64
go.env
$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/travis/gopath"
GORACE=""
GOROOT="/home/travis/.gimme/versions/go1.9.7.linux.amd64"
GOTOOLDIR="/home/travis/.gimme/versions/go1.9.7.linux.amd64/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build696930010=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"

What did you do?
I have the following test case:
func TestContext(t *testing.T) {
	handler := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		w.Header().Set("Content-Type", "application/json")
		w.Write([]byte("{}"))
	})
	server := httptest.NewServer(handler)
	defer server.Close()

	hc := http.DefaultClient
	doSend := func(ctx context.Context, req *http.Request) error {
		resp, err := ctxhttp.Do(ctx, hc, req)
		if err != nil {
			return err
		}
		defer resp.Body.Close()

		_, err = ioutil.ReadAll(resp.Body)
		if err != nil {
			return err
		}
		return nil
	}

	ctx, cancel := context.WithCancel(context.Background())
	req, err := http.NewRequest("GET", server.URL, nil)
	if err != nil {
		t.Fatal(err)
	}
	if err := doSend(ctx, req); err != nil { // Request 1
		t.Fatal(err)
	}

	cancel()
	req, err = http.NewRequest("GET", server.URL, nil)
	if err != nil {
		t.Fatal(err)
	}
	if err := doSend(ctx, req); err == nil { // Request 2
		t.Fatal("no err raised")
	}
}

What did you expect to see?
I expect Request 2 to fail with an error since the context is explicitly cancelled.
What did you see instead?
Every now and then Request 2 completes successfully (i.e. gets a response from the server), and doesn't return an error.

client.Do() is an internal API of ours that wraps ctxhttp: https://github.com/firebase/firebase-admin-go/blob/master/internal/http_client.go#L43
You're right about us not being able to use use ctx+http APIs at the moment, due to us having to support golang 1.6.

The issue occurs very occasionally. I haven't observed it in the local development environments. But it happens every now and then on our Travis builds. Here's some env details from the golang 1.9 build environment where this happened today:
$ go version
go version go1.9.7 linux/amd64
go.env
$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/travis/gopath"
GORACE=""
GOROOT="/home/travis/.gimme/versions/go1.9.7.linux.amd64"
GOTOOLDIR="/home/travis/.gimme/versions/go1.9.7.linux.amd64/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build696930010=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"

/cc @broady @bradfitz

Very strange. For Go 1.9, ctxhttp is a very light shim:
func Do(ctx context.Context, client *http.Client, req *http.Request) (*http.Response, error) {
	if client == nil {
		client = http.DefaultClient
	}
	resp, err := client.Do(req.WithContext(ctx))
	// If we got an error, and the context has been canceled,
	// the context's error is probably more useful.
	if err != nil {
		select {
		case <-ctx.Done():
			err = ctx.Err()
		default:
		}
	}
	return resp, err
}

so, this might be a bug in net/http. Renaming bug given how unlikely this is a bug in ctxhttp.

I can reproduce it in Go 1.10.3 on linux/amd64 using this modified code without the ctxhttp dependency:
package main

import (
	"context"
	"io/ioutil"
	"log"
	"net/http"
	"net/http/httptest"
)

func Do(ctx context.Context, client *http.Client, req *http.Request) (*http.Response, error) {
	if client == nil {
		client = http.DefaultClient
	}
	resp, err := client.Do(req.WithContext(ctx))
	// If we got an error, and the context has been canceled,
	// the context's error is probably more useful.
	if err != nil {
		select {
		case <-ctx.Done():
			err = ctx.Err()
		default:
		}
	}
	return resp, err
}

func main() {
	handler := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		w.Header().Set("Content-Type", "application/json")
		w.Write([]byte("{}"))
	})
	server := httptest.NewServer(handler)
	defer server.Close()

	hc := http.DefaultClient
	doSend := func(ctx context.Context, req *http.Request) error {
		resp, err := Do(ctx, hc, req)
		if err != nil {
			return err
		}
		defer resp.Body.Close()

		_, err = ioutil.ReadAll(resp.Body)
		if err != nil {
			return err
		}
		return nil
	}

	for i := 0; i < 10; i++ {
		go func() {
			for {
				ctx, cancel := context.WithCancel(context.Background())
				req, err := http.NewRequest("GET", server.URL, nil)
				if err != nil {
					log.Fatal(err)
				}
				if err := doSend(ctx, req); err != nil { // Request 1
					log.Fatal(err)
				}

				cancel()
				req, err = http.NewRequest("GET", server.URL, nil)
				if err != nil {
					log.Fatal(err)
				}
				if err := doSend(ctx, req); err == nil { // Request 2
					log.Fatal("no err raised")
				}
			}
		}()
	}
	select {}
}

It takes only 1-10 seconds before there's no err raised. (Sometimes it also exits with another error message: connect: cannot assign requested address)

Some my thoughts.
Problem is located in http transport.

transport want to get connection. but at this step cancel could be missed. Because transport can take an idle connection. https://github.com/golang/go/blob/master/src/net/http/transport.go#L947
after this transport tries to read response https://github.com/golang/go/blob/master/src/net/http/transport.go#L1927 but here cancel could be missed too if response is ready.

Thanks gophers. Do you usually backport this sort of thing to older versions of Go, or is it just going to be included in the next milestone release?


Fatal error: SDL/SDL.h: No such file or directory

I have tried many things, I have searched a lot on stackoverflow and askubuntu and other pages and I have tried everything they said, I can't find anything to fix this...
Building OneLife...
/usr/bin/i686-w64-mingw32-g++  -Wall -Wwrite-strings -Wchar-subscripts -Wparentheses -g  -DWIN_32 -DSTATICLIB  -O0 -I../.. -c -o ../../minorGems/graphics/openGL/ScreenGL_SDL.o ../../minorGems/graphics/openGL/ScreenGL_SDL.cpp
../../minorGems/graphics/openGL/ScreenGL_SDL.cpp:145:10: fatal error: SDL/SDL.h: No such file or directory
145 | #include <SDL/SDL.h>
|          ^~~~~~~~~~~
compilation terminated.
make: *** [Makefile:156: ../../minorGems/graphics/openGL/ScreenGL_SDL.o] Error 1
done compiling.
Building editor...
/usr/bin/i686-w64-mingw32-g++  -static sdl-config --static-libs -Wall -Wwrite-strings -Wchar-subscripts -Wparentheses -g  -DWIN_32 -DSTATICLIB -DUSE_PNG  -O0 -I../.. -c -o ../../minorGems/graphics/openGL/ScreenGL_SDL.o ../../minorGems/graphics/openGL/ScreenGL_SDL.cpp
../../minorGems/graphics/openGL/ScreenGL_SDL.cpp:145:10: fatal error: SDL/SDL.h: No such file or directory
145 | #include <SDL/SDL.h>

The SDL header is doesn't exist simply because you don't have SDL installed.
Either install it with installMissingLibraries.sh or get it portable with [getSDLforMingw.sh] (https://github.com/Defalt36/TwoLifeXcompile/blob/master/exclude-dir/OldScripts/getSDLforMingw.sh) (Makefile.MinGWCross will not find the portable version as it only search installations now).

Thats weird, I should have it. Not only because I checked that your "doEverything.sh" starts "installMissingLibraries.sh", but I have it in "\wsl$\Ubuntu-20.04\usr\include\SDL\SDL.h". And command "sdl-config --version" says I have version 1.2.15 installed.
I remade all in my new pc with w11 too, and the same problem.
Thanks for the quick response.

Thats weird, I should have it. Not only because I checked that your "doEverything.sh" starts "installMissingLibraries.sh", but I have it in "\wsl$\Ubuntu-20.04\usr\include\SDL\SDL.h". And command "sdl-config --version" says I have version 1.2.15 installed.

Yes, you have it but this is the SDL installed in your system, or in another words Ubuntu's SDL. You are cross compiling so you need the libraries for windows and you need then installed in your compiler folder (MinGW).
Aren't you over-sighting this? "Known Issues SDL won't install if it is in a folder with blank spaces in the path."

Got another in editor, but I think I will not need it... I will leave it here anyways.
minorGems/graphics/converters/PNGImageConverter.cpp:26:10: fatal error: png.h: No such file or directory

This is yet another library missing in MinGW: LibPNG.

Fixed: https://github.com/Defalt36/TwoLifeXcompile/commit/c7655205ebb4d4c2247bbb498d685f2ea096365b


handwriting incomplete

When strokeWidth is wide enough (gt 16px、20px.....) and  opacity is 0.5;  handwriting is incomplete;

i have know; rough.js RoughCanvas._drawToContext()  set ctx.lineCap = 'round'

i have know; rough.js RoughCanvas._drawToContext()  set ctx.lineCap = 'round'

If you know how to fix it, we're open to PRs :)

If you know how to fix it, we're open to PRs :)

It seeming rough.js context2d Miss parameters  ‘lineCap’，


Add support for the Hampton Bay 10W Equivalent Outdoor Stake Light

Model number L08557. It appears to be very similar in functionality to existing lights with this color temperature spectrum.

I've been running a local copy of this integration with this change for roughly a year and a half, so I figured it was time to actually submit it so I could keep updating in HACS. :smile

Thanks. I added it in.


Remove device selection as only CPU can run this due to strings as input

CVS-143329

@aleksandr-mokrov please add notebook into this list https://github.com/openvinotoolkit/openvino_notebooks/blob/latest/.ci/check_notebooks.py#L9


Campaign autocert doesn't handle links to other data portals

Campaigns pick up datasets that are really hosted on other portals. The result is it tries to fetch dataset meta from the wrong url.
The following search shows a few examples of these datasets:
http://data.sa.gov.au/data/dataset?q=threatened+species+state+lists&sort=extras_harvest_portal+asc%2C+score+desc%2C+metadata_modified+desc
Check out the ones tagged 'FROM DATA.GOV.AU'

There's a fair bit of cross harvesting on Australian portals. Some call it the "no wrong door" approach. So wherever you go, you'll find all the links. Data.gov.au and data.sa.gov.au do this the most. Not every Government has adopted the idea.

Decision is:

By default, ignore datasets that are harvested from other portals, to avoid duplication of certificates.
Add an option in the campaign to include harvested datasets in the campaign (work in progress in #1442)

I've updated #1442 to do the above :+1:

suggest this can be closed based on #1442


404 cleanup

Proposed changes
Fix some 404s
Fixes #3168
Fixes #3218
Fixes #3323

Verified all fixes in a local build (Netlify is inadequate for this). Merging.

It's the current Edge version's run reference. It's something we will need to bump every time there is a new Edge version. It's important to have the Edge one in case we add new docker run flags that are documented there.
cc/ @thaJeztah in case there is a more clever solution.


Add example of Finnish addresses

An example on how-to-run hakunapi to serve addresses of buildings of Finland available from https://www.avoindata.fi/data/fi/dataset/rakennusten-osoitetiedot-koko-suomi

Read through sample readme, script and conf files under examples/finnish_addresses/.
Seems OK to me. Few minor comments below.
The section "Configuring and running hakunapi" in readme.md:

after "Restart your tomcat ... http://localhost:8080/features" it would be nice to have a short list of actions needed after testing in localhost
for example what parts in "addresses.properties" should be edited before deploying on a real test or prod server to get server resource links to match a real server
no need to have instructions how to setup such server, that's out of scope, but about hakunapi configuration

Other comments:

Maybe "api.contact.email" in "cfg/addresses.properties" should be left empty in demo?

Thanks for changes. The readme with "preparing for production" section is good now.

As commented in #18 : The demo implementation behind https://beta-paikkatieto.maanmittauslaitos.fi/inspire-addresses/features/v1/ is now the demo of Hakunapi called "finnish-addresses" implemented by this pull request.
Some comments from developers (in Finnish) about instructions
Ohjeesta puuttui

komennosta HEADER:
psql -d address_fin -c "\copy suomi_kunnat (sijaintikunta, nimi_suomi) FROM 'suomi_kunnat.csv' WITH (FORMAT CSV, DELIMITER ';' , HEADER)"

Tiedoston nimi oli väärin: (insert_simple_address.sql
psql -d address_fin -f simple_addresses.sql

Tietoturva näkemys

GRANT SELECT käyttjälle ja muut operaatiot tehtiin postgre käyttäjällä

Fixed missing HEADER option from copy command, changed insert_simple_address.sql to the correct file name simple_addresses.sql, altered the scripts to suggest creating address_reader as read-only user and running the managerial parts of the scripts as postgres (or other superuser)


cfssl can be installed with homebrew on OSX

Are you wanting to have this repository transform eventually into an automated tool, or recommend ways to make it easier?  Because installing CFSSL on OS X can be done with:
brew install cfssl

@tonybecker This is great feedback, but I'm not sure if I want to capture all those details in this repository . I want people to take a single path and focus on learning, but I do want to improve the guide for as many people as possible.
I think fetching the binaries is a good default for learning out of the box. While I'm going to close this issue, I'm going to continue to think about ideas like this going forward.
Thanks for taking the time to file this issue.

I came from the new docs to suggest something similar. My thoughts are that this would be "If you have brew installed, you can run brew install cfssl. Otherwise, here are the steps for installing from scratch". This simplifies the instructions for those with brew, without needing to explain how to setup brew for those who don't.


My device can be discovered too?

hey I just wondering if its possible that while im scanning my device can be discovered by other scanner?
I mean other device can see my device around (android and iPhone)
in different words im trying to advertise while im scanning, is that possible?
Thank you

Advertising is different feature using different APIs under the hood: CBPeripheralManager (iOS) and BluetoothLeAdvertiser (Android). This library don't support them.


Add convert tuple to map

In vtl (velocity template engine )
get (0) to get the value of this approach in arrary form as discomfort.
It adds a tuple convert map feature so that you can use the value value to the string key.

There is already QMap which provides a Expression to Value mapping in Map form. The toString() of the expression is also not guaranteed to be unique, so I wouldn't promote such a mapping in the core module.
In your case it might be best your own FactoryExpression that populates a Map with expr.toString() as keys.

Thank you for your advice.
The following information has been helpful in my case.
List<Map<Expression<?>,?> result = query.from(employee).select(Projections.map(employee.firstName, employee.lastName)).fetch();
 for (Tuple row : result) {
     System.out.println("firstName " + row.get(employee.firstName));
     System.out.println("lastName " + row.get(employee.lastName));
 }


Using Open GL with Compose Multiplatform

I have an app that is photo filters app, It applies filters on photos using Open GL, I've recently made the photo filter library multiplatform using kgl which is a kotlin multiplatform library that provides abstraction over Open GL.
On Android I can use a SurfaceView, on Desktop I need to integrate compose with Lwjgl or Jogl, on Web I do have a Canvas but no clear way to integrate Open GL with it alongside Compose
What I am looking for is a Multiplatform Compose Open GL Composable View
I can implement it on my own but there's no way to implement it

I can implement it on my own but there's no way to implement it

That sort of confused me.
There are no plans to make a general purpose widget OpenGL canvas a part of multiplatform. You can roll your own solution, using the interop APIs though, which will certainly require some tinkering.

You will have to write expect wrappers around all GL functions you want to use, and implement the actuals for each platform.
I'm not sure about how you can use OpenGL to draw into Swing panel, which you can interop with iOS using https://developer.apple.com/documentation/glkit/glkview?language=objc.
Honestly, to me it seems quite a bad idea. If you need to draw custom stuff, perhaps have a look into Canvas widget? If it's 3D after all, Compose is just a wrong tool.
Even if some sort of 3D will be supported in the future, it will certainly be abstracted away from the specific graphics API.

I have done this

You will have to write expect wrappers around all GL functions you want to use, and implement the actuals for each platform.

I just need JVM & Web guide, On JVM I am using lwjgl or jogl, whatever works !
On Web, how can I make sure compose and webgl both render on to canvas and in their specific locations
So two composables, one for Web and one for Jvm that render Open GL View, I would love to do this on my own but I don't know how...
Photo processing is not performant enough and that's why I am using Open GL, I think Having a GL View inside compose is an actual use case, Because people write games with it and stuff and Open GL is used a lot. I am not implying that a multiplatform widget should be created for everybody to use, Just that APIs are available so this can be done.
I cannot just ditch Open GL since all of my code is already written using Open GL.

Here's the Idea
Android
I can use a SurfaceView wrapped in AndroidView Composable
Web
On Web I get the underlying Canvas & webgl context, I also get the coordinates of the current composable using BoxWithConstraints and I call glViewport with bounds of the current composable which will let Open GL draw into the Canvas only in the bounded region, Somehow I need to make sure Compose doesn't draw on this region
JVM
I have no Idea, I am thinking Lwjgl Integration which currently requires some hacks.

Photo processing is not performant enough and that's why I am using Open GL, I think Having a GL View inside compose is an actual use case, Because people write games with it and stuff and Open GL is used a lot. I am not implying that a multiplatform widget should be created for everybody to use, Just that APIs are available so this can be done. I cannot just ditch Open GL since all of my code is already written using Open GL.

You can use custom Skia shaders for non-Android targets. Look at RuntimeShaderBuilder, RuntimeEffect.

I cannot give up Open GL at the moment, It would mean rewriting everything, Maybe Vulkan one day since Open GL is deprecated
I guess I am going to keep using SurfaceView for Android and hope that multiplatform solutions catch up so that this is possible
I think Compose one day will have to use Open GL or Web GPU because Skia is 4 - 8 MB which is not doable when loading a web page & at the moment all my web apps load really slow, so I am not too worried, A multiplatform solution would've been more cleaner

Multiplatform solution uses skia internally to perform rendering, there are not plans to change it to something for now. Even if we do it in platform specific way, it most likely be Vulcan for Desktop, wgpu for Web, and Metal for iOS.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


chore(CI): improve milestone handling

Proposed Changes
Following the discussion in https://github.com/camunda/camunda-modeler/pull/4400#issuecomment-2191806614 , let's keep closing the current milestone an explicit step in the release.
This allows us to merge and close issue in the time between Release build and Closing of the release issue.
Checklist
To ensure you provided everything we need to look at your PR:

[x] Brief textual description of the changes present
[ ] Visual demo attached
[ ] Steps to try out present, i.e. using the @bpmn-io/sr tool
[x] Related issue linked via Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}

Verified that automatic milestone creation works.
What would be awesome is if we could attach the description "Tracks the v5.{NEXT_MINOR}.0 release of the app." to it, cf. milestones.

We can follow-up to automate more things, like the description 👍

Cf. follow-up issue.


Update ksproperty-cameracontrol-extended-vfr.md

The correct implied settings for frame rate should be variable. All current cameras without this control use variable by default, and lack of this control should not contradict that.

Thanks for the PR @gerritsMSFT !


Format adds/removes space after description on every run

The doc format is behaving oddly with the description property within an element.
With this example:
someone = person 'Person' {
    description: 'A person'
  }

When formatting once the .c4 file, a space is added to description
someone = person 'Person' {
    description : 'A person'
  }

Formatting a second time, the space is removed...
someone = person 'Person' {
    description: 'A person'
  }

If you have a mixed bag, it reverses itself
Pre-format
someone = person 'Person' {
    description : 'A person'
  }
someoneElse = person 'Person2' {
    description: 'Another person'
  }

Post-format
someone = person 'Person' {
    description: 'A person'

  }
someoneElse = person 'Person2' {
    description : 'Another person'
  }

Here's a live example:
https://github.com/user-attachments/assets/ad485442-1bd9-4079-892d-fcc32cf1ddb5

@pavelpykhtin I recall there was discussion on langium, formatting issues on second run.

@davydkov , yes, I've checked that issue. It is already closed with fix.
This particular issue is related to two overlapping edits which the formatter is trying to apply: add space after a property name and remove space before a colon. If there is one or more space between a property name and a colon, then 'remove space' edit is applied and overwrites any other edits. But if there is no space, then the second edit is ignored and 'add one space' edit is applied.
I hoped that formatter solves such issues automatically, but it seems it's not the case.
Thus, I added a check to ensure that there are no overlapping edits.


revolut.md

url: "https://www.revolut.com/go-from-cash-to-crypto-instantly"

Sorry, I don't think this fits the site very well.


Fix #188

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
#188

Closing PR. I accidentally removed the repo. Redoing


define CRUD services in the routes file

add all service definitions in the routes file

All routes have been created!
NOTE to team: For individual CRUD actions, you just need to work on the controller aspect of the service as Shiho has created all the routes.


Forms - Field labels: Update font size to 15px

This is an iteration from the initial design.

Created a duplicate in Trello and will continue progress/conversation there.


rsi function incorrect calculation in train_and_evaluate.ipynb

Describe the current behavior
rsi function in train_and_evaluate.ipynb calculation is incorrect by construction:
def rsi(price: 'pd.Series[pd.Float64Dtype]', period: float) -> 'pd.Series[pd.Float64Dtype]':
    r = price.diff()
    upside = np.minimum(r, 0).abs()
    downside = np.maximum(r, 0).abs()
    rs = upside.ewm(alpha=1 / period).mean() / downside.ewm(alpha=1 / period).mean()
    return 100*(1 - (1 + rs) ** -1)

In the code upside is actually the downside and vice versa. abs() applied without discrimination or good reason only masks the error in the calculation.
Describe the expected behavior
One fix is:
    upside = np.maximum(r, 0)
    downside = -np.minimum(r, 0)

or, simpler:
    upside = r.clip(lower=0)
    downside = -r.clip(upper=0)

Fixed


Small improvements to performance view

In GitLab by @ltenorio on Jun 7, 2024, 17:02

[x] Update to have two significant digits in the Value (https://earth.bsc.es/gitlab/es/autosubmit-api/-/issues/79#note_281158)
[x] Add Parallelization, RSYPD, and number of considered jobs

In GitLab by @ltenorio on Jun 7, 2024, 17:05
mentioned in issue autosubmit-api#79

In GitLab by @ltenorio on Jun 13, 2024, 16:32
mentioned in commit 93515c3dcbbc406b22c67b852463d533f3b87e8f

In GitLab by @ltenorio on Jun 13, 2024, 16:32
marked the checklist item Update to have two significant digits in the Value (https://earth.bsc.es/gitlab/es/autosubmit-api/-/issues/79#note_281158) as completed

In GitLab by @ltenorio on Jun 13, 2024, 16:32
marked the checklist item Add Parallelization, RSYPD, and number of considered jobs as completed


Client context test peer

A mock up class for testing client context.

@vjpai

@vjpai Unit test for client_context_test_peer.h is added. Thanks for the attention.

Have you run tools/buildgen/generate_projects.sh ? Looks like some parts of it are still missing.

Following up: what is the status here? It looks like the change requested in my previous commit was never added, and the PR is now in a conflict state.

@shangwu-google Many build failed due to compilation errors, PTAL.

This is building now. I think it's mergeable.

@nupurkot I'm going to merge this.

@nupurkot I'm going to merge this.

Thanks @vjpai for taking care of this!


Add Japanese translation to AWS Lambda deployment guide

Summary
This pull request adds Japanese translation version to AWS Deployment Guide.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

TODO: Update this PR to reflect #800 changes

TODO: https://github.com/slackapi/bolt-js/issues/801


Init and static construction

Using the init method fails:
class Users extends DenchikBY\MongoDB\Model
{
}

$users = Users::init()->first();
var_dump($users);exit;

Fatal error: Uncaught Error: Call to undefined method Users::first() in /vagrant/html/app/controllers/AuthController.php:9 Stack trace: #0 [internal function]: AuthController->indexAction() #1 [internal function]: Phalcon\Dispatcher->callActionMethod(Object(AuthController), 'indexAction', Array) #2 [internal function]: Phalcon\Dispatcher->_dispatch() #3 [internal function]: Phalcon\Dispatcher->dispatch() #4 /vagrant/html/public/mvc.php(44): Phalcon\Mvc\Application->handle() #5 {main} thrown in /vagrant/html/app/controllers/AuthController.php on line 9

I cannot understand the Model init method, it calls a 'new static' construction, but there's no constructor method defined in Model, and the parent (Collection) constructor doesn't expects the parameters given by the init method. I'm using php 7.1 but it fails also in php 5.6

Sorry mine error


Explain mermaid feature is not implemented

adding note to misleading documentation about currently unavailable feature, per #5788
Fixes https://github.com/MicrosoftDocs/vsts-docs/issues/5788
Fixes https://github.com/MicrosoftDocs/vsts-docs/pull/5798

@jamie-tillman Thank you for this contribution. From the post made in the linked issue #5788 , it looks like that feature is rolling out but possibly may have gotten delayed and so the documentation may have accidentally gotten in advance of the availability of the feature? (I will find out). Then with your permission I can update the note in your PR to include that information.

That all sounds fine.  Thank you!
On Tue, Oct 1, 2019 at 9:04 AM Steve Danielson<EMAIL_ADDRESS>wrote:

@jamie-tillman https://github.com/jamie-tillman Thank you for this
contribution. From the post made in the linked issue #5788
https://github.com/MicrosoftDocs/vsts-docs/issues/5788 , it looks like
that feature is rolling out but possibly may have gotten delayed and so the
documentation may have accidentally gotten in advance of the availability
of the feature? (I will find out). Then with your permission I can update
the note in your PR to include that information.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/vsts-docs/pull/5798?email_source=notifications&email_token=AMRF2TNXJLLRMXEASPI3KS3QMNDFDA5CNFSM4I4CAOBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEABF54Q#issuecomment-537026290,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMRF2TKDBKIZ6SK5G3ARAY3QMNDFDANCNFSM4I4CAOBA
.

--
Jamie Tillman / Chief Executive Officer & Lead Architect / www.Prolydian.com

@jamie-tillman I am merging this PR with a modified note and I am seeking out more information, and  I'll update that note with more information as soon as I have it.


バージョニングを自動化

参考: https://devblog.thebase.in/entry/automatic-release-on-github-actions
master ブランチに push された際に限定し .go ファイルが更新された際にバージョニングを適宜行う処理を自動化します。

Codecov Report

Merging #47 (5043f4f) into master (1fa903b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #47   +/-   ##
=======================================
  Coverage   82.67%   82.67%
=======================================
  Files          20       20
  Lines        1836     1836
=======================================
  Hits         1518     1518
  Misses        216      216
  Partials      102      102

Flag
Coverage Δ

unittests
82.67% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


fixed syntax error

fixed an error that causes training with forgery neuron to fail

superseded by https://github.com/luizgh/sigver/pull/8


using factory pattern for database connection #106

This PR implements factory pattern for database connection.

Hi there, thanks for your contribution. I don't see any improvements to the code nor the factory pattern I was expecting.
Elaborate why I should merge this please.

@grumpyp I'm sorry, actually I'm a beginner I thought I understood factory design pattern and went on to do it. You must be right I'll try to be more careful.

Hi @Sadaf-A
do you want to keep working on it then or get back here once you feel like tackeling the issue?
There are easier issues to start with I'd say :)

@grumpyp I'd like to try once more


Support UI component permissions

Like in CUBA

Duplicate of https://github.com/Haulmont/jmix-security/issues/53 ?


[backport #658] Check for overflow in Script::bytes_to_asm_fmt()

Backport of #658

You can confirm the commits have identical diffs using git range-diff. So I am also claiming that my ACK from #658 applies here.
ACK 6c3434c526e7b8607ff30b63b2ccd9ef8dbd9d82

Many thanks!

Thanks from me as well!

FYI we will probably do another point release in the next couple days to more thoroughly kill this bug (i.e. we'll add some i686 fuzztests for these functions and find the other overflows)

Great! Shame I couldn't get #662 to work with no-panic then fuzz tests wouldn't be needed.


fix: use hgdata actions

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


Add compatibility with nREPL 0.4

Support with tools.nrepl is kept for now with the assumption
that tools.nrepl namespaces would be loaded automatically by
lein repl or boot repl.
This fixes #39.

@bbatsov Thanks so much for this! I'm in the middle of the woods with little connectivity this week, but I plan to release this around Sunday/Monday.

Thanks for the response!

@bpiel Ping :-)


Baking NavigationMeshInstance using the action prevent further edition of the NavigationMesh properties

Godot version
3.4.0-beta3
System information
Windows 10, GLES3, NVidia RTX 2070
Issue description
When working with navigation and navigationMeshInstance, after clicking on Bake NawMesh in the editor, the properties of the NavMesh ressources are no longer editable (you can not open dropdowns, edit numeric fields...). The issue persists until the resource panel is closed and reopen (either by closing and opening the resource or changing nodes in the scene tree).
Steps to reproduce

Create a new scene
Add a navigation node, and a NavigationMeshInstance as it's child
Add Some content under the mesh to have something to bake to (maybe optional?)
Select the navigationMeshInstance Node
Click Bake NavMesh in the editor

Minimal reproduction project
BugReport340Beta3.zip

Confirmed in 3.3.2-stable too, so it's not a recent regression.
To clarify the issue, it seems to happen when the navmesh subresource of the NavigationMeshInstance was expanded before pressing Bake NavMesh. Then indeed, the Inspector behaves weirdly. You can see in this GIF how the click actions work only every now and then:

I got this issue too in 3.4 beat4

@akien-mga
This is the same selection issue as in #34764 which seems to be fixed in Godot 4.0 and 3.5 so this can be closed too?


Add support for custom connection string parameters in PostgreSQL connection URL

Summary
Add support for custom connection string parameters in PostgreSQL connection URL for Argo Workflows, specifically to allow setting binary_parameters=yes. Currently, the connection string configuration doesn't support adding custom parameters, which causes issues when using PgBouncer with PostgreSQL for archiving and offloading workflows.
Use Cases
When would you use this?
The modification is particularly important for production environments where PgBouncer is commonly used as a standard practice for PostgreSQL connection pooling and management. Or any other custom use case people may have.

Message from the maintainers:
Love this feature request? Give it a 👍. We prioritise the proposals with the most 👍.

Related Slack conversation: https://cloud-native.slack.com/archives/C01QW9QSSSK/p1734516501951449


⚠ Bofrost.at has degraded performance

In a2c8c52, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 986 ms

Resolved: Bofrost.at performance has improved in 1e93f6f.


No patches can be applied to input file

System
Laptop MSI Prestige 15 A10SC:

BIOS Version: E16S3IMS.108
GPU: GTX 1650 Max Q /  Mobile

Description
I can't patch my BIOS file after including the ReBarDxe module. Is there anything else I can do?

In UEFIPatch's folder there were two .txt files, "patches.txt" and "patches-misc.txt". I deleted them and only kept the file with the patches the guide explicits: https://github.com/xCuri0/ReBarUEFI/blob/master/UEFIPatch/patches.txt

It means that there are no patches for your BIOS. Some motherboards (like Chinese Xeons) work without patching and don't have any that apply.
You can still add the ReBarDxe module and 1GB size will probably work


Template for a AIreferenceArchitecture involving components in multi-cloud or on-premise

Scenario Description

We want to know of any AIReference Architecture involving some components in another cloud or on-premise
Reasons for scenario

Some of the components like versioning may be required to be in another cloud or on-premise to comply with organization wide policy
Other Comments

This is a nice suggestion. Definitely somehting we might consider further down the line


Fallback of jit compilation

Hi,
first of all, congratulation to this amazing project :smile:
Unfortunately the jit compilation doesn't work. There is a fallback.

It seems that the Bound_class is the problem, because it is not a tensor directly.
Do you have an idea how we can fix them?
Best regards
Marc

Hi Marc
Thanks for reporting this. Could you please tell me which version of PyTorch (and which platform) you are using?
Thanks
Yael

This issue seems to be related to https://github.com/pytorch/pytorch/issues/82889
It only happens in recent versions of pytorch (I reproduced it on 1.12, but I don't know when it was introduced).
In the meantime, the suggested workaround (set the environment variable PYTORCH_JIT_USE_NNC_NOT_NVFUSER=1) seems to work.
I could set it from inside torch-interpol (on import) but I am not sure it's the best solution.

Too add, I also was having some issues with NVFUSER but torch 2.0.X seems to have fixed whatever the issue was.


Refactored LibrarySyncService

This PR is just the refactoring part of PR #236. Hopefully, it makes reviewing #236 a bit easier.

I'm guessing you'll be merging #236 first. So, this PR will not be needed anymore.


Enby speakers

Hi!
Great initiative.
You can maybe add the list of enbies speakers to your list?
The website is here: https://enbytech.github.io/.
Have a nice day 👍

Hi, @hiwelo — this would be a great addition to the list! Would you be willing to open a pull request with the change?
The instructions for adding groups are here: https://github.com/jlengstorf/awesome-speaker-diversity/blob/master/README.md#please-add-your-group
Thank you!

Yes sure, I'll do this right now 👍


Add MPIJob Support in kubebench

A few things to add support for [mpi-job] (https://github.com/kubeflow/kubeflow/tree/master/kubeflow/mpi-job)

Update rbac to grant kubebench service account permission to operate mpi jobs PR. https://github.com/kubeflow/kubeflow/pull/2442
Add mpijob_manifest_modifier to update manifest files
Add customized post_processor for MPI Job. Retrieving logs part is tricky, instead of using runner, since only launch-job has logs, I directly fetch pod logs from launch job after job complete.
Use command array in mpi job spec.  PR https://github.com/kubeflow/kubeflow/pull/2434
Add StartTime and CompletionTime to chain argo workflow
(This can be done using successCondition: status.launcherStatus = Active  and  successCondition: status.launcherStatus = Succeeded, but anyway mpi-operator doesn't have timestamp to indicate job duration. PR https://github.com/kubeflow/mpi-operator/pull/89)

for item 2, will clean up code and submit PR soon.

/cc @Jeffwan

/assign @Jeffwan

Just remember we already have one to track status. #19  Close this one as duplicate


NativeApp openURLInDefaultBrowser doesn't seem to work

So, I decided to write an extension, and I hit a snag while trying to use the NativeApp module's function openURLInDefaultBrowser. NativeApp.openURLInDefaultBrowser(generateURL(currentEditorText)); doesn't seem to open a tab or a window in the default browser. I'm on a mac with OSX 10.14.6, and the browser is Opera GX. Brackets : Release 1.14 build 1.14.2-17770 (release-1.14.2 f71f00acc).

oh and openLiveBrowser doesn't work either

._. it's working now, idfk why, sorry for the trouble if any

Ok, so, it doesn't work when I use variables as the argument for the URL,
but when I use direct strings it does


New spoilers

Spoilers from FFG stream:
Infiltrator
Court of Deception
Duty
Spoilers from Brad Anders' deck at Birmingham kotei:
Invocation of Ash
Master of Swift Waves
Sneaky Shinjo
Shiotome Encampment

deployed


Added flag to disable gossip explicitly.

Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

This is great, finally the noise in logs will stop :+1:

Looks nice :+1: Small nitpick on the naming of stateFetcher but that might be just me !

Small nitpick on the naming of stateFetcher but that might be just me !

What nitpick? We can improve the naming it if we can find better name for this (:

It just felt weird to add a new component name but on a second read, it clicks :+1: !

Will there be any other Gossip/Peering implementations ?

You mean for fileSD?


chore(firmware): bump Jade firmware version to 1.0.30

Tested all four devices on both the device-specific/ script and the flash_your_device script.
Seeing an intermittent issue on the Core Basic and the T-Dispay devices where the screen doesn't appear after flashing. Re-running the the device-specific/ script and the flash_your_device script worked to fix it. Might be an issue with the laptop or cable I'm testing with. If this continues to be a problem in the future, will investigate as a bug.
Ready to squash-merge.


Make sure analyzer bots catch lints and implicit downcasts configured through analysis_options.yaml

Today I had to fix an implicit downcast error that I accidentally introduced (https://dart-review.googlesource.com/c/sdk/+/144780).  For some reason, neither the trybots nor the post-commit bots caught this error.  I haven't investigated deeply, but I suspect that the bots are running the analyzer in a way that somehow prevents it from noticing the implicit_downcasts setting in the nnbd_migration package's analysis_options.yaml file.
@bwilkerson has noticed a similar issue with lint failures--they are getting committed without breaking any bots.  Since lints are configured using analysis_options.yaml, it's possible that this issue has the same root cause.

The new strict-casts: true analysis option has landed, I can look into how much work it is to enable it for package:analyzer and package:nnbd_migration.


[Bug]:  RuntimeError: Error in model execution (input dumped to /tmp/err_execute_model_input_20241016-170451.pkl): view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead.

Your current environment

The output of `python collect_env.py`
vllm0.6.3

Model Input Dumps
No response
🐛 Describe the bug
vllm/model_executor/models/glm4_vision_encoder.py
The 100 lines of the file should be modified to：
output, _ = self.dense(out.transpose(1, 2).reshape(B, L, -1))
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

Same error when running gguf model with offline inference:
from vllm import LLM

But no problems when running via terminal
vllm serve

vllm version: 0.6.3.dev56+g7da24875.d20241001

have anyone solve this problem


⚠ Bofrost.de has degraded performance

In 8e7327c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1134 ms

Resolved: Bofrost.de performance has improved in 590b891 after 309 days, 3 hours.