[CIT-588] fix broken sm tests

Proposed changes

Fix the broken SM-Mapper integration tests
Added a new integration test to test the sm mapper failure and recovery in between the SM operation
Removed a test that that was testing publishing of software list on mqtt topic

Types of changes
What types of changes does your code introduce to thin-edge.io?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

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

Codecov Report

Merging #467 (e460c83) into main (2af6bdc) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #467      +/-   ##
==========================================
- Coverage   62.63%   62.48%   -0.15%
==========================================
  Files         114      114
  Lines        7231     7248      +17
==========================================
  Hits         4529     4529
- Misses       2702     2719      +17

Impacted Files
Coverage Δ

mapper/tedge_mapper/src/sm_c8y_mapper/mapper.rs
9.27% <0.00%> (ø)

mapper/tedge_mapper/src/sm_c8y_mapper/tests.rs
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Improve downloader

Currently the iroh::downloader module is only used for iroh sync. Regular downloads (for blob download) are using the iroh bytes API directly. What we want to do is:

[x] Move the downloader iroh-bytes
[x] Improve downloader and enable use in iroh node for blob downloads

#2085
#2165

[x] Improve content downloading in docs:

#2127

[ ] Support content discovery and announcements in the downloader

Most of the originally planned work here was done, closing this


Remove references to C compilers

Description
We don't use C in any way so we can just stop specifying a C compiler. This removes some code and has a nice downstream benefit of ensuring that those who build SFML (which includes all FetchContent/add_subdirectory users) no longer have to specify a C compiler either!

SFMLBuildMaster: Build this please.

Is it because we're build C code (e.g. stb* or vulkan headers) as C++ code?


[SPARK-37985][SQL] Fix flaky test for SPARK-37578

What changes were proposed in this pull request?
In SQL test SPARK-37578: Update output metrics from Datasource v2
Execution's metric value is not null seem only can promise the execution is not still running.
But can't promise bytesWrittenListener have received all TaskEnd event, cause test failed.
Why are the changes needed?
Fix flaky test
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existed UT

ping @HyukjinKwon @cloud-fan

Merged to master.
@AngersZhuuuu do we need to backport to branch-3.2 too? Seems like there's a conflict but your JIRA mentions we should fix it in 3.2.0 too.

Merged to master.
@AngersZhuuuu do we need to backport to branch-3.2 too? Seems like there's a conflict but your JIRA mentions we should fix it in 3.2.0 too.

Affected version should be 3.3.0.  Seems I can't change it now. Could you help to change this?

I can change it


Add folium integration and use it in an example notebook

With the discovery of the folium package, I remove the old hacky stuff
I put in before for gmaps and use folium instead!
I also added an example notebook in the examples/ folder that
successfully renders a map centered at Broad Street (now Broadwick
Street).
The next step is to add the points / polygon functionality, and we can call #5 a wrap.

Also — I didn't increment the package version. Do we increment the version on every PR?

That's fine - we only increment when we republish the package.
This feature would definitely warrant a new version, but we don't know how many other deployments we might make before this is ready.


Incorrect message sending behavior on a bad connection

When trying to send a message on a bad connection, I experienced the following unexpected behaviors:

No local echo, even though there was no special content in the message.
Once the message failed to send, the was an "Error sending message." banner and the message content was in the compose box (which is good). However, clicking on the "Send" button did nothing until I closed the compose box and opened it again.

@N-Shar-ma , would you be up for investigating this?

I can't seem to replicate this with the same message on either CZO or upstream main locally 🤔

i am interested to work on this issue can u once elaborate the issue

Hi, So I tried throttling the network speeds on the local dev env to simulate a bad connection (15 Kbit/s, upload and download + 1000ms latency), and I did not notice anything wrong. Could you elaborate more on this?

I was able to successfully reproduce the issue using the Network tab in the developer tools.
I composed a message while on a slow 3G connection and then switched to offline mode and pressed Enter. The outcome was the display of the "Error sending message" banner and the send button became disabled.
I noticed that only pressing the Enter button caused the send button to become disabled, and clicking on the send button did not have the same effect.
It became active again only when I started typing.
https://user-images.githubusercontent.com/96344003/216775539-afda9335-8ad1-46f5-9adb-c113dff64b32.mov

@timabbott, currently, I am unable to reproduce this bug.
However, I have observed that the "send button" remains enabled when the "Error sending message" banner is displayed if the user is offline. Is this behavior intentional, or is it a bug?
https://user-images.githubusercontent.com/96344003/231135468-26208cba-8190-463c-ac18-cb70db4d133d.mov

Hm, if that banner stays around until the compose box is closed (or the banner itself), then it makes sense to have the "Send" button enabled in case the user gets reconnected.


🐛 Bug Report: Basic UI changes

📜 Description
Noticed some basic UI issues which can affect the user experience

The page title contains an emoji 🦖 which seems a little odd
The user profile picture will look much better than the 💻🧑 emoji
Source resources should come after the answer not before the answer
Lack of proper spacing/padding in mobile devices

👟 Reproduction steps
Go to https://app.docsgpt.cloud/
👍 Expected behavior

👎 Actual Behavior with Screenshots

💻 Operating system
Windows
What browsers are you seeing the problem on?
No response
🤖 What development environment are you experiencing this bug on?
Docker
🔒 Did you set the correct environment variables in the right path? List the environment variable names (not values please!)
No response
📃 Provide any additional context for the Bug.
No response
📖 Relevant log output
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🔗 Are you willing to submit PR?
Yes, I am willing to submit a PR!
🧑⚖ Code of Conduct

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

I would like to work on this issue

In not sure that we want to change 3. as it was an intentional change to provide sources while answer is still streaming. In terms of
1 - thats ok
2- I agree
4. Please check current issues to fix mobile things, but yeah I dont remember anyone trying to improve padding.
Thank you! Nice to see you here again!

I myself use docsgpt for my daily use a lot but lol wasn't able to find any good issue and good ones are already assigned to  others. I've been working on web3 and docsgpt doesn't require any WEB3 related work so I'm left with only the frontend part so I'll be happy to contribute on any front-end related part. Also I've been working as a UI/UX designer too so happy to help in other things additional to the hacktober thing.
Also if 2 & 4 issues are needed to be solved you can assign me this

Sorry I wasnt explicit,
I think 1,2,4 are all good!

ok

In a sense that they are needed, sorry if I confused you, why was the issue closed?

sorry i misunderstood

i'll be working on it

@dartpain

Still needs to be resolved @ayaan-qadri

@dartpain Raised PR for this
This is not related DocsGPT but DocsGPT Cloud

The page title contains an emoji 🦖 which seems a little odd

Issue can be closed as per https://github.com/arc53/DocsGPT/pull/1512#issuecomment-2564846773


nice bro, you have made a detection for items with a total of 255, but I have tried it and it is not detected, I have tried using the duplicate item 255 cheat, but the item is not detected

Ahhh.. I'll try to do a few things to make it work better. But keep your expectations low because it's hard to detect those things :(

Don't give up bro,You definitely can,you can improve the plugin at any time, I'm not forcing you, I'm giving advice


Feature Request: Clear History

Two possible functions to delete the history:
Single item: Long press on a item and then "delete" (e.g. a pop-up-dialogue)
All items: A wastebin-button e.g. in the right upper corner
P.S.: Scanning and handling is good so far! Nice first release.

Thanks for the suggestion @Theobald93, will add for next release due to next week.

Included on v1.1.


Is it possible to describe the release via hub release create command ?

Is it possible to describe the release via hub release create command ? I'm using hub to integrate some tasks in my server.

I found out it's possible if you use another line for the description.


[COM] Create an installer

Feature: Develop a Bash Script to install the software used in the project
In the third generation of Lego Mindstorms,
it is possible to install a complete Linux distribution and a complete Oracle JRE.
It is necessary to develop a library to use the hardware compatible with EV3Dev.

    Scenario: Develop an installer for the libraries and configurations.
        Given A brick with EV3Dev installed
        Then It is necessary to create a Bash script to install dependencies
        And Install Java JRE/JDK
        And the BatteryMonitor
        And Install OpenCV
        And Install RXTX

The installer was tested with success with 2 EV3 Bricks. The system allow to copy the JDK from one robot to another to reduce time. Besides exist some optimizations to reduce the time.


Timezone issue

Had an interesting case today. An extract from an Oracle DB being imported into a SQL Server database threw an error:
"The conversion of a datetime2 data type to a datetime data type resulted in an out-of-range value."
This happened on one particular table. Upon inspection there was a column that was type Date in Oracle and Datetime in SQL Server. The SQL Server Datetime type supports values from 1753-01-01 to 9999-01-01.
In the table in question there was a row with a value of 1753-01-01 which should export/import just fine.
However, the export was performed on a machine in a Europe timezone and the import was performed in a USA timezone. Since the JVM was running with the local users timezone, and the Java Timestamps are stored relative to the GMT epoch, when the value was deserialized from the stored value it was shifted to the new timezone which gave it a value of 1752/12/31 and this caused an error when attempted to be inserted into the SQL Server database.
Essentially, I think the tool only works reliably when the JVM timezone is set to GMT/UTC for both import and export. It might work reliably if the import/export machines are in the same timezone, but I'm unclear in my head if Daylight Savings Time could also mess things up depending on when the import/export was performed.
Two suggestions:

Mention this in the documentation as it's easy to miss the consequences, even if you don't hit an error.
Perhaps look at forcing the TZ to UTC by default in code with an override incase someone wants control of the JVM TZ

A simple test case:
Create a table in SQL Server database:
CREATE TABLE TZ_TEST (
	START_DATE datetime NOT NULL
);

Insert a record into it with the earliest possible date value:
INSERT into TZ_TEST (START_DATE) VALUES('1753-01-01')
Export the table after setting the JRE timezone to Europe/Paris
-Duser.timezone=Europe/Paris
Try to import the data with your timezone set to America/New_York
-Duser.timezone=America/New_York
You should get an error message: "The conversion of a datetime2 data type to a datetime data type resulted in an out-of-range value."
For a more subtle version of this effect:
Insert a record into it:
INSERT into TZ_TEST (START_DATE) VALUES('2000-01-01')
Export the table after setting the JRE timezone to Europe/Paris
-Duser.timezone=Europe/Paris
Try to import the data with your timezone set to America/New_York
-Duser.timezone=America/New_York
The record will now show a value of '1999-12-31'

Thank you @athoma22dxc for posting an issue and suggesting a solution. I think that can solved by adding more documentation to the main Readme. The tool can be run with -user.timezone=Europe/Paris parameter to specify source database timezone during import.
Can you please verify that the import works fine with the extra param?
Can you please attach a zip of a sample exported data for reference?
Would you also consider creating a PR with documentation changes? It could add a new section to main readme (named for example: Transferring data between different timezones ) and state that importing data between databases in different timezones can shift the local date times by a difference between timezones. It is therefore recomnmended to set the timezone of source databaze during data import using -user.timezone parameter. An example should bet therein as well, something like:
jdbcimage import -user.timezone=GMT-8:00 url=jdbc:sqlserver://localhost:1433;databaseName=XE -user=sa -password=changeit mysql.zip


Show error when "validator" is failing

Is your feature request related to a problem? Please describe.
Sometimes I am testing things out and the data that I am passing to Superforms is not correct, like the object structure, so when I try to submit the form it just does nothing. No error, just halts from submitting.
Describe the solution you'd like
A console error or something to inform me at least that that the form is not setup correctly.

I'm thinking how to determine whether something went wrong, to avoid false positives. Can you make a MRE here so I can see how a silent error would look like? https://stackblitz.com/edit/sveltekit-superforms-testing?file=src%2Froutes%2F%2Bpage.server.ts,src%2Froutes%2F%2Bpage.svelte

Here is an example:
https://stackblitz.com/edit/sveltekit-superforms-testing-oxddy6?file=src/routes/+page.server.ts
I was thinking about it, and maybe having more diagnostic info in SuperDebug (or an error in the console), would be the best solution.

In this case everything works, it's just that the alias error that is being generated isn't displayed anywhere on the page. It's hard for the program to figure out that this constitutes an error (tracking unused store fields on a svelte page), so I think it's best to just add more stores to SuperDebug to see what happens:
<SuperDebug data={{$form, $errors, $message}} />

Closing this, difficult to make a general solution.


status code on request events

Coverage decreased (-1.6%) to 90.751% when pulling 902a5c6e1fa68cfc22fc349389217259efc928f8 on statuscode into 2e2096503b78f5a1f5235f16a557495cadbd24f8 on master.

Coverage decreased (-1.6%) to 90.751% when pulling 902a5c6e1fa68cfc22fc349389217259efc928f8 on statuscode into 2e2096503b78f5a1f5235f16a557495cadbd24f8 on master.


Disable attributes chaining

Let's say I have a widget in my template formatted like this:
<widget
  template="flexbox"
  class="active-tickets"
  status="active"
  component-template="active"
  time-frame="0"
></widget>

The attributes are broken down into their own lines for readability and clearer diffs. Is there a way to disable inlining/chaining back to a single line them upon formatting?

Hey @ptrwllrd, I have just pushed an update to Twig Language 2 with a brand new formatter using Prettydiff 3 which brings about some major bug fixes. This bug has now been fixed in the newest version of the extension, you will just need to change the settings "Twig-Language-2: Wrap" to 80.


Support configuration for MAS-SSO resources; tune status checks

WHAT
Support customization of MAS-SSO deployment resources
WHY
Support smaller machine pools for less-demanding testing
In support of MGDSTRM-8542

can we add example? for env config for keycloak?


Job Summary

This new feature might avoid the quirkiness of where the test results are attached. You just export the markdown to $GITHUB_STEP_SUMMARY.
Looks like a PM at Github made a project that does this,
https://github.com/test-summary/action

This has been added in #266 and released as v1.35. See #264.
Please move to the latest version or use v1, which always refers to the latest v1.xx release. Please confirm that this is working for you.
Thanks for reporting, though!

Oh neat! Sorry that I missed this change. I’ll give it a spin. 🙂

Yep, I completely missed it!
https://github.com/ben-manes/caffeine/actions/runs/2336457770


Android emulator keeps stopping / crash

After installation everything works fine for iOS but not for Android.
The app on Android emulator keeps stopping.
I have the latest react and react-native version.

It seems like autolinking is not working and manuel linking as well.

Have someone an idea how I can fix it?

I have the same issue!

I have the same issue!

I fixed it for me manually.
Add this code snippet to android/app/build.gradle
implementation project(':react-native-fast-image')

and this one to android/settings.gradle
include ':react-native-fast-image' project(':react-native-fast-image').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-fast-image/android')

I fixed it for me manually.
Add this code snippet to android/app/build.gradle
implementation project(':react-native-fast-image')

and this one to android/settings.gradle
include ':react-native-fast-image' project(':react-native-fast-image').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-fast-image/android')


Add directive for linking symbols as alternate representations

Bug/issue #, if applicable: rdar://109417745
Summary
Adds a directive which is meant to be used in markdown extension files for a specific symbol:
@Metadata {
  @AlternateRepresentation(``MyApp/MyClass/property``)
}

Where all variants of MyApp/MyClass/property will be added as variants of the current symbol.
Its purpose is to be able to define an alternative language representation for a symbol, where the alternative symbol is an unrelated symbol as far as the compiler is aware (i.e. they have different USRs in the symbol graph).
This makes it possible to switch between both symbols through the language switcher:

Whenever possible, the alternative language representations should be defined in-source, by using in-source annotations such as the @objc and @_objcImplementation attributes in Swift, or the NS_SWIFT_NAME macro in Objective C.
However, if this is not possible, this is an alternative to be able to render both symbols as counterparts of each other.
Diagnostics
Links within the directive are resolved, and emit a warning if the link cannot be resolved same as all other markup links:
    warning: 'MissingSymbol' doesn't exist at '/Synonyms'
     --> SynonymSample.docc/SymbolExtension2.md:4:19-4:32
    2 |
    3 | @Metadata {
    4 +     @AlternateRepresentation(``Synonyms/MissingSymbol``)
    5 | }

Duplication of source language availability is also detected:

if the symbol the alternate representation is being defined for (the "original" symbol) was already available in one of the languages the counterpart symbol is available in
if the alternate representations have duplicate source languages in common, i.e. if counterpart1 is available in Objective-C and counterpart2 is also available in Objective-C.

And suggestions will be provided depending on context:

which languages are duplicate
all the languages the symbol is already available in will be available as part of the diagnostic explanation
if the @AlternateRepresentation directive is a duplicate, a suggestion will be made to remove it, with a suitable replacement
if the @AlternateRepresentation directive is a duplicate, a note pointing to the original directive will be added

warning: An alternate representation for Swift already exists
This node is already available in Swift and Objective-C.
SynonymSample.docc/SymbolExtension2.md:4:5: An alternate representation for Swift has already be
en defined by an @AlternateRepresentation directive.
--> SynonymSample.docc/SymbolExtension2.md:5:5-5:57
3 | @Metadata {
4 |     @AlternateRepresentation(``Synonyms/Synonym-5zxmc``)
5 +     @AlternateRepresentation(``Synonyms/Synonym-5zxmc``)
 |     ╰─suggestion: Remove this alternate representation
6 | }
7 |

warning: This node already has a representation in Swift
This node is already available in Swift.
 --> SynonymSample.docc/SynonymExtension.md:5:5-5:56
3 | @Metadata {
4 |     @AlternateRepresentation(``Synonyms/Synonym-1wqxt``)
5 +     @AlternateRepresentation(``Synonyms/OtherSynonym``)
  |     ╰─suggestion: Replace the counterpart link with a node which isn't available in Swift
6 | }
7 |

Dependencies
None.
Testing
Tested by building and rendering the following archive locally, which uses the new directive:
SynonymSample.docc.zip
This contains two symbols which have been configured to be counterparts of each other.
You should be able to switch between both symbols using the language picker from either:
http://localhost:8080/documentation/synonyms/synonym-1wqxt?language=objc
or
http://localhost:8080/documentation/synonyms/synonym-5zxmc
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] Updated documentation if necessary

Before I get into the code, some feedback on this syntax:
@Metadata {
  @Synonym(language: swift) {
     ``MyApp/MyClass/property``
  }
}

It would be structurally simpler if the link was a parameter instead of inner content. That would also simplify the implementation by not needing to validate the type of inner content.

I'm not too happy about the phrase "synonym" for this. In technical terms I tend to refer to this concept as a "counterpart"  or as an "alternate representation".

I wonder if the language parameter is really needed. DocC should know the available languages of the current symbol and—assuming the link resolves—the available languages of the authored counterpart symbol. Assuming that neither symbol already has counterparts, DocC should be able to infer the source language from the resolves counterpart symbol. If on the other hand either symbol already has representations in multiple languages, it raises some interesting questions regarding what this directive should do. For example; should it be possible to author a different Swift representation/counterpart for a symbol that already has a Swift representation/counterpart?

@swift-ci test

@swift-ci please test

@swift-ci please test


Mandatory root fields and strongly typed dates

introduced new mandatory top level fields top_line and summary which are similar to headline. We should either move all three into fields or move all three into the root. Front end decided against the field approach, because "there have to be fallbacks" (.getOrElse(???))

Added strongly editorial dates as strongly typed fields to Metadata, currently they reside in fields as ISO-formatted Strings which is also not "strongly typed"

Added Metadata to WebArticle to give access to the article's strongly typed dates: updated, created, last_edited, last_published and also the technical last_modification

@thisismana guess we need to terraform the articles table to alter the key schema


Feature/enhance navigation

@mauro-oto @etagwerker this adds a basic link navigation to previous weeks. check it out and let me know what you think!

:grinning: !

Thanks!


Move guides out of "Learning more" section

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
https://docs.getdbt.com/docs/get-started/learning-more/using-jinja
https://docs.getdbt.com/docs/get-started/learning-more/refactoring-legacy-sql
What part(s) of the page would you like to see updated?
Content updates
As part of the Docs team's Guides project:

Check to see if links to getting started guides need to be added

Left sidebar updates

Move the Refactoring legacy SQL to dbt guide to the /website/docs/guides/migration/tools folder
Move the Using Jinja guide to the /website/docs/guides/migration/advanced folder
Remove the "Learning more" section
Update links and redirects

Additional information
No response

Moved as part of https://github.com/dbt-labs/docs.getdbt.com/pull/3016


Adding Vortran USB test and dlls

This PR hosts a test example on an attempt to connect to Vortran Lasers through USB rather than RS232.

@AhmetCanSolak  I realized that I had two copies of the photom_gui branch so I re-made the branch with the appropriate files for the last test I did to connect to the Vortran Laser. Let's take a look at the SDK together.

Attemping to solve #151

closing this as we decided to just use COM communication for now.


Bullet alignment and line height inconsistent with iA W

First, THANK YOU for doing this theme. That's the most important part.
Second, I'm experiencing decorator bullet alignment issues on all header and larger font blocks. The • decorators and line heights are off. So while the color palette and typeface port is superb, there's a loss of vertical alignment and rhythm that makes the experience negative.
@perezd was able to fix the decorator alignment with
.items-baseline {
    align-items: self-end;
}

But the vertical spacing and "large dot" thing is going to take some more work

This idea does not work well with multi-line blocks.


189 Add HoldNote objects

Pausing my review until Alex's issues are addressed. I don't think I have anything else to add though, code structure looks fine.

Pressing another input and releasing that same input too early will count as a miss even if user is still holding down the original input they initially pressed on the HoldNote. I think having at least one input held down should prevent a miss, like how you can hold and switch inputs during a slider in osu!.

I tried changing IsBeingHeld to be NumInputsHeld but KeyDownEvent gets called multiple times if you hold down a key (key repeats as if it were a text editor). Need to figure that out.

Pressing another input and releasing that same input too early will count as a miss even if user is still holding down the original input they initially pressed on the HoldNote. I think having at least one input held down should prevent a miss, like how you can hold and switch inputs during a slider in osu!.

Aaron and I discussed switching inputs in the middle of a hold/slider. I think the key you start a hold/slider with should be the same key you release with. This way also seems a bit easier to implement compared to allowing switching inputs in the middle of the hold/slider.


Lossless JPEG Transcoding with JXLCoder: Limited Multi-Core Utilization

I'm using the jxl-coder-swift library for lossless JPEG transcoding with the following code:
let transcoded = try! JXLCoder.transcode(jpegData: Data())
let jpegData: Data = try! JXLCoder.inverse(jxlData: Data())

However, regardless of the number of threads I use to run these functions in parallel, the library does not seem to utilize more than one CPU core. Have you encountered similar limitations?
What steps can I take to improve the performance of transcoding with this library, especially in a multi-threaded environment?

Those methods already utilize libjxl multi-threading.
That means if you're adding more threads then you're just reducing performance by wasting resource on thread scheduling.
At the moment you have to transcode/decode sequentially.
I may accept PR if you prefer to control threading rather than libjxl itself, however I don't think it actually will make things faster, might be slightly more complicated.
Note, if you do so, phones usually have a lot of efficiency cores and small amount of performance cores, that means if you'll try to spawn one thread per one image in pipeline some image decoding will be fully done on efficiency core and this will be also drastically slow.

If so then probably transcoding in libjxl is likely to be single-threaded. Here is clearly their parallel runner is used.
https://github.com/awxkee/jxl-coder-swift/blob/feda1ccd1b19b2e14f9aea40cb13d8e5ae6d0366/Sources/jxlc/JxlTranscode.hpp#L47
Then there is no possible many ways, you can ask authors in libjxl if there is some ways to improve, or transcode many pictures at once, or also might be your software reports incorrect workload status.
Anyway, this library almost do nothing in transcoding/decoding lossless JPEG except calling libjxl APIs.
You can try to play with some magic numbers instead of 'JxlThreadParallelRunnerDefaultNumWorkerThreads' and see if this helps in JxlTranscode.hpp.

I made some simple check, here is quite straightforward transcoding
for i in 0...1000 {
     _ = try! JXLCoder.transcode(jpegData: jpeg)
}

Flamegraph without libjxl thread runner attached

Flamegraph with libjxl thread runner attached

Also, you may note that libjxl internally calls threads for some reason using a thread runner.
And spawning concurrently scheduled transcoding absolutely occupies any available resources.
 DispatchQueue.concurrentPerform(iterations: 300) { _ in
      _ = try! JXLCoder.transcode(jpegData: jpeg)
}

Therefore I conclude that threads used correctly and their use is limited by libjxl itself.

@awxkee I can see, from your side, that all of the cores are used.
I don't know why I can't have the same result on my side... (I'm using Dart to make the calls) could you have a look to see if you see something wrong here: https://github.com/stephane-archer/flutter-jxl-coder/blob/main/macos/Classes/JxlCoderPlugin.swift

I don't know for sure, here some thoughts:

If you'll try to benchmark this code, it will stall on disk reads/writes and any check will be incorrect. You should benchmark memory rather than disk.

let jpegData = try Data(contentsOf: URL(fileURLWithPath: inputPath))
let jxlData = try JXLCoder.transcode(jpegData: jpegData)
try jxlData.write(to: URL(fileURLWithPath: outputPath))

libjxl tries to allocate threads by using std::threads::hardware_concurrency(), check that it returns > 1
If you're using x86 intel mac laptop I might forgot to compile x86 libjxl version with threads... Unlikely I did so but might be
libjxl accidentally might not support threads on x86 mac; this one is also unlikely but might happen
You platform ( Flutter ) might have a limited FFI channel and you probably just can't make transcode calls fast enough to benchmark anything
Because you're checking the workload using Flutter, it may not accurately reflect the CPU's workload, if you're using Flutter tools.
If you're benchmarking memory encoding already sometimes transferring data from your sandbox to FFI might have a limited bandwidth and instead of checking CPU workload you're checking your FFI channel data bandwidth
You're using too small images on too powerful hardware to make notable workload
Benchmark iterations amount is small

Perhaps there is something else that I missed or forgot, but I think some of this points already will be true or many of them at one time.


Pagination hiccup: when you delete the only record on a page, you see an internal server error

Bug
Pagination hiccup: when you delete the only record on a page, you see an internal server error.
Steps to reproduce

Go to the last page of the dataset or publication overview, and make sure there is only one record to be seen:

Delete the record

See internal server error

Expected behaviour
You are directed to a page that has records.
Suggestion:
You go to the adjacent page that has records (current page -1). If there is only 1 page, you remain on that page. If there are no records left at all on that page, you see the blank state on that page.

mm, seems logical.
Deleting a record should reset the page counter back to 1.

It doesn't make sense to go back to page 1, if you're explicitly on the last page to go through some information, you'd have to go back to that page again. It would make sense to go to "current page -1" if there is more than 1 page.
Saying "you're on the wrong page" does not help the audience, we should guide them to a place where they can do things again.


Fix completion item description issue with function rest parameters

Purpose

Fix completion item description issue with function rest parameters

Fixes #18907
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[x] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[x] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #19290 into ballerina-1.0.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.0.x   #19290   +/-   ##
================================================
  Coverage            15.73%   15.73%
================================================
  Files                   47       47
  Lines                 1265     1265
  Branches               197      197
================================================
  Hits                   199      199
  Misses                1053     1053
  Partials                13       13

Continue to review full report at Codecov.

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


Terminal gets messed up and needs restarting after logging back from suspended machine on Ubuntu

VSCode Version: 1.42.1
OS Version: Ubuntu 16.04 LTS

Steps to Reproduce:

Open an internal terminal in VS Code
Suspend machine
Log back in
See terminal being empty or with random chars and cursor placed at some random position. Unable to type in anything

Does this issue occur when all extensions are disabled?: Yes

I am experiencing the same issue with Ubuntu 19.10 and VSCode 1.43.2.

Can you share a screenshot, your settings.json file and let me know whether pressing ctrl++ then ctrl+- fixes the issue?

This is a screenshot of a window (that had three terminals open), after wake from suspend. For some reason, background colors and unicode symbols are rendered. The terminals are all responsive, but I can't see any text I'm typing. Note that typing reset in the terminals does not change anything.
Pressing Ctrl-+ and Ctrl-- indeed fixes the issue.
These are my settings.json.

/duplicate https://github.com/Microsoft/vscode/issues/69665


MultiReactiveSystem throwing errors when upgrading from 1.0.0

I have a MultiReactiveSystem following this wiki guide: https://github.com/sschmid/Entitas-CSharp/wiki/MultiReactiveSystem-Tutorial
This is how it looks like:
IPoolableEntity.cs
using Entitas;
using UnityEngine;

public interface IPoolableEntity : IEntity, IReusable, IView, IPoolable { }

public partial class GameEntity : IPoolableEntity { }
public partial class InputEntity : IPoolableEntity { }
public partial class SoundEntity : IPoolableEntity { }

CleanUpPoolSystem.cs
public class CleanUpPoolSystem :  MultiReactiveSystem<IPoolableEntity, Contexts>
{
  ...
}

Now after upgrading to 1.4.1, jenny gen has generated an IPoolableEntity a huge amount of errors show up:
The namespace '<global namespace>' already contains a definition for 'IPoolableEntity' [Assembly-CSharp]

The type or namespace name `IReusable' could not be found. Are you missing an assembly reference? // Same for IView & IPoolable

looks like you updated Entitas but didn't unzip the new Jenny or ran auto-import.

The new interfaces changed name and end with *Entity now

I see, so IPoolable is now IPoolableEntity?
But what do you suggest I name my previously named IPoolableEntity now?
I followed the lead and changed it to:
public interface ICrossContextPoolableEntity : IEntity, IReusableEntity, IViewEntity, IPoolableEntity { }

public partial class GameEntity : ICrossContextPoolableEntity { }
public partial class InputEntity : ICrossContextPoolableEntity { }
public partial class SoundEntity : ICrossContextPoolableEntity { }

it works now but the name is pretty cumbersome.

IPoolable? :)

Is that actual code from your game? Because, why input and sound is a IViewEntity?
I don't think you need multiple contexts and interfaces for that at all

Oh SoundEntities are mostly GameObjects with AudioSource that are pooled and move around (even attach themselves to other entities), so they have View.
For Input it's a relic from past mistakes, removed ^^'


cleaning MarkDown files

refactor all remaining markdown files and keep it clean

@OlafenwaMoses could you have look?

is there any interest in using a proper style? @OlafenwaMoses

@OlafenwaMoses @OlafenwaMoses  is it a private or community-like repo? so far it looks quite dead...

Great work on this. Kind adapt the changes to the new updates.

@OlafenwaMoses pls see the update ^^

Excellent clean work on the README update. Greatly appreciated.


Laravel 8 support

Please create a tag with laravel 8 support.

The latest release of Ziggy, 0.9.4, supports Laravel 8.


Stock-Market Analysis Using ARIMA

Implemented Time series forecasting using ARIMA to predict Reliance Industries Limited's stock behaviour.
Dataset taken from:https://finance.yahoo.com/quote/RELIANCE.NS/history?p=RELIANCE.NS

@noobkid2411 Really Cool 🙌
Thanks For Contributing 👍

@anubhav201241 I had a small query,we could add more issues regarding some other projects we wish to add right? Or is there any kind of limit on the no. of issues per person?

Nope there is no limit @noobkid2411 Yeah Sure you can add it it will be really great if you do that !!


Clarify distinction between lazy and empty_files in "lazy shuffle" example doc

The "lazy shuffle" example added in #3202 is implemented correctly in terms of code (ie only using empty_files), but the explanation (and even the title) conflate lazy with empty_files. These aren't the same thing in Pachyderm, and afaiu actually using lazy here would be ill-advised. Is there a reason basically all references to "lazy" in that PR shouldn't be changed to "empty files"?
┆Issue is synchronized with this Asana task

Agreed. I’ll correct the example docs to empty files.

On Oct 30, 2018, at 4:44 AM, Gabriel Grant<EMAIL_ADDRESS>wrote:
The "lazy shuffle" example added in #3202 is implemented correctly in terms of code, but the explanation (and even the title) conflate lazy with empty_files. These aren't the same thing, and afaiu actually using lazy here would be ill-advised. Is there a reason basically all references to "lazy" in that PR shouldn't be changed to "empty files"?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

https://pachyderm.atlassian.net/browse/DOC-164


Search bar obstructs UICollectionView (< iOS 11)

When running GitHawk on iOS 10, the search bars cause insetting issues with the collection view below. Assume it must be doing something with the contentInset when the search bar is tapped.

Ack, thanks!

I can only barely reproduce this and its really inconsistent. Wonder if its a simulator-only issue? I don't have any iOS 10 devices on me atm either. Maybe not the highest priority?

Can consistently reproduce this on either a 7 or 5s simulator using 10.1.

Crazy, will-hide is being called before will-show... not sure I understand why. Fixing.


Create documentation to "Get your existing project started with Fides"

Currently, our "Quick Start" documentation delineates how to deploy fidesctl to a local or remote webserver.
Adopters of this project will likely be adding Fides to an existing project, and thus will need some instruction on how to get started with their project. This documentation should outline:

topic
answers the question

depending on project type, where one might want to start annotating in their project
i've got <this type of app>, where should i start annotating?do i start with the database(s)? if so, which one? And how?if i start with system, how do i know it's a system?

how to think about starting annotations from scratch
what does my software project do? for what reasons do I collect PII for this project?where does sensitive data travel to?

who else to engage
if i do this, does my entire team have to do this?do I need company lawyer?who should I get buy in from?how do i get other teams to help or collaborate on annotating our project?

closing all tickets older than a year as not planned


GDScript compilation error with no description in built-in scripts

Tested versions

Reproducible in 4.3 stable

System information
Godot v4.3.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1660 (NVIDIA; <IP_ADDRESS>81) - AMD Ryzen 7 2700X Eight-Core Processor (16 Threads)
Issue description
I'm yet to figure out the context and describe the issue precisely. I've opened my project (previously on 4.2.2) in newly released 4.3 and that gave me a fuzzy compilation error:

Also, this seems to cause this error next:
Invalid call. Nonexistent function 'new' in base 'GDScript'

Also, the editor crashes on exit with no helpful information:

I tried to remove .godot/ folder, but that didn't help.
Context about project setup (something may be irrelevant to the issue, but I don't know what might be actually useful):

I have a custom graph-based (GraphNode, GraphEdit) plugin for the editor. It also allows to configure custom GUI for particular data types via calling a static register_gui() function (which puts that gui in static Dictionary)
configure_ineditor_plugins.gd is an autoload where all those register_gui() are invoked

This autoload is automatically added by another plugin called setup in its _enable_plugin() method. This plugin also provides EditorExportPlugin, but I don't believe this matters.

The class that fails to load on the screenshot is a @tool script (basic GUI implementation provided by plugin) from which other @tool scripts (custom GUIs in editor/ folder, project-specific) inherit. This offending class inherits GraphNode which is experimental, but there are no visible errors in its code.
The error comes from configure_ineditor_plugins.gd, but is caused by the compilation error.

Steps to reproduce
Don't have an MRP yet. This just happens when opening the project.
Minimal reproduction project (MRP)
Don't have one yet, sorry

I'm trying to debug this myself, but am unable to build the engine from sources. Not sure whether it's worth to open a new issue, for now just posting here: https://forum.godotengine.org/t/line-too-long-when-compiling-engine-with-spine-godot-on-windows/92348 . It's been several days already with no answer, so I hope maybe someone can help me with building this to debug the issue

Finally, I've been able to track this down to this line: https://github.com/godotengine/godot/blob/77dcf97d82cbfe4e4615475fa52ca03da645dbd8/modules/gdscript/gdscript_analyzer.cpp#L1300
There was completely no clue about which errors caused the failure, but after debugging the engine I got this error:

I will share a small piece of the relevant code here:

It seems to be related to lambdas. I changed this offending line to reference self.frame (it's a class field) and it compiled OK (though, later when I open a graph, I get a crash with null-reference - have not checked yet whether it's caused by this lambda setup or not).
It's also weird that I don't get any errors for line 140 from this screenshot which looks completely the same to me and should've caused a compile error as well, but it compiled OK.
Ideally, I'd like to see these "other_parser" errors in the output window.
For the particular error - I don't think it is correct (I guess, it's quite possible to capture the frame field within lambda scope - but even if this is not correct way to utilize lambdas, then why it's not telling anything on the other line?)

@AThousandShips can you please help me ping the write person to draw some attention to this as I'm afraid this issue got buried already?

This issue is already tracked for GDScript triage so it is probably not buried


Extremely slow

Compiling simple packages takes literally hours while on my Macbook for example it takes a few minutes. Isn't there a way to make qemu use the full potential & resources of the host?
I don't know anything about it really, so it's more of a question if this is possible.

qemu completely emulates a different architecture so you can run arm binaries on a x86_64 machine. This comes with the price of slow performance. If you need full performance stick to the architecture of your host system

@wouterds as @JSurf said the fact that QEMU emulates a CPU makes it pretty slow. The only way to compile something for ARM at the same speed you compile x86 binaries is to use a cross compiler toolchain.


provide a workaround for the Change Shape bug and a little RE cleanup

Partially addresses #17015
It's just a little unification for the Kitsune Change Shape REs. The Roll Option with both Suboptions is now part of the Ancestry while Heritages an Ancestry Feats enable these suboptions.
This also avoids the bug in #17015, as there are no more duplicate Suboptions.

I'd rather figure out the initial bug than rely on roll options like this.

I would prefer that too. But since I can't fix the bug in the code myself, I wanted to at least offer a workaround for this case. I couldn't find any open issues that also seem to be related to this bug, which would lower the priority of the actual fix. If such help is not desired, please feel free to close the PR.


about downloading ScanNet

Hi,
I haven't used ScanNet before, so I'm not familiar with it and need your help. Due to my limited memory space, I want to only download the needed parts of the ScanNet. Since you say:

Our method relies on the .ply as well as the .labels.ply files.

so does it mean that I can download the ScanNet only using this two command?
download-scannet.py -o [directory in which to download] --type _vh_clean_2.ply
download-scannet.py -o [directory in which to download] --type _vh_clean_2.labels.ply

In addition, how much memory space is needed to store the processed data?

Hi, thank you for your interest.
Yes, you can download the needed files using those two commands. The processed data is about 55Gb.

Wow!! Thanks for your very fast reply. It really helps me!


Demo page is outdated

It seems the demo page currently uses the gh-pages branch, which is outdated. GitHub now lets you deploy Pages from any branch. I would recommend changing the Pages branch to master in Settings->Pages so that the demo site is always up-to-date.

Seems like an excellent suggestion.

Yes, will be nice.


Add thanos ruler route prefix

Signed-off-by: yeya24<EMAIL_ADDRESS>Add routePrefix in spec

Not sure why the vendor check is failing on Travis. LGTM otherwise.

Yes, the CI failure is strange. I didn't change update any dependency and I cannot reproduce this problem in my local environment. Could someone help me re-trigger the Travis CI?

might this be a go version things?

@yeya24 can you rebase this?  Hopefully, the CI failure will also go away after the update.

Got a flaky test here https://travis-ci.org/coreos/prometheus-operator/builds/654476206?utm_source=github_status&utm_medium=notification. Will open a PR to fix it first

@pgier Hello, CI passed. Could you please take a look?


License unclear

Hi, menees, and thank you for this well-thought application. :-)
I wonder about the correct license: While the LICENSE file says it's MIT, the "About" screen says it's charity ware. Which of those applies?

Both apply. The license file is the legal instrument governing the use or redistribution of software. The MIT license is a permissive license that grants use rights, and it can be used for free, commercial, or charityware software.
The MIT license includes the right to "sell copies of the software". Think of charityware as selling the software for some charity (that I can't track and will never know about). So the request on the About box is that if you're going to use the software I provide, then I ask that you donate to the charity of your choice. Legally, you have the right not to do that. However, exercising that right would seem weaselly to me. :-)

From the MIT License itself:

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

Wouldn't that require license headers in the files ("substantial portions")?
Anyway, thank you for clarification. The About box does not really say it's a request, it reads more like "this application requires you to donate to a charity". Maybe you could rephrase it, as in "RpnCalc is dual-licensed as MIT and CharityWare"? Just a suggestion. :-)

I'm happy with it as is.


Numeric Inputs do not display in Text Object Tags

Describe the bug
I want to display a numeric store to my graders to ask if it is correct. I uploaded a csv that contained numeric fields. When trying to display that in the labeling interface, it does not create a div at all. I can see the number in the tabular view and can switch formats between str and num but it does not show up
To Reproduce
Steps to reproduce the behavior:

Upload a csv with at least one text column and one numeric column
Use the following labeling interface code

<View>
  <Text name="text" value="$complete_sentence"/>
  <View style="box-shadow: 2px 2px 5px #999; padding: 20px; margin-top: 2em; border-radius: 5px;">
    <Header value="Is this score correct?"/>
    <Text name="text2" value="$numericScore"/>
    <Choices name="sentiment" toName="text" choice="single" showInLine="true">
      <Choice value="Yes"/>
      <Choice value="No"/>
    </Choices>
  </View>
</View>

Expected behavior
The numeric value should be displayed under the header.
Screenshots
Displayed in the table:

Not displayed on the interface:

Environment (please complete the following information):

AWS Lightsail Container
Docker image  heartexlabs/label-studio:latest
Label Studio Version v1.0.2

@GrayAlex49 please check out the new number tag https://labelstud.io/tags/number.html (in version 1.2.0+)

For anyone looking into simply displaying numeric values which are not strings, and not want to use the Number control widget, I find it more elegant to render it as HyperText:
<HyperText name="text2" value="$numericScore" inline="true"/>

I prefer inline to keep styling consistent.


CVE Warning

version 0.0.23
I'm getting following warning from WhiteSource
🔴 CVE-2022-0155 (CVSS3: 8) Severity: (HIGH)
Component: follow-redirects-1.14.4 (Transitive)
Dependency path: @ibm-cloud/cloudant-0.0.23 >>> ibm-cloud-sdk-core-2.17.4 >>> axios-0.21.4 >>> follow-redirects-1.14.4
Fix: Upgrade to version follow-redirects - v1.14.7

thanks

The<EMAIL_ADDRESS>fix was released less than 24 hours ago. A clean install of<EMAIL_ADDRESS>successfully gets the fixed version as it is within the ranges specified by our package dependencies.
You need to run the command to get your dependencies updated making sure that your package-lock or any cached dependencies aren't making it resolve at an older (unfixed) version.


Add an implementation of pseudo-random generator based on XorShift [HPPC-49]

http://en.wikipedia.org/wiki/Xorshift
Oh, so simple and yet so nice.

Issue: HPPC-49 (migrated from JIRA), created by Dawid Weiss (@dweiss), resolved Mar 11 2011

Comment by Dawid Weiss (@dweiss) (migrated from JIRA)
In trunk.


Please add opus option for audio

please, because I really enjoy using opus for my phone

opus has already been added, it will be available on the next release


安装nvidia闭源驱动后黑屏

安装nvidia闭源驱动后启用独立显卡，登录后黑屏，同样 的xorg配置lightdm能正常使用
Enable discrete graphics card after installing nvidia closed source driver, black screen after login, same xorg configuration lightdm can work properly

Have the same issue here with FreeBSD & x11/nvidia-driver. For some reason Ly works perfectly with graphics/drm-kmod with either an AMD or Intel GPU & SLiM or any other login manager.
In my case I am confronted with a blank screen for ~150 seconds, and finally my desktop environment---as described in my .xinitrc---appears. Real odd, and I am quite miffed as to why it happens.


Create Roshni-pr.md

Hi GSSOC team,
This is the PR i have raised and its not been merged. its level 1 PR.
Could you review and merged it.
@sanjay-kv @MastanSayyad
https://github.com/sanjay-kv/Open-source-Practice

Please mention the pr link not the repo link @Roshni31-s


Add support for ALiBi-style attention bias

Added support for attention bias to flash attention. Bias is added to the result of the first GEMM (qk) before doing softmax.

Can you change the filename to reflect that bias is supported?

@zhanglx13 I renamed the file to just flash-attention.py and then added a description of what features are currently supported.


Set correct aadpodidentity name

Signed-off-by: Edvin Norling<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

What this PR does / why we need it:
aadpodidentity not pointing to correct name in pod definition.
Which issue(s) this PR fixes:
N/A

Fixes #
Special notes for your reviewer:
Checklist

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign cpanato after the PR has been reviewed.
You can assign the PR to them by writing /assign @cpanato in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @NissesSenap! It looks like this is your first PR to falcosecurity/charts 🎉

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: developer-guy
To complete the pull request process, please assign cpanato after the PR has been reviewed.
You can assign the PR to them by writing /assign @cpanato in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: c6eeafed6b3e12ca839099630146ae8b2e3a3568

PTAL @Issif 🙏

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: developer-guy, Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Trojan:Win32/Wacatac.B!ml

Windows 10 Defender detection Trojan:Win32/Wacatac.B!ml

Hi, you are using public version of MsfMania.
It detected since April.

Hi don't recognize the pro version? How much would it cost?

Search about pro version in the README and contact me via mail.


Manila external provisioner

What this PR does / why we need it:
Adds Manila external provisioner
Special notes for your reviewer:
Originated from kubernetes-incubator/external-storage. Some of the advantages are: ability to handle different share types, OpenStack credentials are passed per-StorageClass instead of per-provisioner.
I'll add unit tests in the following commits, I'd just like to have some feedback if the design is ok.
Tested with a Ceph cluster + csi-cephfs and Kubernetes' CephFS mounter.
Release note:

NONE

/cc @dims

cc @tomassedovic @flaper87

/test cloud-provider-openstack-acceptance-test-flexvolume-cinder
/test cloud-provider-openstack-format
/ok-to-test

/assign @dklyle
/assign @hogepodge

@gman0 hey, thanks a bunch for this new plugin.
As a general knowledge question: Why did you pick this approach over building a CSI based driver for Manila (like the Cinder one)? Just curious to know the motivation.

/assign flaper87

@flaper87 Mainly for separation of concerns and less code duplication. The Provisioner interface seems a bit better suited for this use case - only create the share, and let something else take care of the resulting PersistentVolume.
Now, if CSI could accommodate drivers that only doCreateVolume/DeleteVolume (i.e. the Provisioner interface) and then forward the CreateVolumeResponse to another CSI driver, we would get this exact same pattern. I'll ask at the CSI spec repo if that's something worth pursuing and if it's not then yes - in the long run it would be better to do it as csi-cinder does, since all these storage-handling drivers will move to CSI eventually.

/test all

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/approve
/lgtm
Let's get it in and iterate!

Thanks a lot guys!


Question: how does this compare to devbox?

Specifically, this is devbox: https://github.com/jetpack-io/devbox
I'm loving nix (and have started using devbox) and am wondering how the two compare. Thanks!

Hey! I'm working on a FAQ and will post it on the website soon.
Quickly, the main differences are:

devbox uses JSON for specification, while devenv uses Nix. Using a real language brings a lot of power that is delegated to code generation in devbox. Since we have https://nix.dev/tutorials/nix-language I think this is totally worth it.
devbox focuses on containers, while devenv wants to empower local development.
devbox uses code generation, which is extremely suboptimal way to scale this.
We have thousands of Nix developers that are already using different ways to create local dev environments and embracing the power of the community will lead us to supporting pretty much anything.
devenv composes extremely well, for example 0.2 added support for devenv.local.nix.

TLDR: using Nix as a powerful language that made all this possible is far more superior to a JSON format.
As time passes this will become more obvious :)

Thank you - this is perfect! Going to give it a try now <3


Add a basic t480s profile

Things seem to mostly work well (except the trackpad, which I haven't figured out yet).
We do want acpi_call for tlp, though.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/services-fstrim-enable/22430/3


Onboard Rocket as graduated project

Currently in sandbox, Rocket Chip generator is actually a CHIPS Alliance project already in all but formality.

@hcook will help deal with this. Who is the responsible person on the SiFive/UCB side? I would propose we get someone from Berkeley to represent Rocket, and SiFive for Chisel, or vice versa.

@hcook ping!

Done as part of #92


Update urls() class method in order to support django 1.9 urls syntax

This PR updates the urls() class method to support both django 1.6 and django >= 1.9 new urls syntax.

Coverage decreased (-0.2%) to 95.843% when pulling cda8c306d23c802b1fabf6f86bf2800cf7afdac0 on mdsrosa:update-dj-urls-method into a4994eb922c252371449363daae2d5f563b56a33 on toastdriven:master.

What's the possibility of getting this merged?

hey @mdsrosa would you mind rebasing this?

That's no longer required. Thanks for the PR @mdsrosa!


I am not really sure how vim emulation work, I wish to make left cmd as toggle for it

I activated these settings in the karabiner window :

I would assume this would allow me to press left command alone to enter normal mode and then I could  press it alone (if I was in normal mode) to enter insert mode whilst still being able to use left command as normal when pressed in combination with other keys. Is this the case and if not how and can I achieve this behaviour.
Thank you a lot for such a great open source project.

You are right.
It allows you to toggle vim emulation mode by left command key.
I guess other settings are conflicted with it.
If you already changed the physical command key, please disable that setting.

Oh wow, thank everything is working now, I guess I didn't really test it fully.
One remark I have albeit a small, it would really great if the 'status message' would centre the text :

And perhaps even adjust the size of the 'black overlay' to the size of the text to make it less intrusive and still noticeable. But centring the text would be really great, I think.

Thank you, that worked perfectly.


Changed behavior on WebSockets between 2.7.1 and 2.7.2

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
Traefik router is configured with the RedirectScheme middleware with scheme: https and port: 443.
I send an HTTP request with Connection: upgrade and Upgrade: websocket headers.
What did you see instead?
When sending the request, I receive a 302 Found instead of 101 Switching Protocol.
The Location header in the response also contains an ill formatted URL: https://wss://example.com
What version of Traefik are you using?
v2.7.2
What is your environment & configuration?
https://community.traefik.io/t/changed-behavior-on-websockets-between-2-7-1-and-2-7-2/14953
If applicable, please paste the log output in DEBUG level
No response

same issue v2.8.0

Same. Downgrading to 2.7.1 to resolved the issue.

Excellent, been having an issue with my self hosted code-server giving me the following error when going to the URL:
The workbench failed to connect to the server (Error: WebSocket close with status code 1006)
Have been searching for a few days for a solution, have come across this and downgraded Traefik back to 2.7.1 and the issue has gone away.

Closed by #9159

For info, the issue I was having above with getting the websocket 1006 error when connecting to my self hosted code-server instance has been resolved by 2.8.1.
Thank you very much.

I had issues with HomeAssistant and Gotify - both fixed with 2.8.1 👌


:bug: Set worker count to 10 everywhere

Taken from @ncdc's rebase PR https://github.com/kcp-dev/kcp/pull/2945.

This is the root-cause for the deletion failures BTW

Closing & re-opening this PR to migrate it to the new Prow. Please do not be alarmed.

https://github.com/kcp-dev/kcp/pull/3091 replacement


Resample Older Data

Thanks for the software, I want to resample older data, say 20 years, but want to keep latest data say 3 months in its daily format. This would be a great addition if it doesn't exist yet? Gives the benefit of both worlds.
Let me know what I can do to implement this, I can work on it over the weekend.
Best,
Derek

I will close this issue for now (as we got no response from @firmai).


CloudArchitect

Hello,
This is a very nice feature and I  hope you make this feature  available for Windows VM also.
On linux this feature works well with SSH connections. But when we try to connect to the Linux VM using RDP clinet (after configuring xrdp on the VM) using AAD user this feature does not work. How ever we can connect using RDP client to the Linux VM with local linux user. XRDP was installed and configured based on Microsoft blog article (see below).
Install and configure Remote Desktop to connect to a Linux VM in Azure
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/use-remote-desktop
Appreciate your comments on how to connect to Linux using RDP client with AAD user.
Thanks!

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

ID: bb8e2623-1f79-aee3-bd97-8b5f647b6980
Version Independent ID: 6346b54d-cc8d-0b79-a6b8-f9144cfa2459
Content: Log in to a Linux VM with Azure Active Directory credentials
Content Source: articles/virtual-machines/linux/login-using-aad.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the feedback! We are currently investigating and will update you shortly.

@ramagurram AAD login for Linux is still in a preview state and is not configured to work over RDP sessions. It is setup to work over SSH sessions which is why you are unable to get it to work.
The AAD team has requested that any features you would like to see implemented be left on their feedback page
https://feedback.azure.com/forums/169401-azure-active-directory?category_id=166032
I would suggest leaving your feedback for others to see and upvote your idea.

@ramagurram xrdp doesn't support network level authentication. Not sure if this is a must have when working with remote authenticating services, as AAD. You could however try using http://www.freerdp.com/ instead. Couldn't find a specific Azure tutorial for it.. but technically any tutorial out there, for Linux, should work.

@ramagurram .. just ignore me... For AD you could use https://scarygliders.net/2012/05/26/xrdp-authentication-with-active-directory/ .. For AAD .. let's wait for the devs.


Renaming a file deletes the contents of the file

Describe the bug:

Renaming a file deletes the contents of the file
Link to the blitz that caused the error
https://stackblitz.com/edit/nextjs-pztdkr?file=pages%2Fapi%2Fasdf.js
Steps to reproduce:

Go to https://stackblitz.com/fork/nextjs
Open file pages/api/hello.js
Rename the file hello.js to test.js
Notice that all the contents of the file are removed.

Expected behavior:

All file contents should remain unchanged
Version of webcontainer:
Hash: b5c3c7f283d527b6b05ec10dc99d5b2c3d4dec62
Screenshots:

Desktop (please complete the following information):
Browser name  = Chrome
Full version  = 91.0.4472.101
Major version = 91
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.101 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 109708008,
  "usedJSHeapSize": 102472732,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}

Additional context:

Hey there! Thanks for reporting this issue 🙏 . It's indeed a bug.

Thanks for reporting this @kallenkortuem ! We are aware of this issue and it's the same as #199. We're working on it.

We believe this issue should be resolved now after we deployed an update. Feel free to open a new issue if you experience this behaviour again!


A tiny question on a coefficient in training code

Hi Gengshan, I really like this work.
However, when I checked the loss part in your training code (in the drive), I have a small question on 'alpha = 15'. Is it related to the sigma design in the paper, can you explain it a bit in detail? Because the sigma part in the paper (lines below eq.4) is also confused me. Thank you :>.

You are right. It is a misuse of notations in the code and alpha =1/(2*sigma^2).
Fig.6 might help understanding sigma. Smaller sigma (larger alpha) corresponds to gaussian distributions with lower variance in Fig. 6. Sigma approaching zero corresponds to the standard one-hot labels.


Add support for class attribute checking [google]

This is a feature request to add a check for class level attribute documentation.
As described in the google style guide:

If your class has public attributes, they should be documented here in an Attributes section and follow the same formatting as a function’s Args section.

class SampleClass:
    """Summary of class here.

    Longer class information....
    Longer class information....

    Attributes:
        likes_spam: A boolean indicating if we like SPAM or not.
        eggs: An integer count of the eggs we have laid.
    """

    def __init__(self, likes_spam=False):
        """Inits SampleClass with blah."""
        self.likes_spam = likes_spam
        self.eggs = 0

    def public_method(self):
        """Performs operation blah."""

I don't actually maintain pydocstyle so take what I say with a grain of salt.
I think this might not be entirely tenable to do. Most attributes are created in __init__ but many __init__ methods aren't so simple as your example. This would require some amount of branch analysis, etc. Alternatively, instantiating the class would need to happen in order to find public attributes etc. Some attributes are publicly declared in other ways too, e.g.,
class AnotherClass:
    hates_spam = True
# Alternatively
import attr
@attr.s
class YetAnotherClass:
    hates_spam = attr.ib(default=True)

So this would have to be able to take all of those into account. It's not impossible, but it's also not trivial


Filter interface and implementation should be moved to dispatcher package

The reason is to avoid receiver to depend on future additional filter implementations (e.g. the cesql impl, juel, ...)

/assign


revert

Coverage decreased (-0.02%) to 84.119% when pulling 87d94362ead2e58c80688e0acb69a34f6ed5fe13 on StefanBranch into ec3a212b4226bfb93b04b98da892040ccd2b68d0 on master.

Coverage decreased (-0.02%) to 84.119% when pulling 87d94362ead2e58c80688e0acb69a34f6ed5fe13 on StefanBranch into ec3a212b4226bfb93b04b98da892040ccd2b68d0 on master.


Rename wordpress folder

Operating System: macOS 10.13
Vagrant Version: 2
Virtual Box Version: 5.2
Hey, I'm wondering if it's possible to rename the 'wordpress' folder in the site.yml or Vagrant file.

Hi @jordanbommelje.
It's possible to rename synced_folder of site.yml in your project.
For example,
synced_folder: localroot etc... 😃


better default dub.sdl with friendly comments

Currently the default seems to build a library.  And there are some other choices for default that might be better.  So at least put targetType "executable" in, but there are some other values you might want set.
People also learn much more easily from copying an example and changing it than reading a manual (in the beginning).
How about having most of the normal choices for options in there and commented out with explanations about what they do?
Laeeth.

The default behavior is actually to automatically infer if the package can be built as an executable, as a library, or both (target type "autodetect"). It does so based on the existence of either of "source/app.d"/"source/main.d"/"src/app.d"/"src/main.d", so it requires the project to have a well known structure in that regard. But due to this, setting an explicit target type is usually rather rare.
With a bunch of placeholder comments, I'm just a bit worried to lose the minimalism of package recipes that was one of the initial goals. But small selection of common directives may in fact be a  good idea. What about this:
name "dubtesting"
description "A minimal D application."
copyright "Copyright © 2015, Somebody"
//license "MIT" // a license field is required to publish on code.dlang.org
//homepage "http://example.org" // allows to advertise a project homepage (not the github repository)
authors "Somebody"

//dependency "somepackage" version="~>1.0.0"
//dependency "anotherpackage" version="~>2.0"
//targetType "executable" // force building an executable

// See http://code.dlang.org/package-format?lang=sdl for a complete file format reference

The dependency part is debatable, because dub init dep1 dep2 ... can already be used to add actual dependencies to the generated file. But the fact that the question about how to add multiple dependencies just came up on the newsgroup, makes it appear to still be worthwhile.

ah, but quite often - especially when factoring out pieces of a larger base or if you are making a dub for someone else's work the code containing main may not be called app.d or main.d  (I guess it wouldn't be difficult to look for main() if those files aren't there).
if you happen to have a binary around and then it builds a library by mistake then it can be very puzzling why your program doesn't seem to change behaviour when you recompile it!  (it's quite frequent for me not to have app.d and main.d)
re minimalism, I see your point.  you could add dub init --sample to create a full recipe with examples showing various options.  (something like that). (and then put this on the dub init --help page).

also dub --faq. eg help! vibe has stolen my main, how do I get it back (VibeCustomMain).  how do I add dependencies.  dub is filling up my appdata.  etc


(profile::ccs::home) handle EL8+ login.defs home dir perms

To make new home directories world-readable by default in EL8+, it is necessary to modify HOME_MODE in /etc/login.defs.

This is on hold pending further discussion.

OK.
If you want to keep them private (by default) for some security purposes, we'll live with it, but otherwise historically the CCS team's position was that we are all in this together, and these systems are designed for shared work towards common goals, not private silos.

PS. the existing code that makes them public on CentOS 7 hosts has been in place for 4 years.
The changes in this pull request are because the syntax for doing that changed in RHEL8.

So, any thoughts/discussion about this?

No "further discussion" of any kind has happened in six months. :man_shrugging:


Add more tests

Additional tests for helpers/package-json and helpers/version - both at 100%
Added tests for api/npm - 100%
Added tests for api/github ~98% coverage

This gets total coverage up to around 50%.
Snapshots
I have used Jests snapshot feature here quite a lot as I have found it helps keep the test code clean and readable while also making very thorough expectations. The file test/api/__snapshots__/github.spec.js.snap is generated by Jest the first time the tests are run and then used for all subsequent test runs to check the output. Snapshots can be re-generated (e.g. if something changes) using jest -u.

Codecov Report

Merging #98 into master will increase coverage by 19.51%.

@@             Coverage Diff             @@
##           master      #98       +/-   ##
===========================================
+ Coverage   31.16%   50.67%   +19.51%
===========================================
  Files          16       16
  Lines         815      811        -4
  Branches      129      128        -1
===========================================
+ Hits          254      411      +157
+ Misses        561      400      -161

Impacted Files
Coverage Δ

lib/api/github.js
98.63% <100%> (+95.99%)
:white_check_mark:

lib/helpers/package-json.js
100% <ø> (+5.88%)
:white_check_mark:

lib/helpers/versions.js
100% <ø> (+7.84%)
:white_check_mark:

lib/api/npm.js
100% <ø> (+66.66%)
:white_check_mark:

Continue to review full report at Codecov.

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

Thanks @jonbretman !
Looks good to merge from mobile. Will review from computer later and likely merge.

Merged. Thanks again @jonbretman


Package Failed   <<< skiros2_task

When I build the workspace the package skiros2_task fails.
---
Failed   <<< skiros2_task [0.39s, exited with code 1]
Finished <<< flexbe_onboard [1.03s]
Starting >>> flexbe_widget
Finished <<< ros_bt_py [1.76s]
Finished <<< flexbe_widget [0.33s]
Starting >>> flexbe_behavior_engine
Finished <<< flexbe_behavior_engine [0.09s]
Starting >>> pyrobosim_flexbe_behaviors
Finished <<< skiros2 [1.06s]
Starting >>> skiros2_std_reasoners
Starting >>> skiros2_std_skills
Finished <<< skiros2_gui [1.07s]
Finished <<< pyrobosim_flexbe_behaviors [0.59s]
Finished <<< skiros2_std_reasoners [0.91s]
Finished <<< skiros2_std_skills [0.93s]
Starting >>> skiros2_pyrobosim_lib
Finished <<< skiros2_pyrobosim_lib [0.32s]

Summary: 46 packages finished [7.67s]
  1 package failed: skiros2_task
  1 package had stderr output: skiros2_task
Command '['/usr/bin/python3', '-c', 'import sys;from contextlib import suppress;exec("with suppress(ImportError):    from setuptools.extern.packaging.specifiers    import SpecifierSet");exec("with suppress(ImportError):    from packaging.specifiers import SpecifierSet");from distutils.core import run_setup;dist = run_setup(    \'setup.py\', script_args=(\'--dry-run\',), stop_after=\'config\');skip_keys = (\'cmdclass\', \'distclass\', \'ext_modules\', \'metadata\');data = {    key: value for key, value in dist.__dict__.items()     if (        not key.startswith(\'_\') and         not callable(value) and         key not in skip_keys and         key not in dist.display_option_names    )};data[\'metadata\'] = {    k: v for k, v in dist.metadata.__dict__.items()     if k not in (\'license_files\', \'provides_extras\')};sys.stdout.buffer.write(repr(data).encode(\'utf-8\'))']' returned non-zero exit status 1.

This should have been resolved by recent updates to SkiROS2. Could you do
git submodule update --init

And build again?

 Feel free to re-open in case there's a similar issue.


solariot errors when run

I have installed this repository on Ubuntu desktop and followed the install procedure as outlined in the readme. Everything seems to be installed correct and I have created my own inverter model file SG10RS using the provided SG10rt file but with applicable modbus addresses plus made the config.py file with mqtt settings When the solarart.py is run from terminal I get the following errors
Sungrow@Ubuntu:~/solariot$ ./solariot.py
Traceback (most recent call last):
File "/home/Sungrow/solariot/./solariot.py", line 127, in
mqtt_client = mqtt.Client(getattr(config, "mqtt_client_name", "pv_data"))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/Sungrow/.local/lib/python3.12/site-packages/paho/mqtt/client.py", line 772, in init
raise ValueError(
ValueError: Unsupported callback API version: version 2.0 added a callback_api_version, see docs/migrations.rst for details
As I am no way skilled at python could you maybe give me some pointers as to what could be the possible issue
Thanks

Currently looking to use SolarIot, wondering if you managed to fix the issue?

If you are lazy like me, you can downgrade paho-mqtt,
pip install "paho-mqtt<2.0"
It looks like it was a breaking change in 2.0
https://eclipse.dev/paho/files/paho.mqtt.python/html/migrations.html


drag and drop makes Files crash

Description
When I lauch the Files explorer from Git bash and then try to drag and drop, Files crashes and closes.
Steps To Reproduce
open a git bash terminal, navigate to any folder and open files using the command ``start .`
Then try to drag a folder, Files crashes at this moment.
Requirements

If I start Files from elsewhere, the drag and drop function doesn't make the app crash
Using Windows 11 Pro, version 22H2, build: 22621.1265 with Windows Feature Experience Pack 1000.22638.1000.0

Files Version
<IP_ADDRESS>
Windows Version
Windows 11 Pro 22H2 22621.1265 Windows Feature Experience Pack 1000.22638.1000.0
Log file
debug.log

Is this still an issue in the preview version? https://files.community/download/preview

@antho686

Trying it right away

So this issue isn't happening anymore with the new version, thanks a lot!


DO NOT MERGE - Update to Cicero v0.11.x

Signed-off-by: Dan Selman<EMAIL_ADDRESS>

@jeromesimeon strange failures here after updating versions:
[Error Executing Template] "Can't find variable: init"
System error. Unknown language: cicero

Template studio is using the old ergo engine which is now completely gone. It has to be replaced by the new engine, which among other things requires to create an engine object (not static anymore). Most of the way execution is called in template studio is in Util.js.
@irmerk has seen that code so may be able to help.

If I understand what you're referring to, Line 25 in template-studio/src/TemplateStudio/Utils.js should be changed from
import Ergo from<EMAIL_ADDRESS>
to
import Ergo from<EMAIL_ADDRESS>

@irmerk
Not quite. You will need to use the new “Engine” class should be instantiated with e.g. const engine = new Engine(‘cicero’) and call engine.something(...). Also you should be able to get the logic from the template structure now.
This might be a not so trivial refactor.

Closing this PR, moving to a new branch caught up with master.


update README.md

Add Awesome-docker list to docker resources. https://github.com/veggiemonk/awesome-docker
Signed-off-by: Julien Bisconti<EMAIL_ADDRESS>

thanks for suggesting this!
@moxiegirl wdyt? I like that listing, but don't know if there are "competing" lists that also want to be added here when this is added

@veggiemonk Good idea!  @thaJeztah I think it would be great to have a our own database that people can add to and vote up of these things.  (Another thing on my long  "when I have time"  lists.)  Seeing as how that ambition is currently thwarted, let's do this. If it gets out of hand or something we can back it down.
Tho feel free to ping maintainers of course TMMV.

Hi everybody,
It would be amazing to have a place to post docker related projects. That's
the only reason I made this pr. Since that's what people are already doing
qith this listing anyway. Maybe you could fork this list into docker
organization. I could manage it for you if you want. It would be nice. What
do you think?

Julien Bisconti
On Aug 12, 2015 7:30 PM, "Sebastiaan van Stijn"<EMAIL_ADDRESS>wrote:

thanks for suggesting this!
@moxiegirl https://github.com/moxiegirl wdyt? I like that listing, but
don't know if there are "competing" lists that also want to be added here
when this is added
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/15524#issuecomment-130383910.

LGTM to add this in the mean time.  @veggiemonk We can review additions changes to this area as they occur

Sorry for the late response, I meant to write a response yesterday at work, but we had some major issues on a project :blush:
I think linking is good; I checked your project and it already has quite a number of stars (I'm not sure those are preserved if the project is moved?)
Also, if other similar lists want a mention, we would still be able to do so, probably changing the header of this section.
So, LGTM. Thanks for contributing and for maintaining this list, it's a really nice resource!


Renamed and removed fields in scm.Repository

Renamed scm.Repository.WebURL field to HTMLURL
Removed SSHURL because it is not used
Removed HTTPURL field since is same as HTMLURL except .git

Fixes #654.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-4 of 8 (50.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

scm/fake.go
0
1
0.0%

scm/github.go
0
1
0.0%

scm/gitlab.go
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls

Codecov Report

Merging #689 (91e6370) into master (6db5bf8) will increase coverage by 0.01%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #689      +/-   ##
==========================================
+ Coverage   23.32%   23.33%   +0.01%
==========================================
  Files          71       71
  Lines       10334    10326       -8
==========================================
  Hits         2410     2410
+ Misses       7598     7590       -8
  Partials      326      326

Flag
Coverage Δ

unittests
23.33% <50.00%> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

scm/fake.go
0.00% <0.00%> (ø)

scm/github.go
0.00% <0.00%> (ø)

scm/gitlab.go
0.00% <0.00%> (ø)

scm/scm.go
0.00% <ø> (ø)

web/courses.go
53.20% <100.00%> (ø)

web/courses_new.go
56.00% <100.00%> (ø)

web/scm_helpers.go
55.78% <100.00%> (ø)

Continue to review full report at Codecov.

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


Fix Broken Link

Updated hyperlink to correct MS Docs page

I concur with the suggested changes. Even if the pages are only available in English (en-us) at this time, it is usually a good idea to keep the links language agnostic, just in case some of them are (or will become) available in other languages.

@mikeamoss As @illfated stated, please add a commit that removes the /en-us/ from the hyperlink.  We localize into the browser language.

Also @mikeamoss , if you are using an external git tool to manage your branches & content, you can make the necessary edits and add them to your PR branch. On the other hand, if that is not the case, please use the [Commit changes] feature within the suggestions above.

@ItzLevvie @mikeamoss  I believe since you added the change, you must use the Commit suggestion button for each of your suggestions. I can't merge anything until your suggestions are part of the actual PR. This must be done in the next 30 days or it will become necessary to close this PR.

@mikeamoss : It looks like you have not been active on GitHub since November 19, 2019.
Please keep us informed if you are available to follow up on the suggested changes.

@illfated  Closing this item. Please open a new PR so we can go ahead and accept these changes.

Very well, PR #3916 provided (as directly above).

@illfated  I believe all of these have now been merged. Please let me know if you notice anything missing or unmerged

Thanks, I just checked the content in Update Link #3467, the file identity/ad-ds/reference/virtual-dc/Virtualized-Domain-Controller-Cloning-Test-Guidance-for-Application-Vendors.md still contains 4 old-style technet links. Only 1 of them was corrected in PR #3467, so I need slightly more time than I spent on the other PRs to complete a working solution. Anyway, I will ping/CC you in the PR as soon as it is ready for review.

Oh and by the way: I may have mixed up the PRs #3466 and #3467 regarding content and the necessary work to complete the solution. Anyway, I will keep you posted.

Yes, I see it now. PR #3467 was resolved already when it got closed. I will look up and link the relevant commit(s) to #3467 to make it more readable for anyone who comes across that PR via web search or otherwise. That means only #3466 remains to resolve by providing a new PR.


Svelte table filtering issue

Describe the bug
I ported the "React Example: Filters" (https://tanstack.com/table/v8/docs/examples/react/filters) to svelte and I cannot get it working.
When I enter anything into the inputs nothing happens.
Your minimal, reproducible example
https://codesandbox.io/p/github/gusi1994/tanstack-table-filter-svelte/main?layout=%257B%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522rootPanelGroup%2522%253A%257B%2522direction%2522%253A%2522horizontal%2522%252C%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522id%2522%253A%2522ROOT_LAYOUT%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522panelType%2522%253A%2522TABS%2522%252C%2522id%2522%253A%2522clhrhl1a8000a3b6k79f85p6m%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%252C%2522tabbedPanels%2522%253A%257B%2522clhrhl1a8000a3b6k79f85p6m%2522%253A%257B%2522id%2522%253A%2522clhrhl1a8000a3b6k79f85p6m%2522%252C%2522activeTabId%2522%253A%2522clhrhvws300ny3b6km8z3q628%2522%252C%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clhrhl1a700093b6kbmuc6lun%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522FILE%2522%252C%2522filepath%2522%253A%2522%252FREADME.md%2522%252C%2522state%2522%253A%2522IDLE%2522%257D%252C%257B%2522id%2522%253A%2522clhrhvnlj00ex3b6k5r9iggi6%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522SYSTEM_METRICS%2522%257D%252C%257B%2522id%2522%253A%2522clhrhvtbq00j53b6k9z1gvppp%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522TASK_LOG%2522%252C%2522taskId%2522%253A%2522dev%2522%257D%252C%257B%2522type%2522%253A%2522TASK_PORT%2522%252C%2522taskId%2522%253A%2522dev%2522%252C%2522port%2522%253A5173%252C%2522id%2522%253A%2522clhrhvws300ny3b6km8z3q628%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522path%2522%253A%2522%252F%2522%257D%255D%257D%257D%252C%2522showDevtools%2522%253Atrue%252C%2522showSidebar%2522%253Atrue%252C%2522sidebarPanelSize%2522%253A15%257D
Steps to reproduce

Enter something into any input element.
See that nothing happens.

Expected behavior
I expect that the output gets filtered according to the input.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
mac OS
Chrome
Edge
Firefox
Safari
react-table version
8.9.1
TypeScript version
5.0.0
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Workaround
let columnFilters = $state<ColumnFiltersState>([]);

const options: TableOptions<TData> = {
	state: {
		get columnFilters() {
			return columnFilters;
		},
	},
	onColumnFiltersChange: updaterOrValue => {
		if (updaterOrValue instanceof Function) {
			columnFilters = updaterOrValue(columnFilters)
		} else {
			columnFilters = updaterOrValue;
		}
	},
};


Fixes #12: Added support for hass >= 0.92

Fixes #12: Added a manifest that automatically pulls WideQ from the master branch (ideally this would be in PyPy instead) and renamed directories to match with the new custom component format in 0.92.
Updated the README to include the simplified installation instructions.

This is great; thank you!! I didn't quite get from the blog post linked from #12 that the manifests could express dependencies like this. Much better!
And yeah, someday we should put wideq itself on PyPI.


docs(readme) 修正文档

Pull Request Test Coverage Report for Build 10

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 9:
0.0%

Covered Lines:
98

Relevant Lines:
98

💛  - Coveralls


Impossible to call methods from a reference to Model

A reference to the Model class is needed for plugins to be able to query the model.
Can be fixed by changing all methods to take a -mutable- reference to self instead of requiring ownership.

Solved in #66


Changed the AzureChinaCloud's mirror repo to Tencent's

AzureChinaCloud's mirror repo is now unreachable , so i changed it to Tencent's.
This new mirror repo is quite reliable.
Signed-off-by: D3-3109<EMAIL_ADDRESS>

I'm sorry i created twice meaningless pull request.
my fault.


External memory cache files are not updated between runs

Encountered this while trying to test external memory training for this issue on the discussion board.
I was trying to train on RCV1 which by originally has the +1/-1 labels, causing XGB to crash with an error.
I fixed this and pointed to the new file, however the cache files where unchanged, so when I tried to train I got the same error.
This is because the cache files had already been created from my previous run, and when I ran the algo again, they were not generated again. I assume this is expected behavior, but thought we might want to mention that in the tutorial or something to avoid having users train on stale data (or even the wrong dataset) in case they re-use the same cache file names.
Meaning that once you run:
dtrain = xgb.DMatrix("data/rcv1/data.test#dtrain.cache")
then running
dtrain = xgb.DMatrix("data/another-dataset/data.test#dtrain.cache") on a separate run will result in training on the old (RCV1) data unless the dtrain.cache files are deleted.
Are the external memory cache files supposed to be removed automatically at training end?

@thvasilo After digging a lot into XGBoost,  I'm still not familiar with external memory ... I will try having a complete understanding of it before next release hits.

I ran into this problem also when I was playing with external memory. You have to manually delete the cache files between runs, otherwise it will silently continue to use previous cache files and produce incorrect results with no warning (!!).
I can see two ways of solving this:

generate unique cache files for each run
cleanup cache files in the dmatrix destructor

Option two is preferred so we don't fill up the systems memory. We should be very careful when deleting files though.

This is a "feature" to avoid generating the cache files over and over as they are pretty expensive to write out for large datasets. It's this line here: https://github.com/dmlc/xgboost/blob/095de3bf5f9febba9908637197873630021c733a/src/data/data.cc#L265
We can get rid of that line, but I'm not sure if existing users might expect that caching behavior. A few possible solutions:

Don't cache the row pages between runs.
Add some configuration parameter to keep/delete caches.
Somehow be smart and make sure the source data files haven't changed. For example, maybe we can keep some kind of hash of the source data in MetaInfo, or check the timestamps of source data and cache files.

My vote is 1. What do you guys think?
@thvasilo @trivialfis @RAMitchell

Is option 1 to delete the cache files at the end of training?
Is it possible to create cache files that are named after the input
dataset, ideally under the same directory?
--
Sent from a mobile device. May contain autocorrect errors.
On Mon, Oct 14, 2019, 18:24 Rong Ou<EMAIL_ADDRESS>wrote:

This is a "feature" to avoid generating the cache files over and over as
they are pretty expensive to write out for large datasets. It's this line
here:
https://github.com/dmlc/xgboost/blob/095de3bf5f9febba9908637197873630021c733a/src/data/data.cc#L265
We can get rid of that line, but I'm not sure if existing users might
expect that caching behavior. A few possible solutions:

Don't cache the row pages between runs.
Add some configuration parameter to keep/delete caches.
Somehow be smart and make sure the source data files haven't
changed. For example, maybe we can keep some kind of hash of the source
data in MetaInfo, or check the timestamps of source data and cache
files.

My vote is 1. What do you guys think?
@thvasilo https://github.com/thvasilo @trivialfis
https://github.com/trivialfis @RAMitchell
https://github.com/RAMitchell
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/xgboost/issues/4237?email_source=notifications&email_token=ACFBHIY4GFY32VW2GK2ZL3LQOSMKLA5CNFSM4G4WJ2EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBFN4KI#issuecomment-541777449,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACFBHIZWSTWUC6DOYYGRJDTQOSMKLANCNFSM4G4WJ2EA
.

I came across this too some time ago. What kind of performance penalty will be talking about to regenerate the cache files vs. not?
I think current behaviour to keep the files is quite unusual. I would expect most apps that I use to clean up temporary files and not leave a bunch of potentially large files around.
Deleting them seems the way to go.

If we simply delete that line I mentioned, the files will be left on disk after training, but next time you train again they'll be regenerated.
With the current behavior, you can create different cache files by specifying a different path, e.g.

dataset1.libsvm#cache1
dataset2.libsvm#cache2
...

@sriramch might have some numbers on how long it takes to write out a large dataset.

in part, it depends on where the cache files are being written to. if it is being written to tmpfs, it is faster compared to writing them to a spinning disk. that said, from what i could dig from my old notes, it was ~ 27s (to generate the cache files) for the higgs dataset (near dense), while it was ~20s to generate the cache file for the mortgage set (which is quite sparse) for ~ 140m rows.

Did we come up with a consensus here? I would like to delete cache files at the end of each run and remove the CacheExist line.

Sounds good to me.