Large bundle size

This library comes at a whopping 113.1kB minified https://bundlephobia.com/result?p=<EMAIL_ADDRESS>That is not very good for just showing a form on a website.

info Direct dependencies
└─<EMAIL_ADDRESS>info All dependencies
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>
That's a lot of dependencies.
My advice, drop css-in-js, just use css. Css-in-jss is not well suited for libraries unless both the library and the app consuming the library use the same css-in-js solution.
Second, have 2 versions of the lib, a vanilla one without preact and a react one that's a wrapper on top of the vanilla one, since react it's the most popular library.

Hey @bogdansoare thanks for reaching out! Long story short, you're absolutely right, the bundle is too big for what the library does today. This also loads another bundle once the Typeform shows up that it's bigger, and it's impacting user experience the most, so we're focusing on that one first. After that, we will also check how we can shave some size here.
In the meantime, I'm going to leave the issue open to keep collecting feedback, and as a channel for the improvements we do related to this issue.

Hey @Gotusso thank you for the response, glad to hear that you're working on it.
I really like typeform and would be happy if both developers and users will have a nice experience using this library.
All the best.

As of now, we reduced the bundle size by 27%. In the nearest future, we'll also be optimizing a bunch of other stuff, so the bundle will become even tinier.

:tada: This issue has been resolved in version 0.21.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

I'm struggling with the same issue here. The bundle size as of date is still very large.
https://bundlephobia.com/result?p=<EMAIL_ADDRESS>I wanted to embed a TypeForm in a Vue app, but seems that I now need to ship my app with both Vue and React in order for me to achieve this.
Any tips in the meantime?

@jellevandevelde I am afraid that at the moment you will have to ship the app with both React and Vue, since our embed library is not yet decoupled from the react dependency.

I think it would make sense to at least unpin the react, react-dom versions and move the declaration to peerDependencies. Currently two react versions will get bundled unless the user of the library sets the app's react version to the exact same version as @typeform/embed uses.

@raurfang I think you are right. I am adding this to our backlog. If you would like to open a pull request to get this done earlier we would appreciate it.

Sorry, I couldn't find time earlier, thanks @osdiab @mathio for the work on this. Our bundle size is down by ~40kB thanks to #88.
I'll take a look if this can be improved further in a couple of days.

Thank you for the effort made on this issue already. I have to admit I was very surprised to see React appear in my VueJS application, all that for a Typeform popup button.
I'll be removing Typeform for now.

Thank you for the effort made on this issue already. I have to admit I was very surprised to see React appear in my VueJS application, all that for a Typeform popup button.
I'll be removing Typeform for now.

Agreed, should at least use preact.

Agreed, should at least use preact.

For anyone dealing with this issue, you can use preact/compat as a drop-in replacement for the react dependency. This can help shave off ~100 KB.

Lib version >= 1 has drastically lower file size. It is up to 90% smaller and does not have React dependency.
I suggest you upgrade to latest version if you can.

Thank you!


ChroniquesOublieesContemporain Version 3.16.1 (bugfix)

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Fixed the names of the roll buttons for INT, WIS and CHA checks on the NPC sheet

Character Sheet Info Roll20 Internal Use only.


Fix Dynamic Resource Providers Docs

Description
I found it really deceptive that these docs say its supported in all languages and show code examples in them all but then the chooser is blank and we dont actually support it in all languages.

Checklist:

[ ] I have reviewed the style guide.
[ ] If blogging, I have reviewed the blogging guide.
[ ] I have manually confirmed that all new links work.
[ ] I added aliases (i.e., redirects) for all filename changes.
[ ] If making css changes, I rebuilt the bundle.

Hey @meagancojocar  sorry for being behind; I'm looking into where our PR review links went for these and will approve shortly


Adding 'Adafruit_SSD1306_EMULATOR'

A required library for an open-source emulator I'm about to release :)

Hello @sam-peach. I see you have fixed the error reported by the bot. However, Library Manager works exclusively from Git tags, so the bot won't be able to see the fix you made until you create a new release or tag in the library repository.
If you look at the repository contents at the latest tag, you can see that the version the bot is checking still has the problem:
https://github.com/sam-peach/Adafruit_SSD1306_EMULATOR/blob/2.5.1/library.properties#L1
name=Adafruit SSD1306

Remember to update the version field in the library's library.properties file as needed before making a release/tag.
Once you have done that, simply mention @ArduinoBot in a comment here to trigger it to run the automated checks again. After those are passing, your library will automatically be added to Library Manager.
If you have any questions or concerns, please let me know.
Regards, Per

@ArduinoBot

@per1234 thank you so much for the help! All set now :)


Health check: check ServerLocator- and ConnectionFactory-beans created outside the extension

This is a follow-up of #36.
As of now, ServerLocator- and ConnectionFactory-beans created outside the extension. In general. however, this should be possible by:

getting all beans implementing ServerLocator- or ConnectionFactory-beans,
filtering-out the ones already handled by the extension, and
add the beans to the health check.
We will attribute those beans created outside the extension as external bean for the following disucssion.

Challenges:

We need to generate sensible names for the external beans. We cannot reliably detect scope-annotations since there is a multitude of annotations that could be used, including user-defined scopes.
There might be an issuse with "duplicate" beans. IIRC, the quarkus-pooled-jms extension wraps already-existing connection factories into a wrapper-factory for pooling. Thus, we have - on a technical level - two ConnectionFactory-beans representing the same connection.

What if we would ask our users to add an annotation to beans created outside the extension they want to have included in the service health check. They can provide a name in the annotation. I think it is very difficult to determine in the extension what the purpose is of the beans with said type. The application can make this judgement far better.

I'd vote for @Identifier in this case.

@zhfeng I took another look at the pooled-jms extension. I think the concern I had wrt. connection factories created twice is not correct. Can you confirm?

@middagj  @zhfeng So if my assumption wrt. quarkus-pooled-jms is correct and we decide to use @Identifier(...) as annotation to use for externally configured beans, I have an implementation ready. Just say the word, and I will open a MR.

The more I think about this, the more I have a problem with this approach. Readiness-probes control whether the pod should be taken out of the load balancer. With the current approach, we enforce on the user that each server locator / connection factory is automatically added to the readiness check.

Using the health check for a readiness probe is a decision beyond our control. Usually a readiness check needs 3 failing checks. Apart from that, health check only indicate if a new connection can be created, (or a pooled connection still work), it doesn't say if current connections work.
But I agree, there should be an option to disable beans from a health check. Is something similar already implemented for DataSources?

Careful: we add readiness-checks, not health-checks (afaik, all quarkus extensions register readiness-checks). The number of failing readiness-checks is also configurable before the service is regarded as "not ready". But assuming thath /q/health/ready is registered as a k8s/OpenShift readiness check is quite sensible, and we should plan for this.

Yes, we should definitely think about the use case and plan, but at least for k8s, you still need to register the check yourself.

Given that most people will streamline the generation of services, I recon that /q/health/ready is registered in some base (at least this is what one of my cusotmer does). I have introduced a runtime-variable that controls whether such a check should be included or not, defaulting to true. Thus, users have the possibility to turn it off.


Classe HTML de bloc

https://github.com/osunyorg/theme/issues/339
https://github.com/osunyorg/admin/issues/1721


Feedback on openMINDS Matlab

Could you please re-write the text for the link at the end of the openMINDS Matlab document?
The text could be:
openMINDS_MATLAB GitHub Repository.

Could you please edit these sentences?
MATLAB package for the openMINDS metadata framework. The package contains all openMINDS schemas as MATLAB classes in addition to schema base classes and utility methods.
ALTERNATIVE 1:
The MATLAB package for the openMINDS metadata framework contains all openMINDS schemas as MATLAB classes in addition to schema base classes and utility methods.
ALTERNATIVE 2:
Here you can find the MATLAB package for the openMINDS metadata framework. This package contains all openMINDS schemas as MATLAB classes in addition to schema base classes and utility methods.


org.gradle.api.plugins.UnknownPluginException when building from source for Android.

Description
I followed all the steps from the documentation for setting up my environment and had no problems here.
Then I carried out the instructions from the Building from source for Android, but when I try to build an app I get the following error:
Build file '\MyTestApp\node_modules\react-native\ReactAndroid\build.gradle' line: 11

Plugin [id: 'org.jetbrains.kotlin.android'] was not found in any of the following sources:

* Try:
Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Exception is:
org.gradle.api.plugins.UnknownPluginException: Plugin [id: 'org.jetbrains.kotlin.android'] was not found in any of the following sources:

- Gradle Core Plugins (plugin is not in 'org.gradle' namespace)
- Included Builds (None of the included builds contain this plugin)
- Plugin Repositories (plugin dependency must include a version number for this source)

I applied all the steps to the init MyTestApp.
I am aware of others who also encoutered the same error when following the documentation,
so I suspect that the documentation for building from source might be outdated, but I'm not sure.
Version
0.67.4
Output of npx react-native info
System:
    OS: Windows 10 10.0.22000
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 5.55 GB / 15.84 GB
  Binaries:
    Node: 16.14.2 - C:\Program Files\nodejs\node.EXE
    Yarn: Not Found
    npm: 8.5.0 - C:\Program Files\nodejs\npm.CMD
    Watchman: Not Found
  SDKs:
    Android SDK:
      Android NDK: 21.4.7075529
    Windows SDK: Not Found
  IDEs:
    Android Studio: Version     2<IP_ADDRESS> AI-211.7628.21.2111.8193401
    Visual Studio: Not Found
  Languages:
    Java: 11.0.13
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.2 => 17.0.2
    react-native: 0.67.4 => 1000.0.0
    react-native-windows: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

The output says Windows 10, but I am using Windows 11.
Steps to reproduce
First I set up my environment according to the environment setup documentation.
Then I created a test app with npx react-native init MyTestApp
I carried out all the step from Building from source.
At the Building from Android Studio step I get the error in Android Studio.
Snack, code example, screenshot, or link to a repository
No response

Encountered the same error as above but build on actual Windows 10.


[Event Hubs] AmqpAnnotatedMessage round-tripping AMQP encoding/decoding data failing in nightly runs

Form Recognizer nightly test runs are failing with:

Error message:
At least one receiver for the endpoint is created with epoch of '0', and so non-epoch receiver is not allowed. Either reconnect with a higher epoch, or make sure all epoch receivers are closed or disconnected. TrackingId:7ecc46a4-0652-4c7f-9f9c-fd8c9cc0f820_B14, SystemTracker:t8a900cb6e83f4859:EventHub:eventhub~16383, Timestamp:2021-11-18T13:23:09 Reference:2b68d3d9-567b-47ac-9862-bbf1a29e3e76, TrackingId:0f78a9b9-b40b-48f4-9de9-17a2700e18d4_B14, SystemTracker:t8a900cb6e83f4859:eventhub:eventhub~16383|$default, Timestamp:2021-11-18T13:23:10 TrackingId:60314490ae3a474cbfd37df935daf25c_G7, SystemTracker:gateway7, Timestamp:2021-11-18T13:23:10
Stack trace:
MessagingError: At least one receiver for the endpoint is created with epoch of '0', and so non-epoch receiver is not allowed. Either reconnect with a higher epoch, or make sure all epoch receivers are closed or disconnected. TrackingId:7ecc46a4-0652-4c7f-9f9c-fd8c9cc0f820_B14, SystemTracker:t8a900cb6e83f4859:EventHub:eventhub~16383, Timestamp:2021-11-18T13:23:09 Reference:2b68d3d9-567b-47ac-9862-bbf1a29e3e76, TrackingId:0f78a9b9-b40b-48f4-9de9-17a2700e18d4_B14, SystemTracker:t8a900cb6e83f4859:eventhub:eventhub~16383|$default, Timestamp:2021-11-18T13:23:10 TrackingId:60314490ae3a474cbfd37df935daf25c_G7, SystemTracker:gateway7, Timestamp:2021-11-18T13:23:10
at translate (/mnt/vss/_work/1/s/sdk/core/core-amqp/src/errors.ts:665:19)
at EventHubReceiver._onAmqpError (/mnt/vss/_work/1/s/sdk/eventhub/event-hubs/src/eventHubReceiver.ts:145:3)
at Receiver.onError (/mnt/vss/_work/1/s/sdk/eventhub/event-hubs/src/eventHubReceiver.ts:171:38)
at Receiver.emit (events.js:400:28)
at Receiver.emit (domain.js:475:12)
at emit (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea-promise@2.1.0/node_modules/rhea-promise/lib/util/utils.ts:185:20)
at Object.emitEvent (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea-promise@2.1.0/node_modules/rhea-promise/lib/util/utils.ts:197:5)
at Receiver. (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea-promise@2.1.0/node_modules/rhea-promise/lib/link.ts:341:11)
at Receiver.emit (events.js:400:28)
at Receiver.emit (domain.js:475:12)
at Receiver.link.dispatch (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea@2.0.6/node_modules/rhea/lib/link.js:59:37)
at Receiver.link.on_detach (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea@2.0.6/node_modules/rhea/lib/link.js:162:32)
at Session.on_detach (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea@2.0.6/node_modules/rhea/lib/session.js:740:27)
at Connection. [as on_detach] (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea@2.0.6/node_modules/rhea/lib/connection.js:827:30)
at c.dispatch (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea@2.0.6/node_modules/rhea/lib/types.js:925:33)
at Transport.read (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea@2.0.6/node_modules/rhea/lib/transport.js:109:36)
at SaslClient.read (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea@2.0.6/node_modules/rhea/lib/sasl.js:328:26)
at Connection.input (/mnt/vss/_work/1/s/common/temp/node_modules/.pnpm/rhea@2.0.6/node_modules/rhea/lib/connection.js:551:35)
at TLSSocket.emit (events.js:400:28)
at TLSSocket.emit (domain.js:475:12)
at addChunk (internal/streams/readable.js:293:12)
at readableAddChunk (internal/streams/readable.js:267:9)
at TLSSocket.Readable.push (internal/streams/readable.js:206:10)
at TLSWrap.onStreamRead (internal/stream_base_commons.js:188:23)
For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1202256&view=results

This test should have been fixed already. We added a workaround for the service issue when initial partitions are empty


Batch Requests are incorrectly formed

In attempting to get batch requests to work with this library I repeatedly got a '404' with an error like:
<?xml version="1.0" encoding="utf-8"?><error xmlns="http://schemas.microsoft.com/ado/2007/08/dataservices/metadata"><code/><message xml:lang="en">Resource not found for the segment 'https:'.</message></error>
The code is as follows:
var settings = new ODataClientSettings(new Uri($"{tenantCredentials.C4CHost}/sap/c4c/odata/v1/c4codataapi"))
            {
                AfterResponse = async x =>
                {
                    var body = await x.Content.ReadAsStringAsync();
                    Console.WriteLine(body);
                }
            };
            settings.BeforeRequest += x =>
            {
                x.Headers.Authorization = new AuthenticationHeaderValue("Bearer", token.OAuth_Token);
            };
            var client = new ODataClient(settings);
            var batch = new ODataBatch(client);
            batch += async c => await c
                .For("CorporateAccountCollection")
                .FindEntriesAsync();
            await batch.ExecuteAsync();

Examining the batch request that gets generated I get the following:
--batch_14ef326f-b1cb-4ec8-8686-8f3d8191184a
Content-Type: application/http
Content-Transfer-Encoding: binary

GET https://<host>/sap/c4c/odata/v1/c4codataapi/CorporateAccountCollection HTTP/1.1

--batch_14ef326f-b1cb-4ec8-8686-8f3d8191184a--

The batch request should look like the following:
--batch_14ef326f-b1cb-4ec8-8686-8f3d8191184a
Content-Type: application/http
Content-Transfer-Encoding: binary

GET CorporateAccountCollection HTTP/1.1

--batch_14ef326f-b1cb-4ec8-8686-8f3d8191184a--

OData version: 2
Nuget version: 5.4.0
I believe this is a bug, however I could be incorrectly creating the batch request.

Just an update on this.
After further investigation I noticed that this is happening down in the 'Microsoft.Data.OData' library.
This leads me to believe that the standard should accept both of the requests laid out above.
Do you know if this is the case?

Simple.OData.Client uses internally Microsoft.Data.OData when using OData v1-3. I don't know what can cause it to raise an exception and will probably need more information about your service metadata in order to understand what can go wrong.


fileMatch paths do not follow official Ansible code layout guidelines

In README.md it says filepaths should follow the official Ansible code layout guidelines. Clicking on the link and this structure comes up:
collection/
├── docs/
├── galaxy.yml
├── meta/
│   └── runtime.yml
├── plugins/
│   ├── modules/
│   │   └── module1.py
│   ├── inventory/
│   └── .../
├── README.md
├── roles/
│   ├── role1/
│   ├── role2/
│   └── .../
├── playbooks/
│   ├── files/
│   ├── vars/
│   ├── templates/
│   └── tasks/
└── tests/

But this does not work because package.json defines other paths.
Solution is to change package.json or suggest another file structure that matches paths defined in package.json

I am not sure which paths you find as not conforming as the page you mentioned was the main source for building these patterns.

Okay.
Then am doing something wrong.
have a file like collection/playbooks/tasks/sometasks.yml and it will complain about stuff like Missing property "hosts" which do only exists in my playbooks, not it dedicated tasks yml files.
running ansible-lint collection/playbooks/main.yml works fine without any errors.

@blinkiz I suspect you are encountering a bug in vscode where * does go across file paths separators when it should. Also the order on which the patterns are looked up on is not really controllable.
Microsoft fixed the issue one week or two ago inside the json-language-server by adopting a newer glob library but our yaml-language-server did not get a new release yet. If you fail to convince it to pick the right schemas the only option is to wait for the upstream dependencies to be updated as there is nothing inside our extension that can be changed to make it work.
For me it appears to be working fine now but I am aware that there are cases where the patterns do overlap and you endup with an undesired schema being used.
Keep in mind that you can always define your own patterns inside settings.json, which take precedence over the ones exposed in extension settings. Reading https://github.com/redhat-developer/vscode-yaml#associating-schemas should help you.

Adding this to my settings and it starts to work.
"yaml.schemas": {
        "https://github.com/ansible-community/schemas/raw/main/f/ansible-tasks.json": ["playbooks/tasks/*.yml"]
      },

But this does not work because my playbooks/tasks/ files gets the playbook schema.
    "yaml.schemas": {
        "https://github.com/ansible-community/schemas/raw/main/f/ansible-tasks.json": ["playbooks/tasks/*.yml"],
        "https://github.com/ansible-community/schemas/raw/main/f/ansible-playbook.json": ["playbooks/*.yml"]
      },

Thanks for creating this nice feature for VSCode (VSCodium in my case). 🙂

You are welcomed. If you have a minute or two please also do a review on https://marketplace.visualstudio.com/items?itemName=zbr.vscode-ansible&ssr=false#review-details and start/watch the project.


Collaboration request

Hey MakerMap folks,
We're creating an open source wiki/platform/atlas of all the community innovation spaces (and related events), and we'd love to collaborate! We've got 5000+ spaces and sponsorship from the German government for platform development. Let's talk?
We're getting started here: https://github.com/AnanseGroup/ananse-map
Thanks,
Anna Waldman-Brown
anansegroup.com
P.S. @nickpinkston, are you still involved in this?

you got any reply for this.?
we are also trying to map the local resources of Karachi, Pakistan and wanted a framework to build-upon, though we would LIKE to have more control over the data, as in us being able to add, moderate easily without intervention of map maker folks.
Your project also looks promising, maybe more promising even.
Thanks.
Would like to talk to you about this.
Zaid Pirwani

Hey Y'all, I didn't see the initial message - sorry about that.
I'm happy to share with y'all - please email me directly:
<EMAIL_ADDRESS>to coordinate.
Thanks!

Nick

On Thu, Sep 15, 2016 at 6:44 AM, Zaid Pirwani<EMAIL_ADDRESS>wrote:

you got any reply for this.?
we are also trying to map the local resources of Karachi, Pakistan and
wanted a framework to build-upon, though we would LIKE to have more control
over the data, as in us being able to add, moderate easily without
intervention of map maker folks.
Your project also looks promising, maybe more promising even.
Thanks.
Would like to talk to you about this.
Zaid Pirwani
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hwstartup/TheMakerMap.com/issues/31#issuecomment-247331739,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASo-x2hZu4JxvzBVa5OPfTd5vEzm4dlks5qqUusgaJpZM4JMqDo
.

Hey Nick and Zaid,
We’ve actually been collaborating with Renee from Maker Maps to build up a more open-source framework, and plan to have something ready by the end of October! Please check out our Github, and just let us know if you’d like to get involved: https://github.com/AnanseGroup/ananse-map https://github.com/AnanseGroup/ananse-map
Your initiative sounds awesome— good luck!
Best,
Anna

On Sep 15, 2016, at 7:10 PM, Nick Pinkston<EMAIL_ADDRESS>wrote:
Hey Y'all, I didn't see the initial message - sorry about that.
I'm happy to share with y'all - please email me directly:
<EMAIL_ADDRESS>to coordinate.
Thanks!

Nick

On Thu, Sep 15, 2016 at 6:44 AM, Zaid Pirwani<EMAIL_ADDRESS>wrote:

you got any reply for this.?
we are also trying to map the local resources of Karachi, Pakistan and
wanted a framework to build-upon, though we would LIKE to have more control
over the data, as in us being able to add, moderate easily without
intervention of map maker folks.
Your project also looks promising, maybe more promising even.
Thanks.
Would like to talk to you about this.
Zaid Pirwani
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hwstartup/TheMakerMap.com/issues/31#issuecomment-247331739,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASo-x2hZu4JxvzBVa5OPfTd5vEzm4dlks5qqUusgaJpZM4JMqDo
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/hwstartup/TheMakerMap.com/issues/31#issuecomment-247480195, or mute the thread https://github.com/notifications/unsubscribe-auth/ASnZe5ZEp-to8Fm7Yq_d0sgvLiib1dkqks5qqdBkgaJpZM4JMqDo.

Wait - Renee DiResta?
On Thu, Sep 15, 2016 at 8:05 PM, Anna Waldman-Brown<EMAIL_ADDRESS>wrote:

Hey Nick and Zaid,
We’ve actually been collaborating with Renee from Maker Maps to build up a
more open-source framework, and plan to have something ready by the end of
October! Please check out our Github, and just let us know if you’d like to
get involved: https://github.com/AnanseGroup/ananse-map
Your initiative sounds awesome— good luck!
Best,
Anna
On Sep 15, 2016, at 7:10 PM, Nick Pinkston<EMAIL_ADDRESS>wrote:
Hey Y'all, I didn't see the initial message - sorry about that.
I'm happy to share with y'all - please email me directly:
<EMAIL_ADDRESS>to coordinate.
Thanks!

Nick

On Thu, Sep 15, 2016 at 6:44 AM, Zaid Pirwani<EMAIL_ADDRESS>wrote:

you got any reply for this.?
we are also trying to map the local resources of Karachi, Pakistan and
wanted a framework to build-upon, though we would LIKE to have more
control
over the data, as in us being able to add, moderate easily without
intervention of map maker folks.
Your project also looks promising, maybe more promising even.
Thanks.
Would like to talk to you about this.
Zaid Pirwani
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/hwstartup/TheMakerMap.com/issues/31#
issuecomment-247331739>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AASo-
x2hZu4JxvzBVa5OPfTd5vEzm4dlks5qqUusgaJpZM4JMqDo>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hwstartup/TheMakerMap.com/issues/31#issuecomment-247480195,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASnZe5ZEp-to8Fm7Yq_d0sgvLiib1dkqks5qqdBkgaJpZM4JMqDo
.


Lance DB in Nextjs

I am not sure what the issue is, but I think there is a challenge with how nextjs bundles api routes which doesn't work with LanceDB.
going to include the error stack trace below
- error<EMAIL_ADDRESS>Module parse failed: Unexpected character ' ' (1:1)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
> # `@lancedb/vectordb-darwin-x64`
|
| Prebuilt binary package for `vectordb` on `darwin-x64`.

Import trace for requested module:
<EMAIL_ADDRESS>./node_modules/vectordb/node_modules/@lancedb/ ./node_modules/@lancedb/ sync ^\.\/vectordb\-.*$
./node_modules/vectordb/native.js
./node_modules/vectordb/dist/index.js
./app/api/chat/route.ts

Hi @koolamusic vectordb's node binding (index.node) needs a plugin to be parsed by webpack when bundling. You can either use that or simply ignore that file. Please see how you can configure webpack to make this work on our docs - https://lancedb.github.io/lancedb/basic/#note-bundling-vectordb-apps-with-webpack

Thanks for the prompt feedback. This was super helpful. I had initially found a way around that is more like sorcery.
   webpack: (config, { isServer }) => {
      if (isServer) {
        config.externals = [
          ...config.externals,
          // @ts-ignore
          function ({ context, request },  callback) {
            if (request.endsWith('.node') || request.includes('vectordb') || request.includes('@lancedb/vectordb-darwin-x64/README.md')) {
              return callback(null, 'commonjs ' + request);
            }
            callback();
          },
        ];
      }

      return config;
    },
    ```

    But the solution you've shared is way more cleaner. 🤟🏾

Thanks for the prompt feedback. This was super helpful. I had initially found a way around that is more like sorcery.
   webpack: (config, { isServer }) => {
      if (isServer) {
        config.externals = [
          ...config.externals,

          function ({ context, request },  callback) {
            if (request.endsWith('.node') || request.includes('vectordb') || request.includes('@lancedb/vectordb-darwin-x64/README.md')) {
              return callback(null, 'commonjs ' + request);
            }
            callback();
          },
        ];
      }

      return config;
    },
    ```

    But the solution you've shared is way more cleaner. 🤟🏾

@AyushExel can't find it in the docs, but how do I do a where
await table
    .search(currentMessageContent)
    .where(`type = "post" or type="image"`)
    // .orderBy(`cosine_similarity(${VECTOR_SOURCE_COLUMN}, "${currentMessageContent}")`)
    .select(['type', 'text', 'url'])
    .limit(2)
    .execute();
    ```

Hi @koolamusic vectordb's node binding (index.node) needs a plugin to be parsed by webpack when bundling. You can either use that or simply ignore that file. Please see how you can configure webpack to make this work on our docs - https://lancedb.github.io/lancedb/basic/#note-bundling-vectordb-apps-with-webpack
Let me know if you face any problems :)

It seems this link is updated with new content and I found ref information related to webpack on that page. I got the same issue. Could you please share me how did you fix this problem?


[stable/prometheus-operator] Fix conditions in the default rule templates

Is this a new chart
No
What this PR does / why we need it:
Fixes 2 known issues. Currently, it's impossible to opt out of some default rules because the conditions in the templates are generated incorrectly. The flags which were affected were:

prometheus.defaultRules.rules.kubernetesSystem
prometheus.defaultRules.rules.prometheus

This PR addresses the handling of the edge cases, since the groups were renamed in 1.14. It addresses them by adding more entries to handle both cases.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #19309
fixes #19033

Special notes for your reviewer:
I decided not to introduce different chart values and keep the interface the same since the handling is getting even more complex. Please provide feedback if it's needed to proceed this way.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @gianrubio

/ok-to-test

/ok-to-test

/lgtm


Make Schwalbe less humorous.

No, really. I'm serious.

😄👎
😐👍

Lol, the last thing I expected was this to actually get merged. 😂

Definitely a reasonable request 🙈😄


Major version 1, but we expected major version 2

When running upload-log after upgrading.
simpleArcParse has major version 1, but we expected major version 2
Please upgrade to the v2.0.0 release of simpleArcParse
Press enter to exit:

Try re-downloading simpleArcParse.exe, I think I accidentally had the wrong version uploaded.

This should be fixed now, but let me know if it's not.

Yep fixed


Convert str to bytes

This is just a small quality-of-life improvement that allows passing memory arguments to Spec as strings with units, e.g. allowed_mem='100MB' rather than allowed_mem=100_000_000.

The Spec object is now immutable. It could be mutable, like it technically was before, but I wasn't sure what the intended use was. If Spec is just intended as a container of kwargs then it should really be immutable (or even a NamedTuple, though that has similar problems with input validation).

Great. This should be good to merge then.


importance score

does these importance scores make sense ?
[8211428.52734375,
5057594.07421875,
3736915.91015625,
3227586.345703125,
3689871.533203125,
3351593.2578125,
3044111.75,
2899957.0205078125,
2688742.900390625,
2426028.0209960938,
2433061.6528320312,
2243200.3950195312,
2064878.9208984375,
1991253.4248046875,
2140301.0810546875,
2000642.2299804688,
2041371.0180664062,
1918713.4189453125,
2153077.2265625,
1906909.8125,
1454954.0717773438,
1137825.6186523438,
980444.470703125,
993320.5024414062,
853812.5122070312,
829364.119140625,
834055.8408203125,
946702.3803710938,
1130785.9765625,
1324971.9921875,
1355165.20703125,
4391842.64453125]
this is from ur notebook only. why they are so high

How it comes the same layers index for both mixtral and llama
25, 27, 24, 26, 28, 29, 23, 22, 21

Hi @riyajatar37003, yeah these scores are large because I chose not to compute a mean -- rather a sum of importances over the dataset.  As you can imagine, the more samples in the data the larger these values will become.
As for the pruned layer overlap, both models I show have 32 layers and I remove 9. They would likely differ with different models but the reason why layers in those relative positions are pruned is described in the papers. Hope that helps, otherwise I can provide more detail. Thanks!

Thanks


sync() undefined on bgclang build

Original issue: https://charm.cs.illinois.edu/redmine/issues/1320

There is a macro used to protect against calling sync() where it is not defined, CMK_DISABLE_SYNC, that should just need to be set in the bgclang conv-mach header.
../bin/charmc  -optimize -production   -I.   -c -o ckcheckpoint.o ckcheckpoint.C
ckcheckpoint.C:270:9: error: use of undeclared identifier 'sync'
        sync();
        ^
1 error generated.
Fatal Error by charmc in directory /projects/CharmRTS/nbhat/autobuild/pamilrts-clang-nosmp/charm/pamilrts-bluegeneq-clang/tmp
   Command bgclang++ -I../bin/../include -I/bgsys/drivers/ppcfloor/comm/include -I/bgsys/drivers/ppcfloor/spi/include -I/bgsys/drivers/ppcfloor -I/bgsys/drivers/ppcfloor/spi/include/kernel/cnk -I/soft/libraries/alcf/current/xl/ZLIB//include -D__CHARMC__=1 -I. -O3 -ffast-math -fno-stack-protector -std=c++0x -c ckcheckpoint.C -o ckcheckpoint.o returned error code 1
charmc exiting...
make: *** [ckcheckpoint.o] Error 1

Original date: 2016-12-05 22:22:51

That sync() call is quite semantically important. Are you sure this isn't just a missing #include <unistd.h> or some related #define?
Absent the sync() call, we need to have fsync() calls on the checkpoint files before closing them. Without that, we may continue from a checkpoint, thinking the data is safely on disk, only to find that it isn't after a failure.

Original date: 2016-12-07 17:29:40

Yeah, ckcheckpoint was just missing inclusion of unistd.h: ~~https://charm.cs.illinois.edu/gerrit/#/c/2027/~~ https://github.com/UIUC-PPL/charm/commit/150bc9bc4dd97860dedb689e247d494bbbf46942

Original date: 2016-12-07 17:59:15

Now we get a different failure:
Performing '/usr/bin/gmake charm++ OPTS= -g QUIET=' in pamilrts-bluegeneq-clang/tmp
../bin/charmc  -g  -I.   -c -o DummyLB.o DummyLB.C
../bin/charmc  -g  -o ../lib/libmoduleDummyLB.a DummyLB.o
bgclang-ar: creating ../lib/libmoduleDummyLB.a
bgclang-ar: libc++.so.1: cannot open shared object file: No such file or directory

bgclang-ar: libc++.so.1: cannot open shared object file: No such file or directory

../bin/charmc: line 192: 14278 Segmentation fault      (core dumped) bgclang-ar q ../lib/libmoduleDummyLB.a DummyLB.o
Fatal Error by charmc in directory /home/white/charm/pamilrts-bluegeneq-clang/tmp
   Command bgclang-ar q ../lib/libmoduleDummyLB.a DummyLB.o returned error code 139
charmc exiting...

Using powerpc64-bgq-linux-ar instead of bgclang-ar seems to work: ~~https://charm.cs.illinois.edu/gerrit/#/c/2028/~~ https://github.com/UIUC-PPL/charm/commit/c20acb7e1880d79470e10a6c918dad6dc81adeaf
Then there are missing headers, fixed here: ~~https://charm.cs.illinois.edu/gerrit/#/c/2029/~~ https://github.com/UIUC-PPL/charm/commit/9a4838c7da4c20ff7082b486d55624d1dcbcf949
Now a LIBS build of pamilrts-bluegeneq-clang compiles on Mira, after doing 'soft add +mpiwrapper-bgclang'.

Original date: 2016-12-13 17:48:09

bgclang builds are now passing autobuild


Abstract Test Case - HelloWorld

Create an example of an abstract test case for the HelloWorld tutorial

The development of a abstract test case notation is not planned


runtime/cgo: Add initial NetBSD Thread Sanitizer support

Recognize NetBSD in:

go/internal/work/init.go
race.bash
runtime/race/race.go

Add __ps_strings symbol in runtime/cgo/netbsd.go as this is
used internally in the TSan library for NetBSD and used for
ReExec().
Tested on NetBSD/amd64 v. 8.99.12.
Around 98% tests are passing for the ./race.bash target.

NetBSD 8.0 or later needed. Older versions might work, but I won't invest into them. 8.0 is already matched as the minimum version recommended for Go.

CC: @dvyukov
./race.bash gives something like:
Passed 340 of 347 tests (97.98%, 0+, 7-)
0 expected failures (0 has not fail)

http://netbsd.org/~kamil/llvm/go-tsan-2017-12-06.txt
I'm using a locally generated syso file for NetBSD/amd64:  http://netbsd.org/~kamil/llvm/race_netbsd_amd64.syso
I assume that it has to be added by upstream maintainers.

There is one failure related to exit status, which is I guess a plain bug somewhere and needs to be fixed.
For the remaining failures I don't have a good explanation, because core race detection logic and compiler instrumentation are platform-independent. Are these failures flaky? If you run it 10 times do you see the same set of failures each time?
Re syso file, yes, somebody needs to regenerate all of them and add netbsd.
The official syso files are created with:
https://github.com/golang/build/blob/master/cmd/racebuild/racebuild.go
we need to add netbsd support there (probably mostly copy-paste freebsd part).
Filed #24354 for this.

Few tests are indeed racy.
In this case I follow "Pareto principle" and move on upstreaming my local changes, as more than 95% of tests report success. In future I plan to be back to sanitizers in LLVM and correct remaining bugs and enable the execution of regression tests on LLVM/NetBSD buildbot. It will be followed by squashing the Go/TSan bugs.
It will be easier to fix the remaining bugs having TSan/NetBSD in the Go sources.

As you probably saw in llvm tsan tests we use "tsan-invisible-barrier" in tests, which effectively turns them into non-parallel/non-concurrent (though, tsan runtime still thinks that they are parallel and detects races). That eliminates flakiness due to non-determinism and timings. It would be good to have the same for go tests as well.

I don't have strong opinions, but if we will support it - I will make sure to handle it correctly [in future].

Да

I will have a look!

Results for:
$ ./all.bash
$ ./race.bash

and a local diff to enable -race for NetBSD/amd64 originally proposed in https://github.com/golang/go/pull/24322
http://netbsd.org/~kamil/golang/golang-amd64-2018-06-03-race-HEAD.txt
$ uname -a
NetBSD chieftec 8.99.19 NetBSD 8.99.19 (GENERIC) #6: Thu May 31 02:36:15 CEST 2018  root@chieftec:/public/netbsd-root/sys/arch/amd64/compile/GENERIC amd64

Tested with:
$ md5 ./runtime/race/race_netbsd_amd64.syso
MD5 (./runtime/race/race_netbsd_amd64.syso) = 3bff62c1ae46943d4d6aedbbde4b4c51

For the record, C/C++ version of TSan (from HEAD of LLVM) for NetBSD/amd64:
$ check-tsan

  Expected Passes    : 288
  Expected Failures  : 1
  Unsupported Tests  : 84
  Unexpected Failures: 8

https://github.com/golang/go/commit/8994444491d984491d8070ed30aeffc1abe344a2
ping?

ping

(I've used github's online conflict resolver.. hopefully everything is fine).

@bsiegert please help to upstream this, thank you in advance!


Create Priroch1979.yml

Why:

Closes [issue link]
What's being changed:

Check off the following:

[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

Priroch1979:main


On type highlighting

I did a fresh install of ubuntu Disco. After doing pre install and running
bash -c  "$(wget -qO- https://git.io/vQgMr)"

it says
    ...
    ( 173 ) Wombat
    ( 174 ) Wryan
    ( 175 ) Zenburn
    ( ALL ) All themes

Usage : Enter Desired Themes Numbers (OPTIONS) Separated By A Blank Space
        Press ENTER without options to Exit

Enter OPTION(S) : 34

Theme: Dracula
•••••••• ••••••••

████████████████████████████████████████
████████████████████████████████████████

bash: line 655: read: `/apps/gnome-terminal/profiles/default_profile': not a valid identifier
environment: line 402: --get: command not found
environment: line 411: --get: command not found
environment: line 222: : command not found
environment: line 222: : command not found
environment: line 205: : command not found
environment: line 205: : command not found
environment: line 205: : command not found
environment: line 205: : command not found
environment: line 205: : command not found
environment: line 205: : command not found
environment: line 205: : command not found
environment: line 205: : command not found
environment: line 205: : command not found

environment: line 205: : command not found

I know there are other ways to do it(fixes) and i will try them, but posting this just to let you know.
Edit1:
Tried

Install by Script
You also can execute this script. This will list all available themes allowing you to select the one you like.
$ wget -O xt  http://git.io/vGz67 && chmod +x xt && ./xt && rm xt

to get
yahyaa@HOME:~$ wget -O xt  http://git.io/vGz67 && chmod +x xt && ./xt && rm xt
URL transformed to HTTPS due to an HSTS policy
--2019-04-23 23:59:46--  https://git.io/vGz67
Resolving git.io (git.io)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to git.io (git.io)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://raw.githubusercontent.com/Mayccoll/Gogh/master/script/gogh.sh [following]
--2019-04-23 23:59:47--  https://raw.githubusercontent.com/Mayccoll/Gogh/master/script/gogh.sh
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 404 Not Found
2019-04-23 23:59:47 ERROR 404: Not Found.

 Re-open the terminal.
Go to Edit ---> Preferences ---> Profiles.
Create NEW profile called "Default".
Close the terminal. Re-open the terminal.
Confirm "Default" is still listed in your profiles.
If it is, run the Gogh script again.

Worked fine. Thanks.  Re-open the terminal.
Go to Edit ---> Preferences ---> Profiles.
Create NEW profile called "Default".
Close the terminal. Re-open the terminal.
Confirm "Default" is still listed in your profiles.
If it is, run the Gogh script again.

Worked fine. Thanks. But i don't see the three dots and on colour highlighting like when you typed.

thanks mate. Worked like a charm ;)


AMQ Broker Operator 7.8.1 broken in OperatorHub

Describe the bug
Deployment in crasloopback state due missing field in openAPIV3Schema.
To Reproduce
Steps to reproduce the behavior:

Go to OperatorHub in Openshift container platform
Install AMQ Broker Operator
Deployment doesn't conclude due error: panic: runtime error: index out of range [0] with length 0 [recovered]
Describe crd activemqartemises.broker.amq.io and get the message:
Message:               [spec.versions[0].schema.openAPIV3Schema.type: Required value: must not be empty at the root, spec.versions[1].schema.openAPIV3Schema.type: Required value: must not be empty at the root, spec.versions[2].schema.openAPIV3Schema.type: Required value: must not be empty at the root]
Platform (please complete the following information):
amqCRD.txt

OS: Openshift Container Platform
Version 4.6.21
Go [e.g. 1.13.4

This issue will be closed because it has not been updated for a long time.


Updated for Julia v1.0

I updated MAT.jl for it to work with Julia v1.0. Have been using it for a few weeks and seems to be ok.

Dup of other ones.

I understand, it's a duplicate. But vanilla MAT.jl doesn't work with v1.0. Therefore, please fix it or pull the fixes from someone else.

No one has a complete fix yet. And as I mentioned in https://github.com/JuliaIO/MAT.jl/pull/100 that is now merged as https://github.com/JuliaIO/MAT.jl/pull/102, I have no problem merging partial fix as long as they don't break anything else (and also don't introduce any additional copying).

Well, I just pushed last changes to my repository clone in github. It now
passes all tests now in 0.7 and 1.0.1.  (see below)
If you are interested let me know and I will  send you a pull request.
There is a conflict with #100 due to style differences. But you can cherry
pick if you like.
Regards,
Jorge
Testing MAT
Resolving package versions...
Status /tmp/tmp7S1hCT/Manifest.toml
[9e28174c] BinDeps v0.8.10
[b99e7846] BinaryProvider v0.5.2
[a74b3585] Blosc v0.5.1
[e1450e63] BufferedStreams v1.0.0
[631607c0] CMake v1.1.0
[d5fb7624] CMakeWrapper v0.2.2
[34da2185] Compat v1.3.0
[864edb3b] DataStructures v0.14.0
[f67ccb44] HDF5 v0.10.2
[0862f596] HTTPClient v0.2.1
[d9be37ee] Homebrew v0.7.0
[682c06a0] JSON v0.19.0
[b27032c2] LibCURL v0.4.1
[522f3ed2] LibExpat v0.5.0
[2ec943e9] Libz v1.0.0
[23992714] MAT v0.4.0+ [~/.julia/dev/MAT]
[bac558e1] OrderedCollections v1.0.2
[d96e819e] Parameters v0.10.1
[30578b45] URIParser v0.4.0
[c17dfb99] WinRPM v0.4.2
[2a0f44e3] Base64  [@stdlib/Base64]
[ade2ca70] Dates  [@stdlib/Dates]
[8bb1440f] DelimitedFiles  [@stdlib/DelimitedFiles]
[8ba89e20] Distributed  [@stdlib/Distributed]
[b77e0a4c] InteractiveUtils  [@stdlib/InteractiveUtils]
[76f85450] LibGit2  [@stdlib/LibGit2]
[8f399da3] Libdl  [@stdlib/Libdl]
[37e2e46d] LinearAlgebra  [@stdlib/LinearAlgebra]
[56ddb016] Logging  [@stdlib/Logging]
[d6f4376e] Markdown  [@stdlib/Markdown]
[a63ad114] Mmap  [@stdlib/Mmap]
[44cfe95a] Pkg  [@stdlib/Pkg]
[de0858da] Printf  [@stdlib/Printf]
[3fa0cd96] REPL  [@stdlib/REPL]
[9a3f8284] Random  [@stdlib/Random]
[ea8e919c] SHA  [@stdlib/SHA]
[9e88b42a] Serialization  [@stdlib/Serialization]
[1a1011a3] SharedArrays  [@stdlib/SharedArrays]
[6462fe0b] Sockets  [@stdlib/Sockets]
[2f01184e] SparseArrays  [@stdlib/SparseArrays]
[10745b16] Statistics  [@stdlib/Statistics]
[8dfed614] Test  [@stdlib/Test]
[cf7118a7] UUIDs  [@stdlib/UUIDs]
[4ec0a83e] Unicode  [@stdlib/Unicode]
Checking reading v6 format
Checking reading v7 format
Checking reading v7.3 format
Testing MAT tests passed
On Tue, Nov 6, 2018 at 11:35 PM Yichao Yu<EMAIL_ADDRESS>wrote:

No one has a complete fix yet. And as I mentioned in #100
https://github.com/JuliaIO/MAT.jl/pull/100 that is now merged as #102
https://github.com/JuliaIO/MAT.jl/pull/102, I have no problem merging
partial fix as long as they don't break anything else (and also don't
introduce any additional copying).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JuliaIO/MAT.jl/pull/101#issuecomment-436434308, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AOki7w0KuIHuvQmu9ucTPoynlL3zu1EFks5usg6lgaJpZM4YPVCK
.

It has all the same issues as https://github.com/JuliaIO/MAT.jl/pull/86 .

The absense of reinterpreted array is exactly the issue. It's the copying that I'm talking about and what's what I mentioned above too.

ok, I see what you mean now. The performance is highly impacted because convert() makes a copy (see below)
julia> @btime reinterpret(UInt8, x);
57.207 ns (1 allocation: 32 bytes)
julia> @btime convert(Array, reinterpret(UInt8, x));
1.325 μs (2 allocations: 208 bytes)
And it's not a small hit, in this example is an order of magnitude ( ~23X ).
Although, I disagree with your categorical denial of accepting any changes for 1.0.x, now I understand your concern. Seems that you know more about this, but will see if I can fix it.
I think that a slow package is better than a broken package though.

I think that a slow package is better than a broken package though.

I can accept a guarantee that the slowless to be fixed within a certain deadline together with a list of places to fix. If the deadline is not met then the change will be reverted and there shouldn't be a release before the fix. Otherwise, I don't think it'll ever happen.

your categorical denial of accepting any changes for 1.0.x

And no, I never said that. If your change doesn't break 0.6 and doesn't cause extra copy, go ahead and submit it and I have no problem merging it. That's exactly why I merged https://github.com/JuliaIO/MAT.jl/pull/100 (The changes I requested were not fully fixed so I even fixed them myself after waiting for a week.)

Ok, after some digging I think that the culprit (mainly) is that reinterpret() was changed in v1.0 and returns a new type  ReinterpretArray which is an AbstractArray, which is a View of the array. Solution spaces that I can think of are the following:

Change the reference data such that I has the same type as Julia version (Note: I read from another thread that you are not in favor of such direction, but I have to mention it)
Make our own reinterpret() function that return Array types instead of using the generic function. This might deviate further in future versions of Julia.
Create our own array Type and redefine reinterpret() such that it returns the same Type always, e.g. MATArray or MXArray or whatever your prefer,  regardless of Julia version, not sure if we can use the generic reinterpret() from Julia, but probably we can. This also needs an update of the reference data, but at least will be consistent.

I am in favor of solution 3 but before I write anything I wanted to check up with you if you agree upon this and/or check what are your suggestions.

I'm sure I am not getting the whole picture here, but I'd vote for @el-oso's solution 2. I don't really understand what solution 1 is, and solution 3 introducing a new type might add an extra layer of work for the package users. (When I load a MATLAB array into Julia using MAT, I would personally prefer it to be an Array right away, rather than it being a MATArray that I may need to convert to an Array before I can use it in my Julia code. But maybe that's just me?)

2 is imposible.


CI/CD should file an issue when scheduled or push run fails

When either a CI/CD run fails that was triggered by push or schedule (but not pull request), the CI/CD workflow should:

[ ] File an issue
[ ] Attach a link to the failing workflow
[ ] Attach the logs, dump, or artifact that triggered the failure

@Alan-Jowett is this issue fixed by #1385?  Can this be closed now?


don't render labels when no label text is provided

While working on custom checkbox components that include a label, I've discovered that all fields have a label tag rendered when "fieldTypeHasLabel" returns true.  I propose adding an additional check to this function to return false when the field's schema does not have a label property (or altering the v-if statement on the label.
// option 1: add the following to fieldTypeHasLabel(field)
if(isNil(field.label))
  return false;

// option 2: alter the label v-if
label(v-if="field.label && fieldTypeHasLabel(field)", :for="getFieldID(field)", :class="field.labelClasses")

This requires the unit tests to be aware of this change as well.
I can submit a PR for this change if approved.  Just need to know which option makes more sense.

Good idea. I think option 1 is a good solution.


V3

livechatconnector/v3/getchattoken endpoint

could you add evidence of testing ? what about unit tests?

could you add evidence of testing ? what about unit tests?

Some unit tests were added to ensure the proper endpoints are created. Manual validation on the new changes was done to ensure backward compatibility.


Add support for C++20 concepts

CC @lums658 refs #160 I couldn't push to that branch (I think because it's the main branch of your repo), I'm sorry for the very long delay.  In order to support C++ concepts I needed to fix the testing framework for doxygen 1.9.x in order to actually test for this.  This one will take me some time to flesh out entirely (there's a handful of test framework changes which would be a nightmare for not me (well for me too, but way worse for anyone else)).
However, would you be willing to add some code to the newly created testing/projects/cpp_concepts/include/concepts/concepts.hpp and associated usages (where applicable) in src/tests.cpp?  The tests.cpp exists mostly to guarantee that anything added in concepts.hpp actually compiles.  The infrastructure is up, but it would be really helpful to have a more exhaustive list of  concepts code -- I'm not up to speed, just anything that would stress test doxygen / breathe from a documentation perspective.  Note the testing code license is CC0 (please add yourself to the license at the top though!).  Only other rule for code there is keep it PG-13, but otherwise the content (or docs) are less than meaningful anywhere else :wink:
I get the impression it would be fairly easy for you to PR to this branch with some nice modern concepts code there :pray:
There's definitely no rush ... it'll take me some time to finalize things (I anticipate CI being troublesome here), but after the code is in place I will be able to inspect the implementation / create unit tests for the project.  Having the test project in place helps me look at the output of exhale and confirm it's working as desired.

Just want to say that I'm quite interested in concepts support as well.

The doxygen compatibility problems have been resolved, but I still need test code that exhausts concepts options. Been perusing cppreference but still trying to work it out (haven't had the luxury of using c++20 in production and not enough time to catch up on my own).
Questions that come up:

Is there any concept inheritance?
Can there be nested concepts (like nested classes / structs, or can a concept be declared in a nested scope like in a class definition)?
Any other known "fancy edge cases"?

I'm definitely down to merge something we think covers the bases and fix it later if needed. But having code on the tests is required, it's the only reason this project still limps along. The absurd implementation is brittle and small changes elsewhere can and do break other things. Sample code would be most welcome (header file and usage examples). From there I can integrate that code into the rest of the (very complicated) testing framework. I've got some flights coming up, the last one I was able to make some good progress on updating tree view html / css. This one is also high priority for me, lack of concepts is an eyesore (it's almost 2023!)

I apologize for completely missing your response on #160 back in June….  I can try to put in some example / testing code in the next couple of weeks or so.
Best Regards,
Andrew Lumsdaine
On Nov 20, 2022, at 10:40 AM, Stephen McDowell<EMAIL_ADDRESS>wrote:
The doxygen compatibility problems have been resolved, but I still need test code that exhausts concepts options. Been perusing cppreference but still trying to work it out (haven't had the luxury of using c++20 in production and not enough time to catch up on my own).
Questions that come up:

Is there any concept inheritance?
Can there be nested concepts (like nested classes / structs, or can a concept be declared in a nested scope like in a class definition)?
Any other known "fancy edge cases"?

I'm definitely down to merge something we think covers the bases and fix it later if needed. But having code on the tests is required, it's the only reason this project still limps along. The absurd implementation is brittle and small changes elsewhere can and do break other things. Sample code would be most welcome (header file and usage examples). From there I can integrate that code into the rest of the (very complicated) testing framework. I've got some flights coming up, the last one I was able to make some good progress on updating tree view html / css. This one is also high priority for me, lack of concepts is an eyesore (it's almost 2023!)
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/svenevs/exhale/pull/166*issuecomment-1321210466__;Iw!!K-Hz7m0Vt54!iPD9y_1byAiuT3jWTd73k1T8geGKunvJJvMZZDcz0_ZzYPPu02uFx-fTMetoi_8q6s-rjYofWKximfRA0AFfOQ$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AGKPFDMOU3AVW3D6RW7MXWLWJJWDFANCNFSM5ZBLLFOQ__;!!K-Hz7m0Vt54!iPD9y_1byAiuT3jWTd73k1T8geGKunvJJvMZZDcz0_ZzYPPu02uFx-fTMetoi_8q6s-rjYofWKximfQsXwF_1Q$.
You are receiving this because you were mentioned.Message ID: @.***>

No apology necessary, I responded very late... Ok, great, thank you so much!!! To be clear I only need help with the C++ source code, the python side / exhale test case side I will take care of 🙂

Thanks for the fast response. Maybe I can help a bit.

Can there be nested concepts (like nested classes / structs, or can a concept be declared in a nested scope like in a class definition)?

Not to my knowledge.

Is there any concept inheritance?

I don't think so. But you can combine concepts easily, like
template <typename T>
concept Number = std::integral<T> || std::floating_point<T>;

(Taken from https://www.sandordargo.com/blog/2021/02/24/cpp-concepts-with-classes)

Any other known "fancy edge cases"?

According to cppreference there are three types of constraints:

conjunctions
disjunctions
atomic constraints

So having an example per type seems to be a good start.
One edge case might be the inline declaration of a concept. Although bad style it's valid:
template <typename T> requires requires (T x) { x + x; }
T add(T a, T b) { return a + b; }

See it on compiler explorer.. But I'm not sure how this should be documented.

Thank you so much, I've cleared Saturday, let's keep working with #218 and #219.
I will do final passes but ultimately rely on user testing. If it's working on your codebase then we merge. Fix problems if people report them.
Main expected snag is getting breathe_identifier implemented correctly. Historically breathe is very sensitive to whitespace issues. thing< specialized > with spaces, not thing<specialized> for example. Refer to the generated doxygen xml for the answer, that's what breathe uses (or at least used to). The defname and declname I think?
Thank you again for putting time into this, I will get back to you as soon as I can!


fix: search

Linked Issue
#16
Description
Updates logic in server/api/search.ts to find pages that are not labelled as hidden.

Adds pending and error states.
Adds cache for search results
Ensures search results are not included in client bundled

As size is prohibitive and requires activation to be used.

Fixes dark theme text
Adds bodyLimit to search to cap search size.

Feedback and suggestions are welcome!

Note that the search route appears to be computationally expensive. I can't think of a local SSG implementation that doesn't involve sending large text chunks to the client that may not be used in a search query.
Possible alternatives include offloading search to an external service like Algolia and the Algolia Nuxt module. Algolia is also the search provider for the Nuxt Website and many more.

indeed, the search system is not efficient for large blogs.
It used to work for my own blog, which is quite large for a personal blog (and apparently it's broken now ^^) but indeed, bloggrify should propose an alternative for larger blogs
algolia could be the alternative solution for large content and could be propose as an option to the current system.


Incorrect count when multiple dependency updates

We have just started using your tool and find it very useful, but the calculation for the number of dependency updates does not match the list of dependencies.
For example:
You have 1 dependencies with newer available releases:
java-jwt 4.3.0 => 4.4.0
jackson-bom 2.15.0 => 2.15.2
jackson-dataformat-yaml 2.15.0 => 2.15.2
jackson-datatype-jsr310 2.15.0 => 2.15.2

I have submitted a fix in #1 to resolve this, please consider applying this PR.
Thanks!

@raulpadilladelgado
Hi there, please have a look at this and consider merging the PR.
Thanks!

Hi, I just merged and published it as version 1.0.1. Thank you for fixing it!


Issue #613 - update conda env setup for CI images

No longer attempt to update the base conda env
Use named dask-jobqueue conda environment

Ok, so the idea here is to merge this PR, and then attempt to trigger the CI with the new built images?

We can double run to confirm the update or I can split the PR if that's preferable.   (Not sure if there are any restrictions on who can manually trigger / re-run Actions)

Okay, let's merge this one, we'll see then.


WIP - feat(CRD): Traits configuration schema

This addresses #1614. This tries to restore the unmerged pull req #1624.

This PR moves the traits configuration into the API. This enables the generation of the openAPI schema for the traits configuration into the Integration, IntegrationPlatform and IntegrationKit CRDs.

Release Note
feat(CRD): Traits configuration schema

The main part is done. Now I'll see and fix what makes the e2e tests fail.

One more issue to address is that the addons are not listed in the Traits struct. We need to find a way to extend Traits from addons.
One option would be to add an Addons struct along with the Traits, which would serialise as a list of RawMessage (as traits currently), and have a mechanism to contribute/update the CRD with the addon structural schema.

The openshift-build workflow is failing at known flaky tests, which should be irrelevant to this pull req.

Thanks!  OK, let me go ahead and merge...


Revert changes to medLogger related to filesystem.h

(see https://github.com/medInria/medInria-public/issues/423)
After synching with the upstream, MUSIC 3 doesn't compile. This reverts the faulty commits so that we can keep working on this branch while waiting for a fix.

Oki for me! (let's keep in mind that the utf8 is problematic on Windows, that's why this code have been changed on medInria. But the problem should be solved on medInria in a close future)

Since it has been solved on medInria, could this PR be closed?

yes this is no longer needed


Remove an abundance of generated classes

And party like its 1999!
Closes #16

Can you squash to one commit? Did they have squash back then?

Well, whenever my hudson job finishes - they had hudson back then, right? - I can try force pushing my branch using svn-git, but no promises.

 BRB going to the future to see what this magical git squash thing is all about.


Link map and table view when adding and removing features.

This PR fixes #11978 and fixes #11759.

Working pretty good man. I've tested it in local and in staging.


[SPARK-13038] [PySpark] Add load/save to pipeline

What changes were proposed in this pull request?
JIRA issue: https://issues.apache.org/jira/browse/SPARK-13038

Add load/save to PySpark Pipeline and PipelineModel

Add _transfer_stage_to_java() and _transfer_stage_from_java() for JavaWrapper.

How was this patch tested?
Test with doctest.

@jkbradley @yanboliang
In this PR Pipeline and PipelineModel are kept with Python implementations. However, 2 bad things are

It's hard to handle the scenario that Pipeline and PipelineModel themselves are used as stages.

I have to add a public PipelineModel constructor to generate Scala PipelineModel in Python side.

I'll close the previous PR: https://github.com/apache/spark/pull/11266

Test build #53018 has started for PR 11683 at commit f5d750f.

Test build #53018 has finished for PR 11683 at commit f5d750f.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53018/
Test FAILed.

Test build #53021 has started for PR 11683 at commit e2d9a81.

Test build #53021 has finished for PR 11683 at commit e2d9a81.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53021/
Test FAILed.

Merged build finished. Test FAILed.

Test build #53041 has started for PR 11683 at commit 4ce0244.

Test build #53041 has finished for PR 11683 at commit 4ce0244.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53041/
Test PASSed.

I think we have to implement pipeline persistence on the PySpark side because Pipeline and other meta algorithms are implemented directly in Python.
Could you comment on why you need to move Estimator and Model to a different file? It doesn't seem relevant to this PR.

@mengxr Because I add these imports:
from pyspark.ml.util import keyword_only, JavaMLWriter, JavaMLReader
from pyspark.ml.wrapper import JavaWrapper

And these imported classes import Estimator, Transformer, and Model simultaneously. It causes the error of cross reference.
And also, I think it's better to split them because the Estimator, Transformer, and Model are abstract classes, while Pipeline is not.

I think it's better to call save/load from Scala side because we want consistency between Scala/Python implementations. We need to guarantee the model and the pipeline saved by Python/Scala should be loaded via Scala/Python, respectively.
Yes we can implement the pipeline persistence in Python side, but it need more work to keep the consistency.

I'll take a look at this now.  My initial guess is that I agree with @yinxusen about calling save/load from Scala, at least for now.

@yinxusen I added some initial comments.  I think it looks good overall.  However, I do hope to be able to support nested Pipelines, but let's do that in a follow-up PR.  Let me know when to take another look.  Thanks!

test it please

Test build #53159 has started for PR 11683 at commit 168bfc1.

Test build #53159 has finished for PR 11683 at commit 168bfc1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53159/
Test PASSed.

@jkbradley Ready for another look.

I'll add follow-up JIRA for nested pipeline support.

I made a more detailed pass

@jkbradley Thanks! I'll fix them soon.

One more comment: I just spoke with Xiangrui, and he recommended we not add an underscore to the beginning of private class names (since we don't do that elsewhere).  Instead, we can just say that the class is private in the Python docstring.

Sure.
Cheers
Xusen Yin

On Mar 15, 2016, at 1:35 PM, jkbradley<EMAIL_ADDRESS>wrote:
One more comment: I just spoke with Xiangrui, and he recommended we not add an underscore to the beginning of private class names (since we don't do that elsewhere). Instead, we can just say that the class is private in the Python docstring.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/apache/spark/pull/11683#issuecomment-197009204

Test build #53231 has started for PR 11683 at commit 3b2db24.

test it please

Test build #53231 has finished for PR 11683 at commit 3b2db24.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class PipelineMLWriter(JavaMLWriter, JavaWrapper):
class PipelineMLReader(JavaMLReader):
class PipelineModelMLWriter(JavaMLWriter, JavaWrapper):
class PipelineModelMLReader(JavaMLReader):

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53231/
Test PASSed.

@jkbradley Ready for another look

This LGTM.  I'll merge this, but I'll leave the JIRA open for a follow-up PR for handling nested Pipelines.  Thanks!

I'll add new JIRA later.
Sent from my iPhone

On Mar 16, 2016, at 13:50, jkbradley<EMAIL_ADDRESS>wrote:
This LGTM. I'll merge this, but I'll leave the JIRA open for a follow-up PR for handling nested Pipelines. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Hm, if you want to do it under a new JIRA that's OK too.  I'll create one now.

https://issues.apache.org/jira/browse/SPARK-13951


Problem with android 13

Ygg Multicast isn't working in android 13
Tested on miui and divestos

You need to state which phone you have and if it's the factory version of Android or a third party version of Android. I had a Samsung and it was buggy.

Any updates or beta builds?


rio install stuck during stall

Describe the bug
Install rio with just the default rio install this is most commonly happening in Amazon EKS but might be a general issue.
I was trying to validate this other issue: https://github.com/rancher/rio/issues/517
To Reproduce
Steps to reproduce the behavior:

rio install sometimes get stuck during install.
To reproduce this I usually create an Amazon EKS cluster where this seems to be reproducible.

Expected behavior
rio install finishes successfully.
Kubernetes version & type (GKE, on-prem): kubectl version
Amazon EKS
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.6-eks-5047ed", GitCommit:"5047edce664593832e9b889e447ac75ab104f527", GitTreeState:"clean", BuildDate:"2019-08-21T22:32:40Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

Type:
Rio version: rio info
rio version v0.6.0-alpha.1 (7678d67d)

Additional context
rio systemlogs
output:
rio systemlogs
rio-controller | time="2019-10-28T17:01:25Z" level=info msg="Starting rio-controller, version: v0.6.0-alpha.1, git commit: 7678d67d"
rio-controller | time="2019-10-28T17:01:25Z" level=info msg="Updating CRD services.rio.cattle.io"
rio-controller | time="2019-10-28T17:01:25Z" level=info msg="Updating CRD stacks.rio.cattle.io"
rio-controller | I1028 17:01:25.213662       1 leaderelection.go:235] attempting to acquire leader lease  rio-system/rio...
rio-controller | time="2019-10-28T17:01:25Z" level=info msg="listening at :443"
rio-controller | I1028 17:01:25.222774       1 leaderelection.go:245] successfully acquired lease rio-system/rio
rio-controller | time="2019-10-28T17:01:26Z" level=info msg="Starting /v1, Kind=ConfigMap controller"
rio-controller | W1028 17:51:06.013907       1 reflector.go:302] github.com/rancher/rio/pkg/generated/informers/externalversions/factory.go:118: watch of *v1.Service ended with: too old resource version: 2089 (9018)
rio-controller | W1028 18:12:56.559555       1 reflector.go:302] github.com/rancher/rio/pkg/generated/informers/externalversions/factory.go:118: watch of *v1.PublicDomain ended with: too old resource version: 1947 (11815)
rio-controller | W1028 18:56:36.610521       1 reflector.go:302] github.com/rancher/rio/pkg/generated/informers/externalversions/factory.go:118: watch of *v1.Service ended with: too old resource version: 9018 (17418)

This other comment from #517 has more info from the cluster https://github.com/rancher/rio/issues/517#issuecomment-547050917

Available to test in v0.6.0-alpha2.

Install has much better behavior now in v0.6.0-alpha.2. See https://github.com/rancher/rio/issues/599#issuecomment-548165225
Note that clusters without enough space may fail and seem to hang. I noticed with a smaller EKS cluster that only had 17 available pods hit the pod limit and hung during install. After creating a new cluster with more space, it worked fine.


More changes toward trusty.

By default flower and monit shouldn't be exposed, since they are mainly debugging tools.


"no route on the client or the server"

I'm pulling latest code from the repo and installed on Ubuntu 14.04.3.
Meteor 1.2.1
NodeJS: 5.3.0
I get the application up and running fine and I can create a user. However, upon logging in and trying to access any URL in the menu, doesn't matter which one, I'm constantly receiving this message:
Oops, looks like there's no route on the client or the server for url: "http://localhost:3000/rules."

Any ideas as to why this occurs?
Merry Christmas!

This project is still in very early stages. Virtually no routes are in place.


Sigstore v1.2.3 fails with import error

Description
My workflow file was using v1.2.3 and it failed to sign the dists with the import error below. I fixed it by changing the version to v2.1.1.
I'm making this issue because I copied most of this workflow from the official python packaging documentation and they still have it set to v1.2.3 so it might be breaking a lot more workflows.

Version
v1.2.3
RUN:  /home/runner/work/_actions/sigstore/gh-action-sigstore-python/v1.2.3/action.py "./dist/*.tar.gz ./dist/*.whl"
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    GHA_SIGSTORE_PYTHON_IDENTITY_TOKEN:
    GHA_SIGSTORE_PYTHON_SIGNATURE:
    GHA_SIGSTORE_PYTHON_CERTIFICATE:
    GHA_SIGSTORE_PYTHON_BUNDLE:
    GHA_SIGSTORE_PYTHON_OIDC_CLIENT_ID:
    GHA_SIGSTORE_PYTHON_OIDC_CLIENT_SECRET:
    GHA_SIGSTORE_PYTHON_FULCIO_URL:
    GHA_SIGSTORE_PYTHON_REKOR_URL:
    GHA_SIGSTORE_PYTHON_CTFE:
    GHA_SIGSTORE_PYTHON_REKOR_ROOT_PUBKEY:
    GHA_SIGSTORE_PYTHON_STAGING: false
    GHA_SIGSTORE_PYTHON_VERIFY: false
    GHA_SIGSTORE_PYTHON_VERIFY_CERT_IDENTITY:
    GHA_SIGSTORE_PYTHON_VERIFY_OIDC_ISSUER:
    GHA_SIGSTORE_PYTHON_RELEASE_SIGNING_ARTIFACTS: false
    GHA_SIGSTORE_PYTHON_BUNDLE_ONLY: false
    GHA_SIGSTORE_PYTHON_INTERNAL_BE_CAREFUL_DEBUG: false
##[debug]/usr/bin/bash --noprofile --norc -e -o pipefail /home/runner/work/_temp/f66435f8-2a9b-4e0e-a833-a518666e3971.sh
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/runner/.local/lib/python3.10/site-packages/sigstore/__main__.py", line 20, in <module>
    from sigstore._cli import main
  File "/home/runner/.local/lib/python3.10/site-packages/sigstore/_cli.py", line 34, in <module>
    from sigstore._internal.rekor.client import (
  File "/home/runner/.local/lib/python3.10/site-packages/sigstore/_internal/rekor/__init__.py", line 19, in <module>
    from .client import RekorClient
  File "/home/runner/.local/lib/python3.10/site-packages/sigstore/_internal/rekor/client.py", line 33, in <module>
    from sigstore._internal.tuf import TrustUpdater
  File "/home/runner/.local/lib/python3.10/site-packages/sigstore/_internal/tuf.py", line 42, in <module>
    from tuf.ngclient import RequestsFetcher, Updater
  File "/home/runner/.local/lib/python3.10/site-packages/tuf/ngclient/__init__.py", line 14, in <module>
    from tuf.ngclient.updater import Updater
  File "/home/runner/.local/lib/python3.10/site-packages/tuf/ngclient/updater.py", line 48, in <module>
    from tuf.api.metadata import (
  File "/home/runner/.local/lib/python3.10/site-packages/tuf/api/metadata.py", line 56, in <module>
    from securesystemslib import keys as sslib_keys
ImportError: cannot import name 'keys' from 'securesystemslib' (/home/runner/.local/lib/python3.10/site-packages/securesystemslib/__init__.py)

Thanks for the report! This looks like a transitive subdependency breakage, caused by securesystemslib changing their APIs.
We've fixed that as of 2.1.4 on sigstore-python itself: https://github.com/sigstore/sigstore-python/blob/main/CHANGELOG.md#214, but we aren't making new releases on the 1.x series anymore.
Given that, I think the best resolution here is probably to update the version referenced in the official Python docs or (better yet) remove it outright, since the separate action will become obsolete once PEP 740 is complete. I can look into that.

Filed https://github.com/pypa/packaging.python.org/pull/1539 for this upstream.

Fixed in the docs upstream! Thanks @NotJoeMartinez!


Center settings nb

@clarkjefcoat @blaino

Other than spelling and indentation, :+1:

fixed

:+1:


Doesn't generate insert script accurately for columns with " in name

Description
Create a table with a column named ", or " in the column name:
CREATE TABLE table_with_bad_names (
    "space space" VARCHAR,
    "1" INTEGER,
    " " VARCHAR,
    "." VARCHAR,
    "," VARCHAR,
    """" VARCHAR,
    "ヅ" VARCHAR
);

Refresh the table list, then right-click on the table name, click Generate SQL, INSERT. The insert list will not put double quotes around the " column name, so the insert will fail:
INSERT INTO public.table_with_bad_names
("space space", "1", " ", ".", ",", ", ヅ)
VALUES('', 0, '', '', '', '', '');

Result:
SQL Error [42601]: Unterminated identifier started at position 76 in SQL INSERT INTO public.table_with_bad_names
("space space", "1", " ", ".", ",", ", ヅ)
VALUES('', 0, '', '', '', '', ''). Expected " char

DBeaver Version
Community Edition Version <IP_ADDRESS>408181605
Operating System
macOS Sonoma 14.6.1
Database and driver
Postgres 16
Steps to reproduce
Create a table with a column named ", or " in the column name:
CREATE TABLE table_with_bad_names (
    "space space" VARCHAR,
    "1" INTEGER,
    " " VARCHAR,
    "." VARCHAR,
    "," VARCHAR,
    """" VARCHAR,
    "ヅ" VARCHAR
);

Refresh the table list, then right-click on the table name, click Generate SQL, INSERT. The insert list will not put double quotes around the " column name, so the insert will fail:
INSERT INTO public.table_with_bad_names
("space space", "1", " ", ".", ",", ", ヅ)
VALUES('', 0, '', '', '', '', '');

Result:
SQL Error [42601]: Unterminated identifier started at position 76 in SQL INSERT INTO public.table_with_bad_names
("space space", "1", " ", ".", ",", ", ヅ)
VALUES('', 0, '', '', '', '', ''). Expected " char

Additional context
No response

Thank you for bug report.
I reproduced the issue


AAC to Opus transcoding.

Hey folks! I'm trying to transcode AAC file to Opus and then write it to a file. But currently facing an issue - i'm not able to play the file via vlc\ffplay\etc.
For example that's the output from ffprobe -

[m4v @ 0x5630c0e255c0] Format m4v detected only with low score of 5, misdetection possible!
[mpeg4 @ 0x5630c0e27340] time_increment_bits 0 is invalid in relation to the current bitstream, this is likely caused by a missing VOL header
[mpeg4 @ 0x5630c0e27340] time_increment_bits set to 8 bits, based on bitstream analysis
[mpeg4 @ 0x5630c0e27340] time_increment_bits 4 is invalid in relation to the current bitstream, this is likely caused by a missing VOL header
[mpeg4 @ 0x5630c0e27340] time_increment_bits set to 8 bits, based on bitstream analysis
[mpeg4 @ 0x5630c0e27340] looks like this file was encoded with (divx4/(old)xvid/opendivx) -> forcing low_delay flag
[mpeg4 @ 0x5630c0e27340] Marker bit missing at 75 of 3684536 before width
[mpeg4 @ 0x5630c0e27340] Marker bit missing at 89 of 3684536 before height
[mpeg4 @ 0x5630c0e27340] N-bit not supported
[mpeg4 @ 0x5630c0e27340] quant precision 4
[mpeg4 @ 0x5630c0e27340] Invalid Complexity estimation method 3
[mpeg4 @ 0x5630c0e27340] reduced resolution VOP not supported
[mpeg4 @ 0x5630c0e27340] scalability not supported

What's interesting. All the same pipeline works completely fine for AAC->MP3 transcoding.
Can you please tell what i'm doing wrong ? Thanks in advance!
AAC -> MP3 transcoding - https://gist.github.com/Lighty0410/5c6bcf02aae5ac348736f32eda4da287
AAC -> OPUS transcoding. The only difference here is the sample format both for audio encoder\resampler and codec for AudioEncoder  - https://gist.github.com/Lighty0410/f01193891f51122f82324975742b3c29

Hi @Lighty0410!
I don't have a first-hand experience with Opus but based on the output from ffprobe, I think that you cannot simply put Opus encoded audio into a file (unlike MP3 which is an audio codec and also a file format). The ffprobe output indicates that ffprobe tried to interpret the file as m4v (i.e. MP4 video) and failed.
Personally, I'd try adding ".opus" at the end of the file name. It's possible that ffprobe/ffplay/vlc will then be able to guess the format based on the filename extension. If it doesn't help, you'll probably have to mux the encoded audio into one of the supported containers. Quick googling shows that Ogg, WebM, Matroska and MPEG-TS support Opus audio.

@operutka Yeah, i tried to create a file with .opus extension. All the same result. Seems like i really need to put the data to a container indeed.
Another interesting things:

ffmpeg -i file.aac -c:a libopus opus_file.opus works fine and the audio plays fine with just .opus extension.
Even if i set the the file extension to .opus ffprobe still think it's mp4v.
Thanks a lot for help!

@operutka so, i tried using a muxer. And now there's another problem. The application just stuck.
https://github.com/angelcam/rust-ac-ffmpeg/blob/7a7ff68a621a8b2e8810de88748b237180e0feaf/src/format/muxer.rs#L157 - here's the line where everything stops.
Here's what i'm doing - https://gist.github.com/Lighty0410/e67e936410bb20b7d2dd780ba90777f6

@Lighty0410 the problem is here:
https://gist.github.com/Lighty0410/e67e936410bb20b7d2dd780ba90777f6#file-aac_opus-rs-L25
You're trying to create a muxer using a read stream (the ffmpeg can't write anything). I'm not sure why it doesn't end with an error, you'd probably have to dive into the ffmpeg code to find the exact reason. Anyway, you should create your muxer IO like this:
let io = IO::from_seekable_write_stream(output_file);

@operutka Oh god....  Just a typo...
I leave it here. Maybe it will be useful for someone in the future and google will se it - https://gist.github.com/Lighty0410/13104e7357b3121998caa6969110e818
Thank you a lot <3.


updating okta docs

Please look over these docs and let me know if you have any questions. You can check out the code locally by running yarn start after navigating into the repo following branch pull.

There are some spellcheck pre commit hooks we can add as well in the future


reference: replace deprecated function SplitHostname

Forward-porting the patch from https://github.com/containerd/containerd/commit/694a007543fafe751cbfab07093ba1b3ae58f1b6 (https://github.com/containerd/containerd/pull/5922)

Codecov Report
Base: 57.06% // Head: 56.87% // Decreases project coverage by -0.18% :warning:

Coverage data is based on head (2da7af3) compared to base (c47a966).
Patch coverage: 65.55% of modified lines in pull request are covered.

:exclamation: Current head 2da7af3 differs from pull request most recent head 3c71f49. Consider uploading reports for the commit 3c71f49 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3776      +/-   ##
==========================================
- Coverage   57.06%   56.87%   -0.19%
==========================================
  Files         105      105
  Lines       10806    10804       -2
==========================================
- Hits         6166     6145      -21
- Misses       3952     3969      +17
- Partials      688      690       +2

Impacted Files
Coverage Δ

configuration/configuration.go
64.38% <ø> (ø)

context/trace.go
93.47% <ø> (ø)

contrib/token-server/main.go
0.00% <0.00%> (ø)

manifest/ocischema/manifest.go
74.19% <ø> (ø)

manifest/schema1/manifest.go
33.82% <ø> (ø)

manifest/schema1/reference_builder.go
94.00% <ø> (ø)

manifest/schema2/manifest.go
80.00% <ø> (-0.40%)
:arrow_down:

reference/reference.go
78.28% <0.00%> (-0.80%)
:arrow_down:

registry/api/errcode/register.go
30.76% <ø> (ø)

registry/api/v2/descriptors.go
100.00% <ø> (ø)

... and 35 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@corhere PTAL 🤗

@milosgajdos looks like @corhere is on the maintainers list, but does not have the right permissions; https://github.com/distribution/distribution/blob/e9a25da7a47ed0eae24f51ff5e8dd3f79591b301/MAINTAINERS#L9
Can we fix that? ❤

Thanks!


Add skipKeyValidation option (to slshx handler) to skip key validation

Sometimes, it's useful to skip key validation for some reason (for example, I'm validating keys and creating a cooldown system for commands) This PR adds an option to do just that. Haven't tested it out yet, but IMO it should work.

What I'm trying to do is verify the signatures myself so that I can do some logic before handling the interaction, a custom fetch handler would be spoofable (it's run for each initial user) or perform signature validation twice


Solves the FrozenLake RL problem via Q-learning.

This model does not use a neural net, and instead demonstrates Swift host-side
numeric processing as well as python integration.
This model took ~30 seconds to solve on my Linux desktop (CPU only computation).
https://bugs.swift.org/browse/TF-170

Thanks @dan-zheng for the quick review!

Thanks all for the review!


Feature Request: Parallel Scrolling

A feature that would be extremely helpful is mirrored scrolling - aka, if scrolling in the preview, being able to scroll alongside with that portion of code, or vice versa (selection in the code scrolling to the place of preview).
This would mimic browser DevTools while having the convenience of your active code editor present for updates and changes.

duplicate of https://github.com/microsoft/vscode-livepreview/issues/214


New package: Juice v0.2.2

Registering package: Juice
Repository: https://github.com/Juice-jl/Juice.jl
Created by: @khosravipasha
Version: v0.2.2
Commit: bbd0ae83d0ee60a3007bb6e7cf48f2649b1267e9
Git reference: main
Release notes:

The Juice package for circuits in artificial intelligence, both logic and probabilistic

Fixed the mentioned issues, but did not realize changing the version and resubmitting makes another pull request. Can close this pull request.


support for custom column names

Unfortunately sequelize-hierarchy does not respect settings for custom field names. This can be replicate by simply changing the folder model definition in the default test suite to something like that:
id: {
  type: DataTypes.INTEGER, primaryKey: true, autoIncrement: true,
  field: 'folder_id',
}

The result will be an error: SequelizeDatabaseError: ER_BAD_FIELD_ERROR: Unknown column 'id' in 'IN/ALL/ANY subquery'
Caused by this query:
UPDATE `tFolder` SET `hierarchy_level` = `hierarchy_level` + -2 WHERE `id` IN ( SELECT `folder_id` FROM `tFolderAncestors` AS ancestors WHERE ancestors.`ancestor_id` = 104)

This is a pretty serious issue cause it affects everybody who is not using default column names.

BTW: I can try work on a PR if you give me an idea where to look at.

I think this is fixed in v0.7.5 (just released). Can you please test it out and let me know?
NB The primaryKey, foreignKey, throughKey, throughForeignKey and levelFieldName options should contain the model field name, NOT the table field name. i.e. in your case options.primaryKey = 'id' (the default).
In truth, there should be more tests for this module, which would have caught issues like this - but I don't have time to write them!

@php-workx Can you please confirm whether v0.7.5 solved this issue for you?

looks good to me. thx!


Suggestion to modify code in Sprite.rotate() method in com.badlogic.gdx.graphics.g2d.Sprite package

regarding the method of Sprite.rotate() as mentioned, did you forgot to modulate the angle given with fElapsedTime i.e deltaTime?
When I see the code, I notice that you would just add the current angle with the given angle without any integration with deltaTime.
This is critical, because the function was called every frame. Without this modulation, the angle would increase by a large amount every frame.
I had to integrate two times in my code, so that to establish a fairly acceptable movement of my game object.
Also, that angle that supposed to be supplied to the function need to be in degree as per the documentation.
When I print the value of the angle I got a weird critical value of 1.4370242. It was supposed to be 360 degree but it printed very weird value.

Was this supposed to be a comment somewhere else? It looks like gibberish out of context.

It is indeed an issue at least for me and potentially for some one else that are new to this library.
To give some perspective on this weird behaviour of Sprite.rotate() for my case though, you need to instantiate the minimal
Sprite class and SpriteBatch class.
That's it.
Forgot to mention this phenomena happen in both Android app and desktop app I am trying to build.
Then you draw it in the render() method,
SpriteBatch.begin()
Sprite.draw(SpriteBatch)
SpriteBatch.end()
Then, when handling user input by polling technique, we handle it also in render() method.For example if the user presses right key, we would like to increment the angle by one degree per second in real time.
To do that, we integrate with delta time like so:
(For the sake of this discussion, angle is just one of the property of Game or Screen class being implemented.
And initialize it with 0.0f with float data type.)
angle+=1*deltaTime
Then we pass this angle variable to that Sprite.rotate() method like so:
Sprite.rotate(angle)
Then we try to print the angle's value at runtime:
println(angle)
After compiling the code, I realized that when I pressed the right key it rotates to the left or anti-clockwise.
I noticed two things that are weird.
Firstly, the sprite seems to get faster and faster as as time passes by. Initially, the rotation is not that fast but it gets faster and faster the longer the duration of the key being pressed.
Second, is that, when the angle's value is being monitored in the console for the first full rotation.
It is the value that I mentioned before 1.4xxx is printed out when the sprite is completely being rotated 360 degree anti clockwise.
The value should be 360 degree right? since the documentation says that you pass the angle in degree.
I don't know if Sprite.rotate() is not supposed to be manually called without using the OrthographicCamera instance.
In my case, I don't use that class.
Hopefully, this clear things up.
Regarding this post, is not this post should be in the comment section or an issue?
I don't know for sure.
Because I am not a professional programmer and I do not know too much about GitHub.
If you think it should be in the comment section than it's up to you.
But , I still want to send this message out so that something could be done?
As it is an open source library?
Or is it already perfect ?

I don't really follow, but if I understand what you're suggesting, Sprite should absolutely not consider delta time for a rotation. If you tell it to rotate by 20 degrees, it rotates by 20 degrees, not some fraction of 20 degrees that depends on the user's machine and workload. There are lots of uses for libGDX that would be fundamentally incompatible with this change, not to mention that it would break all code using Sprite.rotate() currently. Briefly delving into your second comment, you're using rotate(), which is relative, not setRotation(), which is absolute. You're also increasing angle every frame, which would be correct for an absolute change, but not a relative one (unless you want uncontrolled speedup). What you're printing is angle from your code, not the actual angle the Sprite is rotated at. You could use Sprite.getRotation() to find that out. In your code, angle is the relative change between the last rotation and the current rotation, not the current angle of the Sprite.
We have a Discord for this type of question where there isn't actually an issue in libGDX. Here's the link. I'm closing this.

For example if the user presses right key, we would like to increment the angle by one degree per second in real time.
To do that, we integrate with delta time like so: [...]
angle+=1*deltaTime
Then we pass this angle variable to that Sprite.rotate() method like so:
Sprite.rotate(angle)

Sprite#rotate(float) takes the existing rotation into account, so in your case you're applying the angle multiple times. You should use Sprite#setRotation(float) instead. Also, for questions like this, our Discord server is the better place to seek help.


Picture does not show

Hydrogen renders it. It did need a first click though, but then it shows in small as well.

https://archive.matrix.org/r/fun:grin.hu/date/2023/04/12T00:12:47?at=$cE4cZAY58PHudxyXdqR5dk648_HsBfDoGmazkjVSYuM

{
  "content": {
    "body": "1.jpg",
    "info": {
      "h": 609,
      "mimetype": "image/jpeg",
      "size": 39748,
      "w": 500,
      "xyz.amorgan.blurhash": "TENwV+-q~XxoWCWBRn-rt5M}xtt6"
    },
    "msgtype": "m.image",
    "url": "mxc://oregpreshaz.eu/GbhXUxEGpzNdxKlRwYYTUObT"
  },
  "origin_server_ts":<PHONE_NUMBER>737,
  "room_id": "!CwMDVzZXiVUzvlBHdn:grin.hu",
  "sender": "@dani:oregpreshaz.eu",
  "type": "m.room.message",
  "unsigned": {},
  "event_id": "$cE4cZAY58PHudxyXdqR5dk648_HsBfDoGmazkjVSYuM",
  "user_id": "@dani:oregpreshaz.eu"
}

The Matrix Public Archive uses Hydrogen under the hood and this is an upstream bug in Hydrogen itself. Since that project is in another GitHub org, I can't transfer the issue over there. Please create an issue on that repo with these details.

How is this an upstream bug if I just noted explicitly that I'd tested it and it works on Hydrogen for me?

I can reproduce this bug in Hydrogen itself.

It may be related to your home server. Try clicking the picture first. It will then appear to keep working indefinitely.

https://matrix.to/#/!SzoPnANsRYxITaDPaJ:matrix.org/$R8wi2K9a6VnYmvyZI4AlDhRG1ah86yqNIXexcAY_pQQ?via=matrix.org&via=grin.hu&via=pikaviestin.fi
https://archive.matrix.org/r/matrix-public-archive:matrix.org/date/2023/05/30?at=$R8wi2K9a6VnYmvyZI4AlDhRG1ah86yqNIXexcAY_pQQ

Screenshot on Hydrogen:

@bkil I can reproduce from a matrix.org account with Hydrogen. matrix.org would be the same homeserver that https://archive.matrix.org/ uses.
This actually just appears to be a problem with the media repo not being able to download the content from origin homeserver.
Hydrogen with a matrix.org account:
GET https://matrix-client.matrix.org/_matrix/media/r0/thumbnail/oregpreshaz.eu/GbhXUxEGpzNdxKlRwYYTUObT?width=246&height=300&method=scale -> 400
https://archive.matrix.org/
GET https://matrix.org/_matrix/media/r0/thumbnail/oregpreshaz.eu/GbhXUxEGpzNdxKlRwYYTUObT?width=246&height=300&method=scale -> 400

How come it works on all other HS? Is this a Synapse federation bug then?
https://tchncs.de/_matrix/media/r0/thumbnail/oregpreshaz.eu/GbhXUxEGpzNdxKlRwYYTUObT?width=246&height=300&method=scale

Okay, I've created the issue after conducting further root cause analysis:

https://github.com/vector-im/hydrogen-web/issues/1110


AOT compilation error

When building production app in  Angular 4.0.1 following error occur:
ERROR in node_modules/md2/colorpicker/colorpicker.ngfactory.ts (1,1): Property 'isInputValidColor' is private and only accessible within class 'Md2Colorpicker'

to update 'MD2', version 0.0.18


Localization for Numbers

Numerical inputs add a selectable pattern in a global context with option for "." and ","
allow depending on localisation: 123456789,12345
allow depending on localisation: 123456789.12345
Include option of spaces as a divider. This way users are not confused by multiple "." or "," ( e.g. 123.456.789,12 or 123,456,789.12)
allow depending on localisation: 123 456 789,12345
allow depending on localisation: 123<PHONE_NUMBER>5
Add the ability to limit the number of characters before and after a "." or ",". This would ensure that people only enter valid numbers, and would reduce errors.
e.g. 9 characters before and 2 after "." or ","
allow depending on localisation: 123456789,12
allow depending on localisation: 123456789.12

I think this can be achieved using autoNumeric.

@enowak1031 @coliff Some pattern to be followed for the currency part ?(Initial Release)


Revisit Functional Test Pattern Part 7: The Changed Monster

Refactor MonsterFixupTestBase to have the change notification strategy configuration in the fixture
Test only one change notification strategy per provider as the behavior is not provider-specific
Extract more methods to TestStore
Speed up ExecutionStrategyTest
Part of #9098

:shipit:


Add table aws_backup_selection

References
https://docs.aws.amazon.com/cli/latest/reference/backup/get-backup-selection.html
https://docs.aws.amazon.com/cli/latest/reference/backup/list-backup-selections.html
Terraform - https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/backup_selection

Checks whether Amazon RDS database is present in back plans of AWS Backup