Unexpected String Encoding Behavior in 0.3.1

I've noticed some unexpected behavior in version 0.3.1. It seems that ffi returns a binary string, which causes all strings from Crystal to have the Encoding: ASCII-8BIT by default in Ruby.
Here’s a minimal example to demonstrate the issue:
require "crystalruby"

module StringManipulation
  crystallize :string
  def output
    "foo різних мов світу"
  end
end

str = StringManipulation.output
p str
p str.encoding
p str.force_encoding("UTF-8")

Output:
$ bundle exec ruby examples/string/encoding.rb
"foo \xD1\x80\xD1\x96\xD0\xB7\xD0\xBD\xD0\xB8\xD1\x85 \xD0\xBC\xD0\xBE\xD0\xB2 \xD1\x81\xD0\xB2\xD1\x96\xD1\x82\xD1\x83"
#<Encoding:ASCII-8BIT>
"foo різних мов світу"

In Crystal, strings are UTF-8 encoded by default.
I’m wondering if there’s a way to ensure that UTF-8 encoding is set by default when strings are passed through ffi.
Any guidance or workaround would be appreciated.

Thanks for reporting! I've forced a default encoding of UTF-8 for strings in 0.3.2, which I think is a sound default given it matches that of Crystal.


I have added crazy emojis

i have done the crazy emojis and resolved the issues.
@vansh-codes
please close this issue under probelm #206

Bro you have to fix the conflicts on that PR only, it doesn't works this way
this PR is not valid and I cannot merge it.
you #206 has 100+ lines of code changed and this has only 3 lines changed
Please fix the conflicts on PR #206 or do one thing, sync fork and do the changes in this one and commit


[DNM] exclusive workflow for docking

Description
Testing if separating workflows for packages fixes our CI. Do not merge.

Codecov Report

Merging #493 (1b89f73) into main (e83baa9) will decrease coverage by 41.94%.
The diff coverage is n/a.

Additional details and impacted files

Apparently testing docking needs every other asapdiscovery subpackage. I think we probably need to come up with more unitary tests for the docking software.

Legitimate failure #496 in single-target

I guess another advantage of separating the workflows like this is that it should be easier to know where the failure is (which subpackage tests are failing)

@ijpulidos just while I think of it also need asapdiscovery-alchemy

@hmacdope Oh definitely! haha, I missed that one. Thanks!
Simulation and docking tests should fail (simulation isn't testing anything and docking has a legitimate fail).

Merging to main, this fixes the segmentation fault issue with the costs of having many separate workflows to maintain now.
We can probably review this in the future if needed, if it becomes too inconvenient to have this amount of granularity.
The macos test that is failing was also failing before these changes, so it's something unrelated to these changes. We probably need to skip that one in macos as well.


emphasize blockquotes

...using italics; reverse italics within to be upright.

Probably not a good idea actually: blockquotes are blocks and too much emphasis counters the very notion of emphasis. The bar to the left should provide more than enough emphasis.

Maybe make the text a tad lighter could work. See http://hpurmann.com/2015/03/28/git-moving-commits-to-another-branch/ for an example.


Reconnect Wifi Client via iobroker

Hez,
in the Unifi App and Webinterface you have the option to reconnect a WiFi client initiated by the AP. This is useful if you have IoT devices that sometime hangup in the WiFi connection. (can be recognized if the "experience" value of the client goes down)
If you press "reconnect" the AP drops the connection of the client and the client reconnects and everything is fine. I would like to automated this "reconnect", but the adapter is missing this functionality.

PR #597

implemented in #597
issue can be closed

I think the issue can be closed #597


Delete key when renaming file do delete file

When you rename a file, if you select a part of the name and that you press the delete key this launch the action to delete the file.
Renaming action isn't concidered when delete key is pressed and launch the delete file action

It seems OK now, reopen if needed.
Thx.


[@azure/core-http]/[@azure/core-client] - Remove Constraints Check

Packages impacted by this PR
@azure/core-http (For Track 1) & @azure/core-client (For Track 2). Thus, by extension, all the packages dependent on these package.
Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/issues/21839
Describe the problem that is addressed by this PR
Basically, the problem started with this issue. One of the customers reported that there is an issue with a way multipleOf constraint is handled. That is because of the % operator. While working through this, we have decided  that the constraints check can be removed altogether. I have already confirmed with other language owners that they do not perform constraints check.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
There are 2 design options.
Option 1
Remove the constraints method altogether. This will include removal of method from Serializer Interface and will cause breaking changes.
Option 2
Just do not call the constraints method. This will not cause break in code. But there is a slight possibility that this might cause break in behavior for custom serializers. But, that can be justified with reason that we are not going to perform any constraints validation.
So, I have decided to go with option 2.
Are there test cases added in this PR? (If not, why?)
No. Existing test cases are fine.
Provide a list of related PRs (if any)
None
Checklists

[X] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

@xirzec Please review and approve the PR.

API change check
API changes are not detected in this pull request.

I am happy with disabling constraint checking because I was burnt by it before in Schema Registry and had to remove the constraints from the swagger for the client to work properly: https://github.com/Azure/azure-sdk-for-js/pull/18666.
However, I am on the fence on the taken approach to do so since constraint checking is part of the public surface of the core libraries as Sarangan pointed out. Disabling constraint checking in core but leaving its knobs in the public surface could be confusing or even misleading. Tagging them with @deprecated could help alleviate this but I am not sure if it is ok to for a deprecated feature to be outright disabled.
An alternative approach is to update the code generator to not add constraints in the generated mappers as discussed in https://github.com/Azure/autorest.typescript/issues/829 but then not every library will enjoy the benefit unless it gets regenerated.
I don't have strong opinions either way but I just wanted to point it out.

However, I am on the fence on the taken approach to do so since constraint checking is part of the public surface of the core libraries as Sarangan pointed out. Disabling constraint checking in core but leaving its knobs in the public surface could be confusing or even misleading. Tagging them with @deprecated could help alleviate this but I am not sure if it is ok to for a deprecated feature to be outright disabled.

I think from a correctness POV, you're right, but pragmatically nobody is relying on constraint checking failing today, so the runtime behavior should be largely unaffected for SDKs we've shipped.

We are also seeing failure in Digital Twins models creation path, since that is checking for MinItems constraint while passing empty values.


Add argument null validation to Aspire.Hosting

Description
Fixes https://github.com/dotnet/aspire/issues/2649
Checklist

Is this feature complete?

[x] Yes. Ready to ship.
[ ] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[ ] Yes
[x] No

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[x] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[x] No

Microsoft Reviewers: Open in CodeFlow

There is a test failure from https://github.com/dotnet/aspire/blob/4e7e0de000bb83f824cca2773892a22b5da89245/src/Aspire.Hosting/ContainerResourceBuilderExtensions.cs#L317
Which is wrong: WithImageRegistry not allowing null or WithDockerfile calling WithImageRegistry with a null value?

Ok. So, that means WithImageRegistry should allow a null argument?
On the surface that seems strange to me. Why would someone call it with null rather than just not call it? If it is changed to accept null, what should the doc comment say if you pass null to it?
I'm not fimilar with this API. I'd like some actionable feedback.

@mitchdenny What is the answer here?

Given the API has shipped allow null to be passed to WithImageRegistry.

I don't think it being shipped should stop us. It shipped without a nullable argument.
What is the best behavior?

/azp run

Kicking new build to see what the test failures were.

Flagging this as a breaking change as we did technically change the signature, although most folks won't notice if they just recompile.


Issue on the hotkey to fill in username and password

Hello
I have the following config:
  -
    hotkey: 'ctrl alt shift p'
    action: 'decrypt-password'
    options:
        copy-to-clipboard: false
        type-username: true
        type-password: true

When I use the hotkey doesn't fill in the username, only fills in the password. For example, I go to gitlab.com login page, click on the username then press the crtl+alt+shift+p hotkey, and it appears the menu to choose the password, I click on one and I got the following:

The username and the "tab" is missing. This situation happens with other sites also. The issue started when I updated the pass-winmenu but now I don't remember which version works. I've tested previous versions with no luck. What could be wrong?
Any help will be appreciated
Using:

pass-winmenu v1.13
Win10
Chrome 104.0.5112.102 (Build oficial) (64 bits)

are you able to repeatedly experience this problem? it could be something accidentally stole focus as it was trying to be pasted
you could also try an earlier version of pass-winmenu as it seems to work for me

Thanks for the reply. I'll close the issue because I was using the wrong 'username-detection' config. Now it's working


GraphQL malfunctions when deployed to Firebase after newest changes

Something changed in https://github.com/babichjacob/sapper-firebase-typescript-graphql-tailwindcss-actions-template/commit/793681db63c3fd9a588ca88f39886b3d781e34b8 made GraphQL queries sometimes work and other times give
{
  "error": "Response not successful: Received status code 400"
}

Fixed by da64e8e


Added links and new cars

The price of cars are of new models and petrol fuel

@grow-xd Please arrange cars in ascending order of their price.

done


New TIR example, fix old 6SV examples, general cleanup

No major changes to functionality
Standardized the fit_params() interface
Removed vestigial surf_emissive.py
New thermal infrared example for unit testing
PEP8 line width compliance

Bug fixes are great, good catches.  Code cleanup looks good as well, thanks for doing that!
New example is helpful, but I think it needs a bit of cleanup.  Suggested changes:

remove LUT json files (brings consistency with other examples, removes full-path references, and reduces number of repo files)
remove the data subdirectory, for consistency with other examples
move any relevant files from the data directory to the main example data directory
reduce the number of wavelength files (I count 5 new)
convert example to use full 224 AVCL bands....this isn't required but by dropping the modtran lower bound down to 340 this runs fine, and prevents additional repository files and makes the example more useable - unless there's another reason to do this?

Do authors have time for these, or should I take them on?
As an aside, we're also beginning to incur a bit of size debt with the number of LUT elements we have.  Fine for now, but we should keep an eye on it and consider alternative strategies for data downloads.

All done, Except for AVIRIS-C; I left it at 223 channels to avoid recalculating the lookup tables.  Otherwise, I reorganized as suggested.  Note that, with the dual instrument retrieval, we need three wavelength files (one for each instrument independently and the combination).

Excellent, thanks!  I'll start the merge now then.


internal: Increase codecov consistency

Getting flakes, reduce concurrency a step

Codecov Report

Merging #1508 (b2f25d3) into master (7d2940f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1508   +/-   ##
=======================================
  Coverage   98.62%   98.62%
=======================================
  Files          99       99
  Lines        1824     1824
  Branches      361      361
=======================================
  Hits         1799     1799
  Misses          5        5
  Partials       20       20

Continue to review full report at Codecov.

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

Closing in favor of #1519


Center the content in the boxes

@HimanshuNarware

Issue:
I would like to set the content in all four boxes to center .
Description:
Currently, the content in the boxes is aligned in diffrernt places .I propose that we modify the CSS to center the content within each box and add titles and descriptions to improve clarity and aesthetics.

I request you to assign the above issue to me.

Actually, its already been raised here and assigned to someone #96


Connection unexpectedly closed: timed out

Hello,
I got an error when running following code on Colab:
from mailscout import Scout
scout = Scout()

names = ["Batuhan", "Akyazı"]
# or, names = ["Batuhan Akyazı"]
domain = "microsoft.com"

emails = scout.find_valid_emails(domain, names)

print(emails)

Screenshot on Colab

Having the same issue

Hey,
Sorry to hear you are having problems with mailscout.
When I check the code on my own Colab, I face no issues. As expected, it returns an empty list (as Batuhan Akyazı does not have a microsoft.com account), however, it checks the emails successfully. I can also get a full list with legit domain + name combinations.
Since I can't replicate the issue, I can only guess the reason behind it.
One reason might be that the Colab instance you use has been IP-banned by the mail server. Using a different instance may help in this case (by simply using the Disconnect and Delete Runtime button). Mailscout is also prone to getting your IP banned if used in high concurrency (simply because it means spamming the server), but I believe that's not your case.
I would also suspect the port 25 on the instance you use. But I suppose it would give you a specific error if that was your case. I use Colab Pro so my instances may allow me to use the mentioned port, while Colab Free may block it by default.
Again, these are just my assumptions as I am unable to replicate the issue.
If you have found more evidence to elaborate on, I would be more than welcome to provide further support :)


[Bugfix][Transform] Keep private non-primitive functions in FuseTIR

Prior to this commit, private non-primitive relax functions would be discarded by FuseTIR.  If any calls to these functions exist, the resulting IRModule would be ill-formed.  This commit updates FuseTIR so that it only applies updates to functions with attr::kPrimitive, and calls into those functions.
To retain backwards compatibility, DeadCodeElimination is applied as a post-processing step.

cc @Hzfengsy


Implement Info API

Codecov Report

Merging #72 (c162d2c) into main (be11c84) will decrease coverage by 0.03%.
The diff coverage is 81.25%.

@@            Coverage Diff             @@
##             main      #72      +/-   ##
==========================================
- Coverage   83.82%   83.79%   -0.04%
==========================================
  Files          15       15
  Lines        1255     1271      +16
==========================================
+ Hits         1052     1065      +13
- Misses        143      145       +2
- Partials       60       61       +1

Impacted Files
Coverage Δ

driver/driver.go
96.87% <ø> (ø)

driver/internal.go
100.00% <ø> (ø)

filter/eq.go
100.00% <ø> (ø)

projection/projection.go
100.00% <ø> (ø)

schema/model.go
0.00% <ø> (ø)

tigris/collection.go
76.69% <ø> (ø)

tigris/database.go
59.74% <ø> (ø)

tigris/iterator.go
80.76% <ø> (ø)

update/set.go
100.00% <ø> (ø)

update/update.go
100.00% <ø> (ø)

... and 2 more

Continue to review full report at Codecov.

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


Merge unnoficial IntelliJ plugin

Currently includes:

The plugin base
Suppression of "Unused" warnings

Note that I'll probably downgrade the version for a bit, maybe 0.6 or something, but we can do that after merging.

I think that could be a little problematic. I actually increased the min version, because the with default version used by the project wizard, the kotlin plugin is incompatible with more recent versions of Intellij (see da380bd). You can try getting it as low as possible though

I think [downgrading the version] could be a little problematic.

I think we're talking about different versions? I meant this one: https://github.com/varabyte/kobweb-intellij-plugin/commit/76599d1de4571d6ce6619a66c86b7c8a32ae6b90

Oh yes lol. Turns out 2am is too late to comprehend comments. Downgrading that version is absolutely fine


Implement TextQuestion

Current dependencies on/for this PR:

main

PR #49

PR #50

PR #51

PR #52   👈

This comment was auto-generated by Graphite.


Filter Sign-In Activities by the 'Department' field

We need to be able to filter the Sign-In Activity Reports/Logs by the DEPARTMENT field. We are currently utilizing the DEPARTMENT field in a User's Profile in Azure AD but there is no way to filter those Activity logs using that field. Can you add that capability?

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

ID: bbc77eed-9bcb-d9a8-4d0c-52095b6d0aa7
Version Independent ID: 187f48b4-ae4f-86a2-e7e8-93141aa4f615
Content: Sign-in activity reports in the Azure Active Directory portal
Content Source: articles/active-directory/active-directory-reporting-activity-sign-ins.md
Service: active-directory
GitHub Login: @priyamohanram
Microsoft Alias: priyamo

@gauchan183 Thanks for your feedback! We will investigate and update as appropriate.

@gauchan183 Please post this as a feedback https://feedback.azure.com/forums/169401-azure-active-directory This will allow the community to upvote and for the product team to include into their plans. We also have the "Give Product Feedback" button in our documents now to take you directly to the appropriate feedback page.

@gauchan183 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


⚠ mastodon.cloud has degraded performance

In abdbc10, mastodon.cloud (https://mastodon.cloud/health) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: mastodon.cloud performance has improved in f33169f.


DB admin worked out

Transcript of the sticky note:
DB admin worked out

initialization
live
Every action becomes a "request for action"

(I think what this means is that if anyone wants to make a database-admin-level change to the database, they must first do a request to the whole federation and then all the other nodes must vote or whatever.)
This is closely related to cluster configuration management (issue #436) and the wire protocol firewall (issue #594).

@ttmc I believe this is a duplicate of https://github.com/bigchaindb/bigchaindb/issues/598?

Closed as a duplicate of #240


Farewell to FRP in Blog - Websocket not always working

Hi there,
I was reading the blog post and I had an issue running the example.
The problem is that here you are using ws:// protocol where sometimes (using proxies for instance) the secured one is required.
From Websocket.org:

Note: In some environments the WebSocket connection may fail due to intermediary firewalls, proxies, routers, etc. In that case take advantage of WebSocket's secure capability and check Use secure WebSocket (TLS). Even if you have no issues you can still feel free to test using a secure connection.

In should be simple to detect the failure and fall back to wss:// I can't work on that now as I'm in the office.
Another solution would be just replace it and always use the secured one.

It is using wss now. Thanks for the report!


upgrade to symfony v4.2

@andrzejkupczyk where is the problem to try it without the PR?

Latest deptrac release conflicts with symfony v4.2 but I get your point - I can surely use master branch just to try out this package, thank you :+1:


docs: Enhance dashmate README with Devnet Example

Issue being fixed or feature implemented
As a developer new to Dash / Platform it was hard to find this information for how to make use of a devnet.
Hopefully, this makes it easier to understand how to use your own devnet for developing / building / testing.
What was done?
Consolidated documentation from https://dashplatform.readme.io/docs/tutorial-connecting-to-testnet#connect-to-a-devnet into code examples and links and placed in the docs where you would likely need a reference to them.
How Has This Been Tested?
I previewed the changes in Github
Breaking Changes
None
Checklist:

[x] I have performed a self-review of my own code
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

Please resolve the conflicts

I will close it for now, please feel free to open it again when you have time


fix(ci): Return an error code on the failed AST Changes step.

Closes #4929

#4930  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rzvxa and the rest of your teammates on  Graphite

This a severe issue so I'm merging it without review to prevent any faulty code from getting in to upstream.


How do I use this in a game?

I understand this is a demo but where would I start in taking it out as a usable threejs gameobject to work with?

Not sure what you mean by a 'gameobject', but the Terrain class extends Object3D so can't you just add this to your scene?  https://github.com/felixpalmer/lod-terrain/blob/master/js/app/terrain.js


Support VSCode.dev

I know it's a longshot - but it would be super cool if we could use the Tailscale extension in the VSCode.dev browser environment... this would allow me to connect to my dev sandbox even when I can't install the tailscale client.

It works on gitpod and with coder's code-server.
vscode.dev for github codespaces should be posible, but not the web-only editor


[FEATURE] It would be nice to have a command ready to clear the label given by the Set-PnPSiteSensitivityLabel command

The Set-PnPSiteSensitivityLabel command can be used to label a site with respect to the site. We find it very useful to be able to label sites with this command when there are a large number of sites.
However, if you want to remove the label after it has been assigned with this command, the only way to do so is to manually change the label from the browser. It would be nice if there was a way to do this with a command.

Hello @yoinuma , we have Remove-PnPSiteSensitivityLabel cmdlet which should work for you.

Thanks for the quick reply!
I see that the command already existed. Sorry for the lack of confirmation, I could not find the command Remove-PnPSiteSensitivityLabel on the https://pnp.github.io/powershell/cmdlets/ reference, which is why I asked the question.
I have confirmed that the Remove-PnPSiteSensitivityLabel command works as expected in my test environment.
Thanks!


Needs tests

Something to figure out; how to test this.

Closing due to inactivity. Tests will be added whenever this project is rewritten, as per #15.


xml as the hitch in the new version?

I used to do so http://stackoverflow.com/questions/14432106/change-syntax-color-in-scintilla-net
but it no longer works. how to be? documentation not found ...

This question has been asked and answered multiple times. See #14 and #5 and others... A detailed explanation of how to configure syntax highlighting can be found in the documentation. https://github.com/jacobslusser/ScintillaNET/wiki/Automatic-Syntax-Highlighting

ok good!
ps scintilla1.Styles[ScintillaNET.Style.Asm.CpuInstruction].ForeColor not work(

not only (at the assembly a lot of things not working (please check all

You're likely not setting the keywords correctly. Please review the documentation.

ok thanks! its work!


interruptionEventStream doesn't emit a new item when TalkBack is on (Android)

It seems that interruptionEventStream doesn't emit a new interruption event when certain conditions are met:

App is running on Android (described behavior happens at least on Android 11 and 12)
TalkBack is on (Android 12: Settings > Accessibility > TalkBack > Use TalkBack)
TalkBack's audio ducking is off (Android 12: Settings > Accessibility > TalkBack > Settings > Sound and Verbosity > Audio ducking). Interruption stream works as expected when ducking is on.

Reproduction path:

Enable TalkBack and disable audio ducking (see above)
Open app
Request audio focus
Change TalkBack focus to a different widget (e.g. Text) so TalkBack can start speaking and current playback should be interrupted

Observed behavior:
When changing focus the app doesn't receive a new AudioInterruptionEvent. This makes it impossible to do necessary changes in the audio playback (pause playback etc). The app is unable to update playback state (I'm using this plugin along with audio_service). There's playback state mismatch: app shows that playback is ongoing although it's actually not.
Expected behavior:
App gets notified about new audio interruption via AudioSession.interruptionEventStream.
My audio session configuration:
final config = const AudioSessionConfiguration.speech()
    .copyWith(androidWillPauseWhenDucked: true);

Can you point to any other audio apps that correctly handle the interruption?

After further investigation I can say it's not a bug of this plugin 🙂
I've checked Spotify, Youtube and Google Podcasts apps with ducking off and all these apps continue playback while TalkBack is in use. I'm using text to speech plugin along with audio_service and audio_session. Turns out TalkBack with ducking off stops TTS engine's playback directly but doesn't do anything with media players and their audio focus\session. TTS plugin I'm using receives a stop\interruption event but audio service and audio session don't and this is not an issue of this plugin but rather how things work on Android with that setting off.

That doesn't surprise me, Android considers TTS and music as different kinds of audio streams which is also why I provide that hack in audio_service to force the media buttons to show by playing a little bit of "real" audio (or what Android considers real audio. TTS doesn't count, since it's on it's own special audio stream.) Anyway, that's not exactly the same issue here, TalkBack could have been programmed to request audio focus if they wanted it to, but I guess the fact that they didn't is consistent with their overall approach to treat TTS output differently from normal audio output.


Pick latest flytestdlib with metric name sanitization

Signed-off-by: Ketan Umare<EMAIL_ADDRESS>TL;DR
Metric name sanitization from flytestdlib.
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

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

Follow-up issue
NA

Looks like a unit test that relies on the metric metric names fails... mind updating that?


Update fabric go module to v2

Although Fabric is not intended to be used as a go module that other
projects import, some projects use Go tooling to download repositories
and this does not work for Fabric v2.x since Fabric's go module is
not defined as a v2 module. This change marks Fabric's go module
as v2 so that Fabric v2.x versions can be downloaded with Go tooling.
Fixes #2929
Signed-off-by: David Enyeart<EMAIL_ADDRESS>

@yacovm Yeah, the further I go the more disruptive it is looking. At first I tried to just update the module path in go.mod to include /v2, but then all the imports in all the go files resolved to old v1 fabric dependencies since there was no /v2 in the imports. Adding /v2 to the imports fixed that and it compiles now, but there are several more issues in script files that will need to be resolved. So I'm thinking of abandoning the idea for the reasons you mention. I've asked the issue opener for an example of why it is needed.
BTW the SmartBFT module is not tagged as /v2 at https://github.com/SmartBFT-Go/consensus/blob/v2/go.mod#L1, so I don't think consumers will be able to specify SmartBFT version v2.x. They would get error like
"version "v2.x.x" invalid: should be v0 or v1, not v2"
since the go index servers don't know about any /v2 module.
See https://go.dev/blog/v2-go-modules.

Yeah, the further I go the more disruptive it is looking. At first I tried to just update the module path in go.mod to include /v2, but then all the imports in all the go files resolved to old v1 fabric dependencies since there was no /v2 in the imports.

Have you tried using a replace directive in go.mod to force Go to look at the local folder for github.com/hyperledger/fabric ?

BTW the SmartBFT module is not tagged as /v2 at https://github.com/SmartBFT-Go/consensus/blob/v2/go.mod#L1, so I don't think consumers will be able to specify SmartBFT version v2.x. They would get error like
"version "v2.x.x" invalid: should be v0 or v1, not v2"
since the go index servers don't know about any /v2 module.

They can just specify the shortened commit hash which is published in every release instead of using the tag "name".

The change to v2 appears to be too disruptive, closing.


PurviewEntities.CreateOrUpdateEntitiesAsync does not update classifications for existing entity in purview

For an already existing entity in purview, If I call CreateOrUpdateEntitiesAsync() with new classifications in request body, then this method does not update classificiation tags in purview.

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

ID: 60dc0816-e226-c23b-b960-59280d96b354
Version Independent ID: 1233eebf-b6cf-eb9d-4e20-48e35be13ef5
Content: PurviewEntities.CreateOrUpdateAsync(RequestContent, RequestContext) Method (Azure.Analytics.Purview.Catalog) - Azure for .NET Developers
Content Source: xml/Azure.Analytics.Purview.Catalog/PurviewEntities.xml
Service: azure
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@jsquire Can you please help with this issue? Is there any update ?

@pratikrane-ms:  There's not much help that I can offer, my role for this is only triage.   The Purview team owns this library and I have no insight.


fix(Textfield): character counter inital value

Solves: https://github.com/digdir/designsystemet/issues/2042

fix the casing lint error in title for you :)


Audio NFT player

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #3248
[ ] Requires deployment <rubick/magick_issue>
[ ] <brief_description_of_what_I've_added>

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[ ] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

LGTM

@Jarsen136 one doubt though
https://deploy-preview-3268--koda-nuxt.netlify.app/rmrk/gallery/10249720-466e8b6338599eaf1e-GAVCUBE-GAV_CUBE_101-0000000000000101?page=1
is this also an audio nft?

@Jarsen136 one doubt though https://deploy-preview-3268--koda-nuxt.netlify.app/rmrk/gallery/10249720-466e8b6338599eaf1e-GAVCUBE-GAV_CUBE_101-0000000000000101?page=1 is this also an audio nft?

Hey, it's just a image 🚶

pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 57.73 USD/KSM ~ 0.866 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0x7c842769c2375b71e825b089613f1cf732bf60cea5c84dab09f07a22a82deceb
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Add custom unmarshaller to NullJSONText

Fixes #671
NullJSONText needs to be valid after successful json unmarshalling.

Pull Request Test Coverage Report for Build 162

4 of 6 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 73.916%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

types/types.go
4
6
66.67%

Totals

Change from base Build 161:
0.4%

Covered Lines:
1193

Relevant Lines:
1614

💛  - Coveralls

@jmoiron what do you think about this pr?

Could we merge this PR?


Add typescript definition

This is all right if we have built-in support typescripts at the moment-jalaali.
In this PR, I moved the code by: Ali Taheri Moghaddar https://github.com/alitaheri in https://github.com/DefinitelyTyped/DefinitelyTyped repo to here.

would be nice to have this one merged

@alitaheri If you agree, please merge it.


created radial bar charts (The Paper's team)

Thanks for this PR :)

fig.0

The test failed in Excel(Online version).
Please check the following two points.
Point 1) Please refer structure of data visualization template.
("main.css" and "manifest.yml" required)
See fig1 and https://github.com/e2d3/e2d3/wiki/Structure-of-data-visualization-template

fig.1

Point 2) Elimination of the cause of "d3.scale.radial is not function".
See fig.2.

fig.2

Thanks for this PR :)

fig.0

The test failed in Excel(Online version).
Please check the following two points.
Point 1) Please refer structure of data visualization template.
("main.css" and "manifest.yml" required)
See fig1 and https://github.com/e2d3/e2d3/wiki/Structure-of-data-visualization-template

fig.1

Point 2) Elimination of the cause of "d3.scale.radial is not function".
See fig.2.

fig.2

Confirmed that the error no longer appears in the state of pull request #322 , so merging is done. (The category is set to Uncategorized.)
(The error is not displayed, but the graph and sample data are not displayed.
Test Environment
OS: Ubuntu2004 Desktop
Excel:ExcelOnline
Browser:Firefox


Fix/bybit perp v5

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
ByBit V5 API Derivative connector implementation.
This PR was previously tracked by https://github.com/hummingbot/hummingbot/pull/6902 before splitting Spot and Derivative connectors into separate PRs.
Tests performed by the developer:
Tips for QA testing:

Regarding the balance retrieval issue reported, for both Classic and UTA accounts, I get the balances correctly.

If anyone faces issues fetching balances, please report along with the information below:

Account Type
Market

Closed in prio to new PR (https://github.com/hummingbot/hummingbot/pull/7196)


Can an INDI have two FAMC to the same FAM?

In PR #274 we added to next-minor multiple PEDI for the same FAMC, which is also referenced in #338. But I now thing we should roll that change back as we can simple use several FAMC instead.
Two-FAMC version, keeping PEDI as {0:1} cardinality:
0 @F1@ FAM
1 CHIL @I1@
0 @I1@ INDI
1 FAMC @F1@
2 PEDI FOSTER
3 DATE FROM 1901 TO 24 JUN 1904
1 FAMC @F1@
2 PEDI ADOPTED
3 DATE FROM 24 JUN 1904

Two-PEDI version:
0 @F1@ FAM
1 CHIL @I1@
0 @I1@ INDI
1 FAMC @F1@
2 PEDI FOSTER
3 DATE FROM 1901 TO 24 JUN 1904
2 PEDI ADOPTED
3 DATE FROM 24 JUN 1904

Although it seems odd to have two FAMC pointing to the same FAM, I don't see any prohibition of it in the current spec; thus I think that making PEDI {0:M} would create ambiguity with two ways of encoding the same data.

Discussed in steering committee
We like the 2-PEDI version because the 2-FAMC version is likely to be misunderstood by current applications
We should add an explicit recommendation against two FAMC pointing to the same FAM


tests: add rule type check for flowbits v6

Task #6309
Redmine ticket: https://redmine.openinfosecfoundation.org/issues/6309
Previous PR: https://github.com/OISF/suricata-verify/pull/1512
Suricata PR: https://github.com/OISF/suricata/pull/9971
Output from console:
===> flowbits: Sub test #11: FAIL : expected 1 matches; got 0 for filter {'filename': 'rules.json', 'count': 1, 'match': {'id': 11, 'lists.packet.matches[0].name': 'flowbits', 'lists.packet.matches[0].flowbits.action': 'noalert'}}

Output from stdout file:
Notice: suricata: This is Suricata version 7.0.3-dev (2fe2d8250 2023-10-19) running in USER mode [LogVersion:suricata.c:1149]
Warning: threshold-config: Error opening file: "/usr/local/etc/suricata//threshold.config": Permission denied [SCThresholdConfInitContext:util-threshold-config.c:177]
Warning: detect-flowbits: flowbit 'fb3' is checked but not set. Checked in 4 and 1 other sigs [DetectFlowbitsAnalyze:detect-flowbits.c:602]
Warning: detect-flowbits: flowbit 'fb4' is checked but not set. Checked in 5 and 0 other sigs [DetectFlowbitsAnalyze:detect-flowbits.c:602]
Warning: detect-flowbits: flowbit 'fb5' is checked but not set. Checked in 13 and 0 other sigs [DetectFlowbitsAnalyze:detect-flowbits.c:602]

New PR: https://github.com/OISF/suricata-verify/pull/1529


Error loading /Library/Audio/Plug-Ins/HAL/JackRouter.plugin/Contents/MacOS/JackRouter

I'm getting the following error when piping a stream to a decoder instance on OS X 10.10:
2015-02-16 10:17:30.823 node[15208:127520] Error loading /Library/Audio/Plug-Ins/HAL/JackRouter.plugin/Contents/MacOS/JackRouter:  dlopen(/Library/Audio/Plug-Ins/HAL/JackRouter.plugin/Contents/MacOS/JackRouter, 262): Library not loaded: /Library/Frameworks/Jackmp.framework/Versions/A/Jackmp
  Referenced from: /Library/Audio/Plug-Ins/HAL/JackRouter.plugin/Contents/MacOS/JackRouter
  Reason: image not found
2015-02-16 10:17:30.824 node[15208:127520] Cannot find function pointer New_JackRouterPlugIn for factory 7CB18864-927D-48B5-904C-CCFBCFBC7ADD in CFBundle/CFPlugIn 0x100b1e9a0 </Library/Audio/Plug-Ins/HAL/JackRouter.plugin> (bundle, not loaded)

Can you give me more complete sample code to repro?

me is usin audiokit in xcode

I was a problem in xcode, I unistalled jack, and it solved

Closing stale issues


[SIL.rst] Fix warning.

FYI @eeckstein I just fixed this, but I think the issue here was that sphinx didn't like the s at the end of the sil-basic-block``. I rephrased it so a non-plural word would suffice.

@swift-ci smoke test

thanks!


chore: Improve Renovate behavior

Description
Please review this change per commits. I provided commit message in each.
User-facing documentation

[x] CHANGELOG is updated OR update is not needed
[x] documentation PR is created and is linked above OR is not needed

Testing and quality

[x] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing
No automated testing.
How I validated my change
Ran renovate-config-validator and it said all's ok.

At least step image changes get grouped now https://github.com/stackrox/stackrox/pull/13493

Removed label: no need to backport. The config appears to be read from the main branch. E.g. https://github.com/stackrox/stackrox/pull/13494


npm 7.21.1  node16.9.1 Ubuntu 22.04.1 LTS can not run,  because The requested module 'wechaty' is a CommonJS module, which may not support all module.exports as named exports. Could every body tell me why?

import { WechatyBuilder, ScanStatus, log } from "wechaty";
^^^^^^^^^^^^^^
SyntaxError: Named export 'WechatyBuilder' not found. The requested module 'wechaty' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:
import pkg from 'wechaty';
const { WechatyBuilder, ScanStatus, log } = pkg;
at ModuleJob._instantiate (node:internal/modules/esm/module_job:124:21)
at async ModuleJob.run (node:internal/modules/esm/module_job:179:5)
at async Loader.import (node:internal/modules/esm/loader:178:24)
at async Object.loadESM (node:internal/process/esm_loader:68:5)
at async handleMainPromise (node:internal/modules/run_main:63:12)

Try using the node LTS version

Try using the node LTS version

我和他是一样的错，更换了nodejs最新LTS版本也不行

我应该是这样解决了，你试试，sudo npm install qrcode-terminal --save sudo npm install wechaty  sudo npm install wechaty-puppet-wechat --save
------------------ Original message ------------------
From: "ronaldox";
Sendtime: Tuesday, Dec 13, 2022 2:28 AM
To: "wangrongding/wechat-bot";
Cc: " @.***>; "Author";
Subject: Re: [wangrongding/wechat-bot] npm 7.21.1  node16.9.1 Ubuntu 22.04.1 LTS can not run,  because The requested module 'wechaty' is a CommonJS module, which may not support all module.exports as named exports. Could every body tell me why? (Issue #27)
Try using the node LTS version
我和他是一样的错，更换了nodejs最新LTS版本也不行
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

或者先参考这个方案。
https://github.com/wangrongding/wechat-bot/issues/54#issuecomment-1347743629


Want to add a weather project using API

Want to add a weather project using API written in python

@dellucifer Go ahead.

@dellucifer
Thanks!
Feel free to add another project.

Sure @theadeyemiolayinka


SyntaxError: unsupported binaryType: must be either "nodebuffer" or "arraybuffer"

I'm trying to integrate the server HMR into my existing project.
The first time the page is loaded correctly, I can navigate in my app without problems, but if I call a route hitting enter on the address bar, sometimes I get this error: SyntaxError: unsupported binaryType: must be either "nodebuffer" or "arraybuffer" (similar to this: https://github.com/websockets/ws/issues/725)
Any suggestions?
thanks

@foxhound87 Can you provide an example repo, or update this example to show it?  I've never seen this error before :D

@ericclemmons
You can see the code of my project (on the server-hmr branch):
https://github.com/foxhound87/rfx-stack/tree/server-hmr
I fixed the error as shown in the ws issue adding these lines:
https://github.com/foxhound87/rfx-stack/blob/59d44bd6f3eff823563fe7b5a3c9180b34f3da3d/webpack/config.server.dev.js#L55-L57
but though I don't get errors (and on the terminal it seems going all ok) the server hmr is not working at all, changes are not applied.
I'm using webpack-dev-middleware and  webpack-hot-middleware to provide client-side hmr in conjunction with the server-side hmr, would be appreciated if you could take a look at the configurations.
Thanks

You're right, that regex doesn't catch socket.io because it has a . in it.  (It's a pretty naive regex just for this example).
Personally, I ended up using this monstrosity after having problems with node-externals:
  .externals(
    function(context, request, callback) {
      if (
        /node_modules/.test(context)
        || (
          /^@?\w[a-z\-0-9\./]+$/.test(request)
          &&
          (
            nodeModuleExists(request)
            ||
            nodeModuleExists(`${request}.js`)
          )
        )
      ) {
        return callback(null, `commonjs ${request}`);
      }

      callback();
    }
  )

@ericclemmons Thanks for the answer.
I'm trying webpack-node-externals like this:
import nodeExternalModules from 'webpack-node-externals';
...
    externals: [nodeExternalModules({
      whitelist: ['webpack/hot/poll?1000'],
    })],
...

I'm having trouble using ['es2015', { modules: false }] in .babelrc.
As I'm using es6 syntax into the webpack config too, I added that into the query of  the loader only for the dev-server config, but it seems not to work yet.
If I set that globally into the .babelrc the es6 imports stop working (with SyntaxError: Unexpected token ).


support for flags in cidr.to_str function

Hi
I noticed that there is no way to set 'flags' for cidr.to_str function.
the flags is available at libcidr.h as constants (CIDR_NOCOMPACT, CIDR_USEV6, etc)
is there a plan to add it  ?

Now part of the v1.0.0 release thanks to #2.


Add support for flag value auto-completion

This looks like a great library. One thing I noticed missing was support for flag auto-completion with cobra.
This SO answer shows a good example of how that can be implemented: https://stackoverflow.com/a/70541824/337735
(Incidentally I found your library on this SO answer too. And here you are! https://stackoverflow.com/questions/50824554/permitted-flag-values-for-cobra#comment133213356_73297333 :))

Hi codyaray, thank you very much for your feature request and already including useful references!
After checking the (somewhat minimalist) documentation and then consulting cobra's code I'm wondering how to best make this new feature convenient to use to module users.
My idea is as follows:
ef := enumflag.New(..., FooModeIds, ...)
cmd.PersistentFlags().VarP(..., ef, ...)
ef.Completion(cmd, FooModeCompletions)

where
FooModeCompletions := map[FooMode]string{...}

Or would you instead prefer a new, additional "constructor" along the lines of
ef := enumflag.NewWithCompletions(..., FooModeIds, FooModeCompls, ...)
What do you think?


how can I know updated fields or dirty fields in after save or after created callbacks?

@sdispater hey I want to know updated fields in callbacks how can I do that in Orator's model observer or from callbacks?

You can search throught Orator ORM source-code.
I have found model.is_dirty(attribute_name) method. You can try this method on your model-observer class.


Gradle assemble for some reason runs static analysis

See for example https://github.com/JetBrains/jewel/actions/runs/6040679439/job/16392007638?pr=102

Also to note, the assemble task executes the detektMain task, which should instead be executed by the check task (but isn't)

@lamba92 can you look into it?


feat: configured spell checker github actions

This Pr introduces spell checker github action to the workflow
fixes #2291

@mowies PTAL

@mowies PTAL
I have configured this according to the docs.Let me know if I haver missed something

okay!!

fyi: i created https://github.com/keptn/lifecycle-toolkit/pull/2443 to prepare the config and to fix any errors that came up already

Thanks a lot @mowies !!
Happy to contribute to this community


Faster buffers

Make flatbuffers a little faster on substrings
for nitc/nitc/nitc
before: 0m7.168s
after: 0m7.068s (-1.4%)

+1 !

reroll: cf comment of @xymus ; also added a commit to do the same to FlatString::with_info

+1

reroll: improve doc

+1


Adding FreeBSD compatibility

This pull requests removes a missing source file from the Makefile and adds FreeBSD compatibility. On FreeBSD, make compile-linux works as well.

sqlite3-json1.c is not missing. It is downloaded during the build process via the download-external make task.

I see. I’ve just used the SQLite 3 port which probably does not include the dependency. In this case, please ignore the Makefile commit.

I can't edit your changes. You'll have to revert the 62f32cd commit.

Thanks!


Bump Flex to 1.17

Following https://symfony.com/blog/upgrade-flex-on-your-symfony-projects
The blog article tells the world not to use Flex versions older than 1.17. I think it would be wise to update the skeleton to reflect this request.

We won't retag the skeleton for older versions, but I'm fine doing this on 5.4
In practice this shouldn't matter since composer will always select the latest.

Okay, let's merge to 5.4 then.

In practice this shouldn't matter since composer will always select the latest.

That is correct. However, this constraint will be persisted in the composer.json file of every new Symfony application that will be created from the skeleton. If we discourage the use of older versions, I believe that letting new applications start with that constraint a good way to document that.

I'm fine with this change. At the very least, it sends a "subtle" message in the right direction :)

Thank you @derrabus.


Make protocol case insensitive

Passing protocol='HTTP' should result in same behavior as protocol='http'.  Right now 'HTTP' ends up using https.

Fixed by https://github.com/Azure/azure-sdk-for-python/pull/380


Upgrade API client to v13.1.0

Trello: https://trello.com/c/whH4DvW1/125-high-severity-vulnerability-in-python-requests-package
Upgrade the API client to fix a vulnerability in the Python requests package.
This means a couple of major version bumps, hopefully all OK:

v13 update_user (not used in this app)
v12 Search API index creation (not used in this app)
v11 get_direct_award_project_services and find_direct_award_project_searches - used in this app but with keyword arguments, so should not be affected by the breaking change

Please double check if there's anything I've missed.

Cool just let me run the tests locally...


Fixups required for generation changes

Will rerun API generation with these in place, then run a build.
cc @evildour as there are BigtableClient generator changes

Okay, it's looking good - merging.


stack overflow on fragment cycle

Description
During validation aren't all fragment cycles detected.
This causes a stack overflow in latest beta<EMAIL_ADDRESS>Steps to reproduce
Validate the following query for your schema:
query Introspection{
  __schema {
    types {
      ...cycle
    }
  }
}

fragment cycle on __Type {
  kind
  ...frag
}

fragment frag on __Type {
  kind
  ofType {
    ...cycle
  }
}

Here a code sample.
Expected result
The validation should pass and in the diagnostics should be one like:
{
  "errors": [
    {
      "message": "compiler error: `cycle` fragment cannot reference itself",
      "locations": [
        {
          "line": 9,
          "column": 1
        }
      ]
    }
  ]
}

Actual result
In<EMAIL_ADDRESS>thread 'main' has overflowed its stack
fatal runtime error: stack overflow
[1]    2496 abort      ./target/debug/gus start -m testing/server

So the program crashes.
In<EMAIL_ADDRESS>An internal error occurs because salsa detected this cycle, so still an error but no crashing stack overflow.
So in terminal it is:
panicked at .../salsa-0.16.1/src/lib.rs:490:48:
Internal error, cycle detected:

DatabaseKeyIndex { group_index: 3, query_index: 43, key_index: 3 }
DatabaseKeyIndex { group_index: 3, query_index: 44, key_index: 3 }
DatabaseKeyIndex { group_index: 3, query_index: 38, key_index: 1 }
DatabaseKeyIndex { group_index: 3, query_index: 43, key_index: 5 }
DatabaseKeyIndex { group_index: 3, query_index: 44, key_index: 5 }
DatabaseKeyIndex { group_index: 3, query_index: 31, key_index: 4 }
DatabaseKeyIndex { group_index: 3, query_index: 43, key_index: 6 }
DatabaseKeyIndex { group_index: 3, query_index: 44, key_index: 6 }
DatabaseKeyIndex { group_index: 3, query_index: 38, key_index: 2 }

note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Here the code sample for this version.
Environment

Operating system and version: Ubuntu 20.04.6 LTS
Shell (bash/zsh/powershell): zsh
apollo-rs crate: apollo-compiler
Crate version: 1.0.0-beta.5

Thanks for the report, should definitely fix this. It's explicitly outlawed in https://spec.graphql.org/draft/#example-cd11c.


Logout event will return null result, when Webhook

when user logout, I can't got real user info, this webhook result:
{
	"id": 0,
	"owner": null,
	"name": "a69c8eaf-eb06-432d-b856-edc522b92369",
	"createdTime": "2023-10-13T06:51:08Z",
	"organization": null,
	"clientIp": "<IP_ADDRESS>",
	"user": null,
	"method": "POST",
	"requestUri": "\/api\/logout",
	"action": "logout",
	"object": null,
	"isTriggered": false,
	"extendedUser": null
}

@Sun-FreePort

Add logout to events
Enable Is user extened
Make sure extendedUser has no-empty value in Preview

@hsluoyz Hi.
Yep, I still keep enabled your setting, but result still keep empty. so I print more information in there:

Casdoor version: 版本: v1.432.0
My system: Windows 10 64x
Docker from offical all in one
Use organizations: build-in
Use user (logout it): admin

If you need any information, tell me please.

@hsluoyz Hi.
Yep, I still keep enabled your setting, but result still keep empty. so I print more information in there:

Casdoor version: 版本: v1.432.0
My system: Windows 10 64x
Docker from offical all in one
Use organizations: build-in
Use user (logout it): admin

If you need any information, tell me please.

Other build-in user to logout would tell me empty result.
this is my screen on Casdoor backend:

@Sun-FreePort fixed by: https://github.com/casdoor/casdoor/commit/69b144d80f927dd886bb6dc72b21980b1d67ba8e


Add inline padding to hotkey labels

I'd love to tackle this one for you!

@kthurman59 sorry I didn't see your comment until I just finished and was coming here to close the issue lol.  Feel free to work on any others though!  ( I'll try to be better about reading comments before I start working on issues from now on )

No worries at all!  I will check those out!


Correctly highlight negative floating point values in C++

Negative floating point numbers in C++ did not highlight correctly.
From:
float negative = -123.0f; // negative floating point numbers

To:
float negative = -123.0f; // negative floating point numbers

Looks reasonable, thanks! It's a shame really how bugs such as this crop up after 10+ years in the wild…


Add a WolframAlpha function

http://products.wolframalpha.com/api/documentation.html

This issue was moved to DesertBot/DesertBot#2


[transformation] Dead Code Removal

@artbobrov Should we look at this pr? If so, please fix the tests and resolve the conflicts before doing this.

Sorry for my question, but should we sore .idea files on Github? Maybe should you delete them?


new tlds.csv based on latest info from IANA

A new 'tlds.csv' was generated with the /bin/helpers/generate-tlds-csv.js script.
This script downloads and parses the latest list of TLDs from IANA; it further processes the list by matching each TLD to a country.
The new 'tlds.csv' contains a 3rd column denoting the TLD "type" as provided by IANA (i.e. "generic", "country-code", "sponsored").

Just checking, been a while. The TLD list has been refreshed again: https://github.com/katmore/tld-enum/releases/tag/v1.0.7


Possibility to have full height PDF

Can we output a PDF that is the full height of the webpage? In other words, don't break up a screenshot into multiple pages. I have a use case where I would like to see the entire webpage on a single PDF page.
P.S. Thank you for this awesome project!

You can set a custom PDF height - would it work for you?
page.pdf({path: 'my.pdf', height: '10000px'});

Saw that but pages vary in height and in this application, many pages will be screenshot at the same time. It will be less than optimal if all pages in the pdf are the same height.

You can query the document height first and then use it as an argument to page.pdf
let height = await page.evaluate(() => document.documentElement.offsetHeight);
await page.pdf({path: 'my.pdf', height: height + 'px'});

Ah yes, I'll try that. I tried with Headless Chrome previously and it was slow since it seemed to pull the page twice. If this can be done without pulling the page twice, this will be awesome.

Nice, I just tried it. It looks like it works and it seems like only one roundtrip for the page. Thank you!

Hey,
I am trying the same approach to get a full page PDF, The height of the page increases but the height of the content stays the same.
My code is like this
const puppeteer = require("puppeteer");

function sleep(ms) {
	return new Promise(resolve => {
		setTimeout(resolve, ms);
	});
}

(async () => {
	const browser = await puppeteer.launch({
		headless: true
	});

	const page = await browser.newPage();

	await page.goto("http://localhost:3000/longpage", {
		waitUntil: "networkidle2"
	});

	let height = await page.evaluate(
		() => document.documentElement.offsetHeight
	);

	console.log("Height", height);

	await page.pdf({
		path: "hni.pdf",
		printBackground: true,
		margin: "none",
		height: height + "px"
	});

	await browser.close();
})();

Thanks,
- Nithin

I managed to print full page pdf with the following code.
const height = await page.evaluate(() => document.documentElement.offsetHeight);
await page.pdf({ path: 'my.pdf', height: `${height + 1} px`, printBackground: true });

I added 1 px in page height and it removed the blank page. I know it may not an elegant solution but seems to work for me.
Adding 1px remove the blank page issue referred in issue #589

How can I evaluate the height if a custom width is used for PdfOptions ?

don't use format in page.pdf(), it will override the height option

I am using headless mode and I tried the same way to get a full page pdf, which I assume will be one page. Instead I got several pages, and the document height is always 600 for different urls and pages.
I get the document height by this const height = await page.evaluate(() => document.documentElement.offsetHeight);

@nithinkashyapn
How did you solve the issue ? I am getting the same issue. Able to get the height of the PDF but the content is missing.

I am using headless mode and I tried the same way to get a full page pdf, which I assume will be one page. Instead I got several pages, and the document height is always 600 for different urls and pages.
I get the document height by this const height = await page.evaluate(() => document.documentElement.offsetHeight);

Experiencing the same

@zhongnansu @NelsonFrancisco
I just ran into the same issue and found that using document.documentElement.scrollHeight instead of document.documentElement.offsetHeight did the trick for my setup.
let height = await page.evaluate(() => document.documentElement.scrollHeight);
await page.pdf({path: 'my.pdf', height: height + 'px'});

You can query the document height first and then use it as an argument to page.pdf
let height = await page.evaluate(() => document.documentElement.offsetHeight);
await page.pdf({path: 'my.pdf', height: height + 'px'});

I am getting the offsetHeight as '0' to my page.
How do I solve it?

There is a solution to this?
I'm having the same problem

There is a solution to this?
I'm having the same problem

It is unbelievable that Chrome has no perfect solution to such a SIMPLE problem.
I tried Safari's export to PDF and it directly generated a SINGLE PAGE.

@Lesnock  Had the same problem. I found a solution to get the content height in this thread: https://stackoverflow.com/questions/1145850/how-to-get-height-of-entire-document-with-javascript.
// height of content
const pageHeight = await page.evaluate(() = >{
  let height = 0;
  function findHighestNode(nodesList: any) {
    // eslint-disable-next-line no-plusplus
    for (let i = nodesList.length - 1; i >= 0; i--) {
      if (nodesList[i].scrollHeight && nodesList[i].clientHeight) {
        const elHeight = Math.max(nodesList[i].scrollHeight, nodesList[i].clientHeight);
        height = Math.max(elHeight, height);
      }
      if (nodesList[i].childNodes.length) findHighestNode(nodesList[i].childNodes);
    }
  }
  findHighestNode(document.documentElement.childNodes);
  return height;
});
// width of device
const pageWidth = await page.evaluate(() = >document.documentElement.offsetWidth);
// pdf
const pdf = await page.pdf({
  height: pageHeight + 'px',
  width: pageWidth + 'px',
});

For anyone looking for a solution, none of the options above worked for me, this is what in the end worked for my needs:
    const page = await browser.newPage();
    await page.setContent(html);

    const height = await page.evaluate(() => document.documentElement.offsetHeight);
    const width = await page.evaluate(() => document.documentElement.offsetWidth);

    await page.addStyleTag({
      content: `@page { size:${width}px ${height}px; }`,
    });

    await page.emulateMediaType('screen');

    /**
     * @see {@link https://pptr.dev/api/puppeteer.pdfoptions}
     */

    const pdfOptions: PDFOptions = {
      pageRanges: '1', // <-- limit range to 1
      preferCSSPageSize: true, // <-- make sure @page is taken into account
      printBackground: true,
    };

    const pdf = await page.pdf(pdfOptions);

Credit to this SO comment from Nithin

thanks for all of you.
for me, my html content has a lot of sections and every section width is 595px and height is 842px .. and with this code :
await page.pdf({     height:842px,     width: "595px",     printBackground: true,     landscape: false,   });
it works well, and every section in one page of pdf and toke the same width and height


[Task]: How to inference fastflow model by using pt file? (anomalib 0.3.3)

What is the motivation for this task?
Hello, I use fastflow model (anomalib 0.3.3) for work. (can't update to lastest version)
Until now, For fast inference speed,

After traning, I made lightning model ('.ckpt')  to torch model ( '.pt'  ) like below
logger.info("Training the model.")
trainer.fit(model=model, datamodule=datamodule)
logger.info("Loading the best model weights.")
load_model_callback = LoadModelCallback(weights_path=trainer.checkpoint_callback.best_model_path)
trainer.callbacks.insert(0, load_model_callback)
best_model =model.load_from_checkpoint(trainer.checkpoint_callback.best_model_path)
torch.save(best_model.state_dict(), 'best_model.pt')

And I tried to inference, an error occured (in anomalib/tools/inference/torch_inference.py)
weights ='best_model.pt'
inferencer = TorchInferencer(path=weights, device= 'cuda')
KeyError("model is not found in the checkpoint. Please check the checkpoint file.")

I saw a solution about config file with export mode but there is no paramter in my config file(anomalib 0.3.3)

Describe the solution you'd like

How I can convert lightning '.ckpt' to torch '.pt' file and infence by torch '.pt' file?

If there is better way to improve inference speed, please tell me

In Advance, thank you for your help!!
Additional context
No response

you just need to change the "export_mode" in config.yaml about fastflow model.

And I also want to know how to improve the inference speed.

how to speedup？？

You could use OpenVINO inference to speed it up on CPU. As @Sj-Yuan mentioned, you need to change the export_mode to openvino


add cmake support

This might fail on OS X, which has cuda in lib instead of lib64.
On Thu, Dec 3, 2015 at 3:51 PM, Fisher Yu<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/PrincetonVision/marvin/pull/24
-- Commit Summary --

add cmake support
-- File Changes --
A CMakeLists.txt (18)
-- Patch Links --
https://github.com/PrincetonVision/marvin/pull/24.patch
https://github.com/PrincetonVision/marvin/pull/24.diff

Reply to this email directly or view it on GitHub:
https://github.com/PrincetonVision/marvin/pull/24

@danielsuo Can you test it on Mac? I think it will work. I do want to clean up the file in a following commit.

Resolved both the lib vs lib64 issue and the pthread problem. It turns out
that TRY_COMPILE on OS X doesn't want to work for pthread.h because we need
to enable both C and CXX in the project command.
On Thu, Dec 3, 2015 at 5:01 PM, Fisher Yu<EMAIL_ADDRESS>wrote:

@danielsuo https://github.com/danielsuo Can you test it on Mac? I think
it will work. I do want to clean up the file in a following commit.
—
Reply to this email directly or view it on GitHub
https://github.com/PrincetonVision/marvin/pull/24#issuecomment-161798718
.

Both 2cf9229 and c7b834c cause the error pasted below on Ubuntu 15.04 for me. Fixed with 6f9f4f0.
/usr/bin/ld: CMakeFiles/marvin.dir/marvin_generated_marvin.cu.o: undefined reference to symbol 'pthread_create@@GLIBC_2.2.5'
/lib/x86_64-linux-gnu/libpthread.so.0: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
CMakeFiles/marvin.dir/build.make:375: recipe for target 'marvin' failed
make[2]: *** [marvin] Error 1
CMakeFiles/Makefile2:60: recipe for target 'CMakeFiles/marvin.dir/all' failed
make[1]: *** [CMakeFiles/marvin.dir/all] Error 2
Makefile:75: recipe for target 'all' failed
make: *** [all] Error 2

Resolved by commit @947c456d80e4710d130b13ac4314029f3cf48259


Change FreeGasChangeHeight to 6195000.

Change FreeGasChangeHeight to 6195000 for mainnet test before FLM launch.

@shargon Could you help to merge this?


fix(pd-store): fix hstore backend api tests failure [WIP]

skipping core tests for hstore temporarily to test API tests, feel free to ignore...

close by #2463


Zwifit Frontend and Zwifit-incline-tracker incline control not working

Describe the bug
Zwifit is unable to control treadmill incline until a random time (20+ minutes) after the treadmill is started.  I'm trying to work out how to switch my (Proform Pro 1000) into mode 2 (control mode) where the treadmill can be controlled by Zwifit.
When I say mode 2, I've seen it enter this mode via Zwifit debug code I've added when I was lucky enough to enter this mode. This allows the treadmill to be controlled, so I'm calling it control mode (a bad name for it I know).
This functionality is required so that the Zwifit Frontend and Zwifit-incliner can control the incline. The treadmill eventually switches to this mode, making a series of tones.  Does Zwifit instruct the treadmill to enter mode 2 (control mode), or does the treadmill tell Zwifit that it can now be controlled?
 Key not plugged it. Display off. ifit light on.
Mode 1: Treadmill key inserted after being in state 8
Mode 10: Treadmill started
Mode 2: Expect control mode to be entered here.
Mode 3: Treadmill stopped
Mode 8: Key removed
Mode 2 (control mode) when the treadmill can be controlled by Zwifit, seems to occur randomly after 20 minutes or so. So randomly that I'm having problems going into this mode as I type this up. Until this mode is in operation, changes in speed, Incline, etc made on the treadmill are seen by Zwifit.
Expected behavior
When Zwifit makes a connection with the treadmill, entering mode 10, it should then switch to allow control of the treadmill so that both monitoring and control are possible.
Screenshots
N/A
Desktop (please complete the following information):
Raspbian, Zwifit, Treadmill
Additional context
None

To clarify, during testing, the Raspberry Pi 3 is sat on the Proform Pro 1000 dashboard, directly under the treadmill display.
With Zwifit already running, as soon as the Treadmill is powered up, it connects that instant.
Also, I have entered this control mode at least 5 times and have been able to control the incline etc.

I've run two test and have some promising results.
Mode 2 has the ability to control the treadmill.
Entering mode 2 doesn't appears to be random as first thought. It takes precisely 5 minutes after pressing the start button (resulting in the belt moving). So 5 minutes after the belt starts moving.
Is this something hardcoded into Zwifit, or is this the treadmill getting the user to do a 5 minute warmup?
Test 1:

20:30:2 - Mode:  8 - Treadmill turned on - ifit light on
20:30:18 - Mode:  1 - Treadmill key inserted after being in state 8
21:6:15 - Mode:  10 - Treadmill started - Belt moving

5 minutes exactly

21:11:15 - Mode:  2 - Control mode - Belt moving
21:11:51 - Mode:  3 - Treadmill stopped - Belt stopped - Results shown
21:12:3 - Mode:  4 - Stop button pressed again - done shown on display

2 minutes exactly

21:14:3 - Mode:  1 - switched back to this automatically.

Test

2:21:20:48 - Mode:  8
21:21:23 - Mode:  1
21:34:27 - Mode:  10

5 minutes exactly

21:39:28 - Mode:  2

I hope this helps.


Hcaptcha using your library without proper license attribution

This repo is licensed under the MIT License, which requires the following:

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

It seems like hcaptcha violation of this requirement. They appear to be using the code, with some modifications, without including the required license attribution. As shown in the attached image, a significant portion of the original code remains intact despite these modifications.
Source: https://gist.github.com/youdie323323/0163e5cdea861ba8a09b5d6ae00f5091
Begin from line 2192. thanks

Hi @youdie323323 , thanks for letting me know :)


Register windows users from backend

Mohammad,
How do I register windows users from backend, since my scenario admin will register/create users from his machine (which he logged-in using his credential) using user console modules.
Advance thanks

You can use UserManager to create local users account. Then, for each account add an external login matching the windows user account:

loginProvider
providerKey

Windows
Windows User SID

Thanks Mohammad


Java: Bug:

Describe the bug
An error occurred while adding other AI service like Qwen API, according https://learn.microsoft.com/en-us/semantic-kernel/concepts/ai-services/chat-completion/?tabs=csharp-AzureOpenAI%2Cpython-other%2Cjava-AzureOpenAI&pivots=programming-language-python
TypeError: OpenAIChatCompletion.init() got an unexpected keyword argument 'base_url'
To Reproduce
`from semantic_kernel import Kernel
kernel = Kernel()
kernel.add_service(OpenAIChatCompletion(
ai_model_id="my-deployment",
api_key="my-api-key",
base_url="https://my-custom-deployment.net", # Used to point to your service
))`
Platform

OS: Linux
IDE: Visual Studio
Language:  Python 3.12.4
Source: semantic-kernel 1.8.2

@Cheung-Z Thanks for reporting this issue. The Java version has been moved to it's own repository, can you create the issue here: https://github.com/microsoft/semantic-kernel-java/issues

In fact, i don't know why the github-actions bot changed the title to Java, this is a python problem actually.
I doubt the version of semantic-kernel caused this bug.
Looking forward to your reply.


Amend Common Permalink Issues

See https://github.com/gohugoio/hugo/issues/5488#issuecomment-449552225
Hugo now supports unquoted URLs in canonifyURLs replacer
EDIT
@digitalcraftsman We need to wait until the next release of Hugo before this PR is merged. See the following comment: https://github.com/gohugoio/hugoDocs/pull/690#discussion_r243731410
I'll post a reminder when it's due.

@digitalcraftsman
With the release of Hugo 0.53 this PR can be merged.

I didn't expected a release today. Anyway, merry christmas :santa:

Merry Christmas! All the best!


Add chamfer and rounding to the top of text3d() result

Is your feature request related to a problem? Please describe.
I wish I could create text that has a rounded or chamfered top.
Describe the solution you'd like
I wish I could pass a rounding arg to text3d, like most other shapes3d() things.
Describe alternatives you've considered
I'm open to suggestions. Haven't found a way at all though.
Example Code
// how it is now
text3d("Hello World", anchor=BOTTOM, h=20, size=8);

// how I wish it worked
text3d("Hello World", anchor=BOTTOM, h=20, size=8, rounding=2, except=[BOTTOM]);

I've battled this problem as well.  It would be a great help when making cookie cutters with text. (Something friends and familly ask for often).

Sadly, we have no access to the font geometry data to be able to do this, and an offset-based variant of linear_extrude() doesn't yet exist in OpenSCAD.
HOWEVER, there is a convoluted way to do what you want:

Create some text in Inkscape with the font you want.
Convert the text to paths.
Save an SVG of it.
Open the SVG file in a text editor.
Take the  strings from that file put it in a .scad file.
In OpenSCAD, use the Pathbuilder library to convert those strings to pointlist paths.
Put the paths together in a list to make a Region.
Use BOSL2's offset_sweep() from rounding.scad to get your rounded result.

Actually, I am unsure if offset_sweep() can handle Regions, so it is possible you will have to treat the region subpaths individually and do some creative difference.

offset_sweep requires a path.  I suppose in principle the module version only could be extended to do regions.   Not sure if there are hidden complications there.  (There might be, associated with flipping the direction of the end treatments for subtracted components.)
Just in case this is unclear, the original request of this issue is impossible.  OpenSCAD doesn't give us access to the geometry of the text.

How can we request that this be updated in the OpenSCAD system then? Seems like a massive oversight.

The inability to extract geometry from OpenSCAD is a huge limitation in many ways, not just for text. There is development under way to address this, but who knows if or when it will appear in the main release.

Are we keeping this open for some reason?


FLTK does not scale on HiDPI systems

Right, that's a problem. FLTK seems to be pixel based and doesn't have any HiDPI supporting API. But as it doesn't really affect the usability, I'll flag it as low priority for now. The next sprint isn't about AppImageUpdate anyway, and we have 14 other issues that need some time.

@probonopd would you try the Qt UI, please? I could imagine it works as intended. If that's the case, we could consider ceasing development of the FLTK UI in favor of the Qt version, once we consider it stable.

Agree, things like HiDPI should be easier to support using Qt than with FLTK. Hence closing here. (Not on my HiDPI machine at the moment, will reopen if the issue persists there.)

Reopening to keep track of the whole story. We can close this once the FLTK UI has been depracated.

Closed by #81.


Docker container

Hey @tclavier ,
Thanks for this. I'm not familiar with Docker yet. Can you move the docker section of the readme to after the Additional Resources heading?

Yes I can, but imho, Docker it's to run it and it can be near Install section :-D

I'm not sure most younger/beginner programmers would be familiar with Docker.

It's done, and TravisCI says "All is well" :-)


fix(module:tabs): mouse drag (scroll) on top of tabs title

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: 7263
What is the new behavior?
If tabs are not scrollable, the browser will take a native scroll on mobile
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

Please check above, maybe we can make it work well after dragend event as except :) After that, we can merge this PR.

Please check above, maybe we can make it work well after dragend event as except :) After that, we can merge this PR.

This is because of weird :hover behavior on drag events. I tried to escape that by implementing a custom hover directive that is with mouseover and mouseout events. At this event, I'm adding a custom ant-tab-hover class that has CSS for the hover effect. This class is with a fixed name because if the user changes the tab prefix.


fix(ui): use custom masonry grid component

Webpack bundle size diff
Total diff: -9.3 KB

M
build/static/js/12.40042c61.chunk.js
37.3 KB
36.7 KB
-682 B

M
[no source]
1.4 KB
1.4 KB
-6 B

M
Components/Grid/AlertGrid/AlertGroup/GroupHeader/GroupMenu.js
2.3 KB
2.3 KB
-2 B

M
Components/Grid/AlertGrid/GridSize.js
407 B
403 B
-4 B

M
Components/Grid/AlertGrid/AlertGroup/Alert/AlertMenu.js
2.3 KB
2.3 KB
-2 B

M
Components/Grid/AlertGrid/Grid.js
4.3 KB
2 KB
-2.3 KB

+
Hooks/useGrid.js

398 B
+398 B

+
Components/Grid/AlertGrid/Swimlane.js

1.3 KB
+1.3 KB

M
build/static/js/10.ec9162dc.chunk.js
25.1 KB
16.5 KB
-8.6 KB

M
bricks.js
2.6 KB
2.6 KB
-13 B

M
[no source]
498 B
412 B
-86 B

-
react-masonry-infinite
2.9 KB

-2.9 KB

-
react-infinite-scroller
5.6 KB

-5.6 KB


Firefox Mobile website has funny button placement

The two buttons at the bottom of the page are only a couple pixels off the bottom.
I tested on Chrome's developer tools, chrome, edge, Chrome mobile, and Firefox mobile and it turns out it's just Firefox mobile that has the problem. I think what's happening is that Firefox renders the page, pushes it down to render the browser top bar, then figures there's nothing below and doesn't have a scroll bar. Chrome does something similar, but lets you scroll down if you want.
Completely functional, so it depends on how OCD you are.

I agree, that totally looks bad. It looks like it can be solved by applying display: flex to things like main. The problem right now is that the margin of child elements is allowed to escape the main container and it's messing with the flex math of the body.


Cannot create CR with custom name by OperandRequest

/kind bug
What steps did you take and what happened:
I was looking for a way to have OperandRequest to create the Operand CR with a custom name rather than using the default CR name defined in alm-examples in CSV. By looking at the code, it looks the way to achieve this is using instanceName when define OperandRequest, e.g:
apiVersion: operator.ibm.com/v1alpha1
kind: OperandRequest
metadata:
  name: foo
spec:
  requests:
  - registry: bar
    registryNamespace: default
    operands:
    - name: elastic-cloud-eck
      kind: Elasticsearch
      apiVersion: elasticsearch.k8s.elastic.co/v1
      instanceName: baz
      spec:
        version: 7.13.3
        nodeSets:
        - name: default
          count: 1
          config:
            node.store.allow_mmap: false

However, after I apply the above change, I see ODLM leaves the Operand spec empty:
apiVersion: operator.ibm.com/v1alpha1
kind: OperandRequest
metadata:
  name: foo
spec:
  requests:
  - registry: bar
    registryNamespace: default
    operands:
    - name: elastic-cloud-eck
      kind: Elasticsearch
      apiVersion: elasticsearch.k8s.elastic.co/v1
      instanceName: baz
      spec: {}

This causes the CR failed to recreate, see logs in ODLM:
E0717 10:37:56.184088       1 operandrequest_controller.go:145] failed to reconcile Operands for OperandRequest default/foo: the following errors occurred:
  - the following errors occurred:
  - failed to create custom resource: Elasticsearch.elasticsearch.k8s.elastic.co "baz" is invalid: [spec.nodeSets: Required value, spec.version: Required value]

What did you expect to happen:
ODLM should allow me to create OperandRequest w/ non-empty Operand spec.
Anything else you would like to add:
Environment:

ODLM version: latest
Minikube/KIND/OCP version: kind v0.12.0-alpha+718ede73045ce3
Kubernetes version: (use kubectl version): v1.21.2
OS (e.g. from /etc/os-release): RHEL

Hi @morningspace
It is because we did not preserve the unknown filed for CR's spec section. PR is created #702

Thanks, I see it's been opened for a while, wonder when will this be merged :-)

It will be merged on next Monday, but it will take a while to include this fix in public build.


App size become huge

Hi..
After integration of twilio conversation sdk app size becomes 627 mb before integration its about 70-85 mb.
implementation "com.twilio:conversations-android-with-symbols:3.0.0"

Hi!

If you don't need native symbols in order to upload to crashlytics or similar service - just use the sdk version without symbols:

com.twilio:conversations-android:3.0.0

If you need symbols - you should setup android NDK in your app correctly in order to make gradle be able to strip symbols.


Who can I pay to make this work in Big Sur?!

I'd pay money, seriously.

Hopefully someone can update the source code and recompile it.
Sent from my iPhone

On Jan 22, 2021, at 13:33, Lance Jordan<EMAIL_ADDRESS>wrote:

I'd pay money, seriously.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hopefully someone can update the source code and recompile it.
Sent from my iPhone

On Jan 22, 2021, at 13:33, Lance Jordan<EMAIL_ADDRESS>wrote:

I'd pay money, seriously.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hopefully someone can update the source code and recompile it.

The problem is not the source code, it's actually 1-2 lines of simple terminal commands of copying font files to their proper places – the problem is the hacked font files. It's not clear what and how was modified in them. :(

Hopefully someone can update the source code and recompile it.

The problem is not the source code, it's actually 1-2 lines of simple terminal commands of copying font files to their proper places – the problem is the hacked font files. It's not clear what and how was modified in them. :(

TinkerTool can change the typography, has anyone tried it?
http://www.bresink.com/osx/TinkerTool.html
http://www.bresink.com/osx/0TinkerTool/Images-en/7-Fonts.png

TinkerTool can change the typography, has anyone tried it?
http://www.bresink.com/osx/TinkerTool.html
http://www.bresink.com/osx/0TinkerTool/Images-en/7-Fonts.png

FiraSystemFontReplacement es un proyecto similar a este y tiene los mismos problemas que nosotros, tal vez podríamos unir fuerzas.
https://github.com/jenskutilek/FiraSystemFontReplacement
Aquí proponen una solución parcial
https://github.com/jenskutilek/FiraSystemFontReplacement/issues/53

FiraSystemFontReplacement es un proyecto similar a este y tiene los mismos problemas que nosotros, tal vez podríamos unir fuerzas.
https://github.com/jenskutilek/FiraSystemFontReplacement
Aquí proponen una solución parcial
https://github.com/jenskutilek/FiraSystemFontReplacement/issues/53

FiraSystemFontReplacement is a similar project to this one and has the same problems as us, maybe we could join forces.
https://github.com/jenskutilek/FiraSystemFontReplacement
Here they propose a partial solution
https://github.com/jenskutilek/FiraSystemFontReplacement/issues/53

FiraSystemFontReplacement is a similar project to this one and has the same problems as us, maybe we could join forces.
https://github.com/jenskutilek/FiraSystemFontReplacement
Here they propose a partial solution
https://github.com/jenskutilek/FiraSystemFontReplacement/issues/53

I will pay for it :)

TinkerTool can change the typography, has anyone tried it?
http://www.bresink.com/osx/TinkerTool.html
http://www.bresink.com/osx/0TinkerTool/Images-en/7-Fonts.png

http://www.bresink.com/osx/0TinkerTool/Images-en/7-Fonts.jpg
This seems to only change "default font for apps to choose" (to offer for use?), not system UI font.

I've bid farewell to MacOS after Big Sur destroyed 2 2015 MBP 15" machines.  And by destroyed, I mean totally fried everything, battery, motherboard, memory, ssds, everything.  So I'm now happily running on open sourced Linux Mint distro and do not miss a thing!  I hadn't use any of Apple's native apps anyway.  Good luck with jail-breaking proprietary kneejerk spyware funded by the NSA :) I'm done.

I’ve cycled between Mojave, Catalina and Big Sur, all twice over the past 2 years and Big Sur is a whale on my 2014 15”.
Catalina was okay, but Mojave still runs the best

On Jun 4, 2021, at 8:05 AM, Lance Jordan @.***> wrote:

I've bid farewell to MacOS after Big Sur destroyed 2 2015 MBP 15" machines. And by destroyed, I mean totally fried everything, battery, motherboard, memory, ssds, everything. So I'm now happily running on open sourced Linux Mint distro and do not miss a thing! I hadn't use any of Apple's native apps anyway. Good luck with jail-breaking proprietary kneejerk spyware funded by the NSA :) I'm done.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I've bid farewell to MacOS after Big Sur destroyed 2 2015 MBP 15" machines. And by destroyed, I mean totally fried everything, battery, motherboard, memory, ssds, everything. So I'm now happily running on open sourced Linux Mint distro and do not miss a thing! I hadn't use any of Apple's native apps anyway. Good luck with jail-breaking proprietary kneejerk spyware funded by the NSA :) I'm done.

That's a shame. But this is the first time I have heard such things.


Proposal: Pass the Dio instance to interceptors

this solves problems people are having with authentication requests, were they have to create new Dio instances inside interceptors

New Pull Request Checklist

[x] I have read the Documentation
[x] I have searched for a similar pull request in the project and found none
[x] I have updated this branch with the latest main branch to avoid conflicts (via merge from master or rebase)
[x] I have added the required tests to prove the fix/feature I'm adding
[x] I have updated the documentation (if necessary)
[x] I have run the tests without failures

Can we do anything to prevent deadlocks? If not should we point this out with a warning or something similar?
Can we introduce this via a special interceptor? I'm afraid that this will break a lot of existing interceptors otherwise

Closing according to the above discussion.


pass open error to .ready callback

What about catching open errors like this:
core.ready(err => {
  // ...
})

instead of what you have to do currently:
const open = cb => {
  core.on('error', cb)
  core.ready(() => {
    core.removeListener('error', cb)
    cb(null, core)
  })
}

I'm happy to submit a PR for this

See also: https://github.com/datproject/dat-node/pull/185

That's what we used to do, but the concern that made the change was that it would make critical errors not crash the process through the error event. What do you think?

I'd be open to only having the ready handler and then no error event

/cc @pfrazee who might have opinions about this

That's what we used to do, but the concern that made the change was that it would make critical errors not crash the process through the error event. What do you think?

This should be for the consumer to decide - while appropriate for a CLI it's not appropriate for long running processes like Dat Desktop. And if the process should be crashed, you can still do it responsibly by handling the error callback argument.

Maybe wording is the problem here too, I'm assuming you're talking about the risk of people calling core.ready(() => {}) without handling the error argument. Maybe calling it core.open(err) would help?

Ya that could work. Think I like that better also

I'm fine with w/e

should open return an error if the core is already open?

Closing cause old and lots have changed


DnsZoneLexer: lexer for zone file

Lexer for DNS zone file

CI is failing, please make sure tox -e regexlint passes.

Fixed!

Merged, thanks!


new odo workflow: odo config --context

Add --context flag to odo config command.
Context specifies where the context / directory where .odo config directory is.
The default value for --context is the current directory.
If the context directory doesn't exist or .odo the directory is not present in context directory error out.
example:
odo config set mem 2G

update config in ./.odo/
odo config --context /my/project set mem 2G

update config in /my/project/.odo/
When there is no .odo dir or config file in the context directory
$ odo config set mem 2G
Unable to update configuration.
Directory /my/proj/ doesn't contain odo component.
Run `odo create -h`  for more info about component creation.

related to https://github.com/redhat-developer/odo/issues/1365


creating yaml config for learning everything

Git commit hashes seen in results/:
d60ac7a5303babaa621a1f78db1edf7240fa251c

AGGREGATED DATA OVER SEEDS:
                NUM_TRAIN_TASKS     NUM_SOLVED AVG_NUM_PREDS AVG_TEST_TIME AVG_NODES_CREATED     LEARNING_TIME  NUM_SEEDS
EXPERIMENT_ID
cover-learn_all  1000.00 (0.00)  44.70 (14.91)   5.33 (0.67)   1.90 (2.66)       6.52 (0.39)  4855.78 (566.66)         10

Output of
python scripts/grammar_search_analysis.py --env cover_multistep_options --approach grammar_search_invention --experiment_id cover-learn_all --num_train_tasks 1000 --timeout 300 --min_perc_data_for_nsrt 1 --segmenter contacts --neural_gaus_regressor_max_itr 50000 --option_learner direct_bc --seed 458 (successful seed)
Finished proxy analysis for cover_multistep_options in 17.110 seconds
                       Env                    Preds  expected_nodes_created Score  expected_nodes_created Time
0  cover_multistep_options  Clear,HandEmpty,Holding                  2.942130e+04                     2.099716
1  cover_multistep_options        HandEmpty,Holding                  2.977531e+04                     2.637141
2  cover_multistep_options  HandEmpty,NOT-HandEmpty                  3.523277e+04                     1.930141
3  cover_multistep_options                HandEmpty                  3.605127e+07                     1.987011
4  cover_multistep_options                  Holding                  3.905726e+04                     2.088376
5  cover_multistep_options            NOT-HandEmpty                  4.274485e+04                     1.752943
6  cover_multistep_options                   [none]                  3.605512e+07                     1.498375

Output of
python scripts/grammar_search_analysis.py --env cover_multistep_options --approach grammar_search_invention --experiment_id cover-learn_all --num_train_tasks 1000 --timeout 300 --min_perc_data_for_nsrt 1 --segmenter contacts --neural_gaus_regressor_max_itr 50000 --option_learner direct_bc --seed 459 (failed seed)
Finished proxy analysis for cover_multistep_options in 17.061 seconds
                       Env                    Preds  expected_nodes_created Score  expected_nodes_created Time
0  cover_multistep_options  Clear,HandEmpty,Holding                  3.714028e+04                     2.107268
1  cover_multistep_options        HandEmpty,Holding                  3.744828e+04                     2.584901
2  cover_multistep_options  HandEmpty,NOT-HandEmpty                  4.243671e+04                     1.893703
3  cover_multistep_options                HandEmpty                  3.655115e+07                     1.966178
4  cover_multistep_options                  Holding                  4.636025e+04                     2.032862
5  cover_multistep_options            NOT-HandEmpty                  5.018671e+04                     1.815054
6  cover_multistep_options                   [none]                  3.654988e+07                     1.608502

output of
python predicators/main.py --env cover_multistep_options --approach grammar_search_invention --experiment_id cover-learn_all --num_train_tasks 1000 --excluded_predicates all --timeout 300 --min_perc_data_for_nsrt 1 --segmenter contacts --neural_gaus_regressor_max_itr 50000 --option_learner direct_bc --seed 459 --grammar_search_use_handcoded_debug_grammar True

(note: --grammar_search_use_handcoded_debug_grammar True)
Hill climbing summary:
	On step 1, added NOT-((0:block).grasp<=[idx 0]0.0), with heuristic 46360.252 (an improvement of 36500944.194 over the previous step)
	On step 2, added Forall[0:block].[((0:block).grasp<=[idx 0]0.0)(0)], with heuristic 37448.280 (an improvement of 8911.972 over the previous step)

Filtering out predicates that don't appear in preconditions...

Selected 2 predicates out of 2 candidates:
	Forall[0:block].[((0:block).grasp<=[idx 0]0.0)(0)]
	NOT-((0:block).grasp<=[idx 0]0.0)
Evaluating predicates: frozenset({Forall[0:block].[((0:block).grasp<=[idx 0]0.0)(0)], NOT-((0:block).grasp<=[idx 0]0.0)}), with total cost 5.0
	Total score: 37448.280043403705 computed in 2.379 seconds
Done.

But then evaluation is failing? 🤔

Btw here are the learned NSRTs for the failed run
Learned NSRTs:
NSRT-Op0:
    Parameters: [?x0:robot]
    Preconditions: [((0:robot).grip<=[idx 0]0.0)(?x0:robot)]
    Add Effects: [NOT-((0:robot).grip<=[idx 0]0.0)(?x0:robot)]
    Delete Effects: [((0:robot).grip<=[idx 0]0.0)(?x0:robot)]
    Ignore Effects: []
    Option Spec: Op0LearnedOption(?x0:robot)
NSRT-Op1:
    Parameters: [?x0:block, ?x1:robot, ?x2:target]
    Preconditions: [Forall[0:block].[NOT-Covers(0,1)](?x2:target), Forall[1:target].[NOT-Covers(0,1)](?x0:block), NOT-((0:robot).grip<=[idx 0]0.0)(?x1:robot)]
    Add Effects: [((0:robot).grip<=[idx 0]0.0)(?x1:robot), Covers(?x0:block, ?x2:target)]
    Delete Effects: [Forall[0:block].[NOT-Covers(0,1)](?x2:target), Forall[1:target].[NOT-Covers(0,1)](?x0:block), NOT-((0:robot).grip<=[idx 0]0.0)(?x1:robot)]
    Ignore Effects: []
    Option Spec: Op1LearnedOption(?x0:block, ?x1:robot, ?x2:target)

Notice that the first one (for picking) doesn't include any block. It seems obvious that adding Holding should improve it...

I'm beginning to suspect that this is due to suboptimal demonstrations. In this environment, we would expect the first skeleton to succeed every time. Checking the skeleton counts in the demos...

Planning succeeded! Found plan of length 4 after 3 skeletons, discovering 0 failures

Among several others

Retrying with offline_data_max_skeletons_optimized 1

what the heck
Terminating hill climbing, could not improve score

Hill climbing summary:
	On step 1, added NOT-((0:robot).grip<=[idx 0]0.0), with heuristic 9585.195 (an improvement of 32737858.765 over the previous step)

Filtering out predicates that don't appear in preconditions...

Selected 1 predicates out of 5 candidates:
	NOT-((0:robot).grip<=[idx 0]0.0)
Evaluating predicates: frozenset({NOT-((0:robot).grip<=[idx 0]0.0)}), with total cost 2.0
	Total score: 9585.194661174035 computed in 1.826 seconds
Done.

Using --offline_data_max_skeletons_optimized 1 --sesame_max_samples_per_step 100 doesn't drop any demos, so that confirms what I thought about the skeletons being downward refinable.
Okay super bizarrely this works...
python predicators/main.py --env cover_multistep_options --approach grammar_search_invention --experiment_id cover-learn_all --num_train_tasks 1000 --excluded_predicates all --timeout 300 --min_perc_data_for_nsrt 1 --segmenter contacts --neural_gaus_regressor_max_itr 50000 --option_learner direct_bc --seed 459 --grammar_search_use_handcoded_debug_grammar True --offline_data_max_skeletons_optimized 1 --sesame_max_samples_per_step 100

Hill climbing summary:
	On step 1, added NOT-((0:robot).grip<=[idx 0]0.0), with heuristic 10001.314 (an improvement of 33969022.095 over the previous step)
	On step 2, added ((0:robot).grip<=[idx 0]0.0), with heuristic 10000.294 (an improvement of 1.020 over the previous step)
	On step 3, added Forall[0:block].[NOT-Covers(0,1)], with heuristic 7669.304 (an improvement of 2330.990 over the previous step)
	On step 4, added Forall[1:target].[NOT-Covers(0,1)], with heuristic 7336.301 (an improvement of 333.003 over the previous step)
	On step 5, added NOT-((0:block).grasp<=[idx 0]0.0), with heuristic 7335.305 (an improvement of 0.996 over the previous step)

Filtering out predicates that don't appear in preconditions...

Selected 5 predicates out of 5 candidates:
	Forall[0:block].[NOT-Covers(0,1)]
	NOT-((0:robot).grip<=[idx 0]0.0)
	Forall[1:target].[NOT-Covers(0,1)]
	NOT-((0:block).grasp<=[idx 0]0.0)
	((0:robot).grip<=[idx 0]0.0)
Evaluating predicates: frozenset({Forall[0:block].[NOT-Covers(0,1)], NOT-((0:robot).grip<=[idx 0]0.0), Forall[1:target].[NOT-Covers(0,1)], NOT-((0:block).grasp<=[idx 0]0.0), ((0:robot).grip<=[idx 0]0.0)}), with total cost 16.0
	Total score: 7335.304876264762 computed in 1.884 seconds
Done.

Why would filtering out the 50 "bad" demos as above affect predicate invention?

I guess if we look here at the "successful" case:
	On step 5, added NOT-((0:block).grasp<=[idx 0]0.0), with heuristic 7335.305 (an improvement of 0.996 over the previous step)

the improvement is tiny. I would expect the improvement to be much better for the only predicate that can distinguish which block is going to get picked...

Note that in this latter case where 4 predicates are getting selected, the first skeleton generated always appears refinable. This makes sense. But I would expect that the number of nodes created should increase with these operators. Oh, but maybe it doesn't, because the block isn't included in the Pick operator, so there only appears to be one operator.

And it makes sense that this would never happen when options are given because the option spec would force the pick operator to include the block option, so that would increase the branching factor.

Two possible conclusions:

Force block to appear in the option arguments (e.g., by noticing change in the low-level state)
The option policies should be able to take in the whole world

Both could be true?

Results after sampler update:
AGGREGATED DATA OVER SEEDS:
                NUM_TRAIN_TASKS    NUM_SOLVED AVG_NUM_PREDS AVG_TEST_TIME AVG_NODES_CREATED     LEARNING_TIME  NUM_SEEDS
EXPERIMENT_ID
cover-learn_all  1000.00 (0.00)  49.60 (0.92)   5.40 (0.80)   2.82 (3.10)       6.33 (0.29)  4898.16 (692.09)         10

closing in favor of #1233


About TransCap with VoteNet proposal

May I ask how you train the model using VoteNet?

May I ask how you train the model using VoteNet?

It seems that the released code doesn't support training or testing with predicted bboxes by votenet