Getters/setters for properties

So, I just found out this repo and looking at the limitations, I noticed something odd:

Ruby's method semantic is incompatible with the seperate properties and methods in GDScript since there are only methods in Ruby. For this we have to use get which looks weird [...] rect2.get(:position)

All properties in Godot already have getters/setters, e.g. it's get_position()/set_position() for position property. Isn't it possible to bind Ruby methods to these methods? Or auto-generate Ruby attribute accessors? It should be possible to use position/position= directly as methods instead of awkward get/set.

Hi @KoBeWi,
Sorry for the late reply. I can only try to explain the problem since I haven't been working on the plugin for so long.
What you said is possible, but might introduce heavy overheads. Consider a ruby code gets a godot object a, it does not know what properties and methods a responds to. Suppose the user call from the ruby side a.some_method, it is now uncertain how should we interpret this method call. It can be (1) getting a property named some_method, or (2) calling a method named some_method.
In order to support this, we have to first use reflection to list the properties and methods of an object, then decide what to do. But GDScript is also a dynamic language, thus it is possible that every time we use a godot object, it has a different properties list. In the worst case, we would need to reflect every time when calling on a godot object to ensure correctness.

tbh I don't know how this works in GDNative. Maybe it's possible to predefine some Ruby classes based on the API, so that the setters/getters are known (at least for core classes). And in worst case, they can be cached on first lookup.

You are right. It is possible to cache the method table and reflect only when method_missing, and it should be sufficient for most cases.


Resolve Issue #36 update requirements.txt

Problem
The previous version of the archivebox yunohost package had an outdated requirements.txt, which is required to properly install archivebox 0.7.2
Solution
I updated the contents of requirements.txt from 0.6.2 to 0.7.2
PR Status

[x] Code finished and ready to be reviewed/tested
[x] The fix/enhancement were manually tested (if applicable)

Automatic tests
Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization) I'm going to try anyways

Hmmm... Seeing less consistent archival results. Please test before commit.
Target: all
This branch (raspberry pi 3B):

Past branch (raspberry pi 4B):

But on the positive side, I am seeing a lot more of " it just works", e.g. on the current branch, it would often get stuck on something until you manually tell it to pull it (e.g. title), but now it seems to be handling that just fine.


WIP: Master template fix

Fix master template downloader issue

Please fix the spelling errors in your code and return whether the process succeeded in the method.
Adding documentation would be nice as well.

Fixed Spelling and add documentation

looks good to me


Can I use it for not an authorized users?

I'm using advanced environment.
And me need to use it for not authorized users.
Can I do it?

Hi, no, you can't add comments as an unauthorized user.

I'm sorry, I don't have enough time to implement this feature.

That's to bad.
But thank you for answer, you saved my time.


New Minecraft PE kb and images

New Minecraft PE kb.  This is the server that iOS clients connect to.
Images are screen shots.

Fix those couple of things and LMK and I'll merge it in and release it.

Cool - Thanks a lot for the guidance, Matt!!  I'll make the changes and let you know when ready as you suggested.
Thanks again,
Bob Stolzberg | Sr. Cloud Integration Engineer
CenturyLink
Mobile/Text:<PHONE_NUMBER>
E-Mail<EMAIL_ADDRESS>www.CenturylinkCloud.com
From: Matthew Osborn <notifications<EMAIL_ADDRESS>Reply-To: CenturyLinkCloud/PublicKB <reply<EMAIL_ADDRESS>Date: Mon, 18 May 2015 10:05:08 -0700
To: CenturyLinkCloud/PublicKB <PublicKB<EMAIL_ADDRESS>Cc: User <bob.stolzberg<EMAIL_ADDRESS>Subject: Re: [PublicKB] New Minecraft PE kb and images (#128)
Fix those couple of things and LMK and I'll merge it in and release it.

Reply to this email directly or view it on GitHubhttps://github.com/CenturyLinkCloud/PublicKB/pull/128#issuecomment-103132146.
This communication is the property of CenturyLink and may contain confidential or privileged information. Unauthorized use of this communication is strictly prohibited and may be unlawful. If you have received this communication in error, please immediately notify the sender by reply e-mail and destroy all copies of the communication and any attachments.

Images in list or bullets need to have an empty line (hit enter twice) with 2 spaces (per indent aka sub lists would get 4 spaces) in front of the image.
1. some test

   //start image tag here

stale


Update Langfuse filter pipeline

Fixes #2
Update example/langfuse_filter_pipeline.py to use the updated Langfuse filter pipeline.

Great, thanks.
Is there any change needed in the README?

thank you for reviewing this so quickly.
you could refer to this guide which includes screenshots in our docs: https://langfuse.com/docs/integrations/openwebui

I prefer to keep pk and sk as an example.
Thanks for your contribution.

@marcklingen Somehow, I managed to commit directly to your repository without a PR: commit link. Is this an intended feature, or could it be a bug of GitHub? Could you possibly open a dummy pull request so I can test if this happens again?


Can we have demo CNN application using ARM micro controller with Camera for object detection please?

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): Experimental Micro-controller version
Are you willing to contribute it (Yes/No): Not now.

Describe the feature and the current behavior/state.
Will this change the current api? How?
Who will benefit with this feature?
Any Other info.

https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/experimental/micro/examples/micro_vision might the closest example code for now. It's still image classification not yet detection though.

I'm closing this as we do have a vision example available now.


remove dependency on setuptools

Remove the dependency setuptools from pyproject.toml
Description
pkg_resources is no longer used in the spotdl codebase.
However, it is an undeclared dependency of pykakasi. @miurahr It would be nice if you could merge https://github.com/miurahr/pykakasi/compare/master...dotlambda:pykakasi:patch-1.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate)
Types of Changes

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

Checklist

[ ] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have read the CONTRIBUTING document
[ ] I have read the  CORE VALUES document
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

@dotlambda  Thanks for the suggestion. If you are interested in pykakasi project for contribution, please look a new home of the project at https://codeberg.org/miurahr/pykakasi


Fixed multiple events per change, added dynamic positioning of watchers

Defer naming problem, ideally creating subsection for TFC would be perfect. Merging.


URL-Encode Password in influxdb/client

Hey
Password with special characters had to be escaped manually in order to work before. Because that is something the user shouldn't have to do, I added the encoding of the password.
In order to not brake other programs that fixed this themselves (influx-cli for example) I added an additional url.QueryUnescape before escaping it.
Was fixed for example in Dieterbe/influx-cli#15

The client code is getting heavily reworked right now, so I am not sure if it is worth merging. @corylanou ?

@otoolep the entire client is getting re-worked from the ground up.  100% re-write so there will be nothing to merge.

OK.
Thanks @MakaHost -- I'm going to close this, since the code is being entirely re-written, that should be soon, so keep an eye out for similar issues.


Keypoint format

I get the coco 17 key-points or any other key-point format of my own custom data, and I know I should convert the coco format to human3.6, but how? The definition between coco and human 3.6 is different especially for the body. Is there any way to convert the format between these datasets?

The lib/data/dataset_action.py has a coco2h36m function that accepts a numpy ndarray of keypoints and does a rough conversion to the human3.6m format. I think I've seen other projects take a similar approach.

The lib/data/dataset_action.py has a coco2h36m function that accepts a numpy ndarray of keypoints and does a rough conversion to the human3.6m format. I think I've seen other projects take a similar approach.

Thank you so much ! I will check this out, as soon as possible.

pose track and coco ,human3.6 ,AMASS ,there are all have different keypoints style, how to get unified keypoints ?

On the other hand the original Cascaded Pyramid Network uses COCO format and the fine-tuned version which was available was H3.6 format. I was wondering whether there is documentation available somewhere about how they fine-tune CPN. Thanks :)


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1737 ms

Resolved: Bofrost.at performance has improved in c903274 after 6 minutes.


tab divider is displayed at the selected tab

Description
Something very minor
tab divider is displayed at the selected tab
it should probable only be displayed between not selected tabs

Steps To Reproduce

create new tabs
switch between tabs
(sometimes it is also displayed if you hover over other tabs)

Requirements
only shown tab divider between non selected tabs and stop it from flickering
Files Version
v<IP_ADDRESS>
Windows Version
10.0.22621.2506
Log File
debug.log

Thanks for the feedback, I've been seeing quite a few of these reports. @chingucoding do you know if this is a bug with the WinUI tab control?

Since the issue is also present in the WinUI Gallery, I'm fairly sure its not on our end. The issue is that the divider is also present on the last tab even though it should be hidden which probably makes a workaround a bit harder to implement.


Explicitly mention 404 error using the HTTPS backend

Description
Using the --from tfstate+https:// new backend support to access terraform states, if I request the wrong file, I expect to know about it (like "Sorry, no t3rr4f0rm.tfstate is available at this location"), more than a json parsing issue (as there's no file to parse obviously)
$ driftctl scan --from tfstate+https://<redacted>.net/tfstates/wrong.tfstate
Scanning resources: ⣾ (0)
2 problems:

- Unsupported state file format: The state file could not be parsed as JSON: syntax error at byte offset 1.
- Unsupported state file format: The state file does not have a "version" attribute, which is required to identify the format version.

A HTTP/404 is returned:
$ curl -IL https://<redacted>.net/tfstates/wrong.tfstate
HTTP/2 404
server: nginx
date: Fri, 26 Mar 2021 16:45:54 GMT
content-type: text/html
content-length: 146

Environment

OS: linux
driftctl version: 0.7.0

How to reproduce

Possible Solution

Additional context

I can take this since I worked on initial implementation. I think every status code <200 and >=400 should throw an error.

I can take this since I worked on initial implementation. I think every status code <200 and >=400 should throw an error.

We should ensure that we follow 301 & 302 redirections too

I can take this since I worked on initial implementation. I think every status code <200 and >=400 should throw an error.

We should ensure that we follow 301 & 302 redirections too

I can confirm the backend already support 301 & 302 redirections.

@eliecharra You changed milestone to v0.9 but the PR already got merged into v0.8, should we put the milestone back to v0.8 then?

It was a mistake indeed @sundowndev


[sil-mode] Change [attribute] highlighting to use a non-greedy + so that [take] [initialization] does not hightlight the ][ in between the two.

[sil-mode] Change [attribute] highlighting to use a non-greedy + so that [take] [initialization] does not hightlight the ][ in between the two.

@swift-ci Please smoke test and merge


Optionally write ca.crt to ConfigMap

Is your feature request related to a problem? Please describe.
If you need to specify a BackendTLSPolicy when using the GatewayAPI you need the ca.crt inside a ConfigMap.
See:

https://gateway-api.sigs.k8s.io/api-types/backendtlspolicy/#cacertficaterefs
https://gateway-api.sigs.k8s.io/guides/tls/#using-explicit-ca-certificates

Currently if you create a certificate, the ca.crt will be only created inside a secret.
Describe the solution you'd like
Would be nice, if you could also specify configMapName (optionally), next to the secretName inside the Kind:Certificate.
Result would be if specified, that a new ConfigMap is created with the appropriate ca.crt entry (for sure not including the private key!)
Furthermore is does make sense to also add configMapTemplate (same way as secretTemplate is defined).
/kind feature

I think this use case is better solved using trust-manager, as using the certificate ca.crt will not be safe when the certificate is renewed. Please see the FAQ entry for details: https://cert-manager.io/docs/faq/#chain-cacrt.
/close


The question about the visualization

Dear sir:
In your journal version, which visualization tool do you use? pca or tsne ?

TSNE


SWR should suspense when key is falsey

When using SWR with Suspense, it's guaranteed that render won't continue unless data is ready:
function App() {
  const { data } = useSWR('key', fetcher, { suspense: true })
  assert(data !== undefined)
}

However if the key is conditional or falsey,
function App() {
  const { data } = useSWR(() => ready ? 'key' : null, fetcher, { suspense: true })

  assert(data !== undefined) // error!
}

SWR will treat undefined as a returned data because there's no promise created.

Related to https://github.com/zeit/swr/issues/266 I believe 👍

Commented here: https://github.com/zeit/swr/pull/357#issuecomment-627089889


FAI-11873 - Return exit code from containers

Description

Provide description here

Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change

Related issues

Fix #1

Migration notes

Describe migration notes if any

Extra info

Add any additional information

test failed


esp32: initializing a display should not initialize the framebuffer as well

Trying to enable the framebuffer char device would lead to a stack overflow due to a recursive loop - fb_register, up_fbinitialize, board_lcd_initialize, fb_register again.
I removed the last call to fb_register, ending the recursion. This has no impact on the functionality because in the bring-up code there are separate calls to board_lcd_initialize (when using LCD char device) and fb_register (when using framebuffer char device). So there was no need to call couple fb_register together with the display initialization.

@robertalexa2000 I think the issue is because fb_register is already called in your board bring-up:
esp32-sparrow-kit/src/esp32_bringup.c:  ret = fb_register(0, 0);
I think you shouldn't remove it from esp32/common, otherwise other boards that doesn't call fb_register() in the esp32_bringup.c file will failed.
Correct solution: remove fb_register() from your (and all other boards)

@acassis wouldn't it make more sense to keep my patch and add fb_register to those boards not calling it rather than to remove it from esp32_bringup.c? I don't think we should bind the LCD char device to the the framebuffer char device - we can initialize and use the framebuffer even if we decide not to register an LCD char device.

@acassis wouldn't it make more sense to keep my patch and add fb_register to those boards not calling it rather than to remove it from esp32_bringup.c? I don't think we should bind the LCD char device to the the framebuffer char device - we can initialize and use the framebuffer even if we decide not to register an LCD char device.

fb_register() only will be called for boards that enabled CONFIG_VIDEO_FB and CONFIG_LCD_FRAMEBUFFER, so in this case you we really requesting to create a /dev/fb0 device. If you don't want to bind to framebuffer device, just don't enable the CONFIG_VIDEO_FB.
Keeping it at common/ and removing for each board's esp32_bringup.c avoid code duplication too.

Hi @acassis, sorry for taking so long to reply. Coming back to this - I understand what you are saying, but in that case we would couple the framebuffer char device with the LCD char device. Essentially, in order to enable framebuffer support, we will need to enable CONFIG_LCD_DEV - and there is no need for that since the framebuffer char device and the LCD char device offer completely distinct APIs.
Besides, I believe that this approach will still not work. If we were to call fb_register from inside common/, we would still end up in a stack overflow because fb_register itself calls board_lcd_initialize, which in turn would call fb_register again. I also took a look at other boards and they all initialize the frame buffer from _bringup.c, so I believe if we were to remove board_lcd_initialize from fb_register we would solve the issue on esp32 but break the framebuffer support on other boards.


$queryRaw fails when passing bigint as parameter

Bug description
My Postgres database uses BigInt for identifiers, but $queryRaw fails loudly when I try to pass a bigint as a parameter:
Raw query failed. Code: `08P01`. Message: `db error: ERROR: insufficient data left in message`

How to reproduce
From the Node REPL:
await prisma.$queryRaw`SELECT ${1n}::bigint;`

Uncaught PrismaClientKnownRequestError2 [PrismaClientKnownRequestError]:
Invalid `prisma.queryRaw()` invocation:

  Raw query failed. Code: `08P01`. Message: `db error: ERROR: insufficient data left in message`
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at cb (/app/prisma-client/runtime/index.js:33896:17) {
  code: 'P2010',
  clientVersion: '2.26.0',
  meta: {
    code: '08P01',
    message: 'db error: ERROR: insufficient data left in message'
  }
}

Passing in a number does not fail, but reproduces https://github.com/prisma/prisma/issues/7395 (the resulting value is not a bigint):
await prisma.$queryRaw`SELECT ${1}::bigint;`

[ { int8: 1 } ]

Expected behavior
Expected 1n to be serialized as a Postgres bigint; expected query not to fail.
Prisma information
This is reproducible without a schema
Environment & setup

OS: Alpine Linux 3.13
Database: Postgres 13.3
Node.js version: 14.17.1
Prisma version: 2.26.0

Prisma Version
npm exec prisma -v
6.14.11

This is reproducible, we need to fix bigints in raw queries in general.

This is currently blocking for my tweetscape project that uses a lot of complex raw queries that have bigint parameters. Are there any suggested workarounds to accurately query a bigint field while #12367 is still in development?

This is currently blocking for my tweetscape project that uses a lot of complex raw queries that have bigint parameters. Are there any suggested workarounds to accurately query a bigint field while #12367 is still in development?

This is ugly but it works:
await prisma.$queryRaw`SELECT ${1n.toString()}::text::bigint;`

Confirmed this works in 3.14.0. You don't need to specify ::bigint, just WHERE id=${bigIntHere} works as-is 🎉


AI chatbot for your documentation

Hey!
I noticed Chainguard and found it very cool! You’re building something really valuable for the community!
As I was struggling with understanding, I’ve integrated your documentation with an AI chatbot tool my team is building (https://glimelab.ai/) and found it remarkably user-friendly and informative. We'd love to show you a demo of how it enhances the user experience. Would you like to check it out?

Spamming projects on GitHub is a good way to ensure companies never do business with you.


Chrome store page gone

The chrome web store is in 404, is it normal ? https://chrome.google.com/webstore/detail/dknlfmjaanfblgfdfebhijalfmhmjjjo

The extension was temporarily hidden. The page should now be accessible.


Basic environment setup and validation script

Script should have the following functionality

Change default password
Try to create a project
Try to invite a new user
Try to invite a new user
Try to create a resource
Try to verify that external auth (e.g. Auth0) is working

Consider using command line tool for script

https://docs.google.com/document/d/18THwI8gKVeCeRd8lanKXsO_CYP_0K4kvEFWvROYZG84/edit


lint:po - lint locale/*.po files to ensure same vars and components ({foo}, {0}, <0>, <0 />, </0>)

Fixes AAH-804
Quick and dirty *.po linter to check for common problems.
Right now, this will catch msgid/msgstr pairs with a different set of varible names, or components...
$ cat problem.po
msgid "Hello {name}"
msgstr "Bonjour {nom}"

$ npm run lint:po
...
Difference between msgid="Hello {name}" and msgstr="Bonjour {nom}":
  Missing from msgstr: {name}
  Unexpected in msgstr: {nom}
  at problem.po:2

Related to https://github.com/ansible/ansible-hub-ui/pull/1143#issuecomment-954998391 (and thus https://github.com/ansible/ansible-hub-ui/pull/810)
And enabling on github actions, which should consequenty fail with the current fr.po problems :)
TODOs:

catch nesting errors (<0><1></0></1>), reordering is fine but nesting still needs to make sense
maybe rewrite in python/js
widen {\w+} to also accept P{Letter} or something.. {opération} is not detected now
extend for api use, add there .. /(\{(\w+|\d+)\})|(<\/?\d+\/?>)|(%(\(\w+\))?.)/g, ensure counts (3x %s vs 4x %s, etc.)
make sure everything from placeholders.rb / AAH-804 is addressed

Added a github actions error formatting, fixed the line numbers, removed the package.json for loop and rewritten in ruby...


Key-less non-singleton entities should use parent or ancestor keys as their keys

When an entity table does not have keys, attempting to update an existing row in it ends up creating a new row in the table. To prevent this, all entity tables, except singleton entity (#67) tables, must have key columns. If an entity does not have its own key fields, then the key fields of its parent or ancestor must be used. This prevents such entities from being composed by multiple parents or ancestors.

Turns out that there is support for this already. And it allows keyless entities to be composed by multiple parents or ancestors! It supports multiple parents by using UNIQUE INDEX instead of PRIMARY KEY.
But when I added dedicated unit tests for this feature, it exposed a few flaws in the implementation. I am keeping this ticket open for those new unit tests and fixes.


Live log view in manager UI with SSE

Adds a log view in the manager UI that updates live with SSE, and an /api/livelog endpoint.
Adds in_memory_backend that sends new log entries to any connected clients via channels.
The in_memory_backend also keeps the latest 128 messages in memory and sends them to newly connected clients.

Did you use crystal tool format?

Yes!

no, i was thinking of just that. how can we make it bigger?

no, i was thinking of just that. how can we make it bigger?

It depends on setup I guess, but in some cases it could be possible to download the actual log files? If we know where the log files are stored and have access to them. So, maybe not when using stdout/journal for logging, but if you've specified a logfile in config.
But I agree, we should just implement a download for this solution for now.

Would you like to add something else or are we good to squash merge this @viktorerlingsson ?

Would you like to add something else or are we good to squash merge this @viktorerlingsson ?

We're good to merge! Just haven't gotten around to it


Unenroll from organization membership if you hit the back button in onboarding.

During onboarding, if you select an Organization, then hit the back button and either skip or selected a different organization, you'll be removed from the previously selected organization.
Note: This is a work in progress, and still needs a little cleaning up. Also contingent on whether or not @izzyalonso thinks this is a good idea.

A very simple solution. :+1:


Hangs when running via API

I tried to run pkg as part of a node script, following the example in the README:
const { exec } = require('pkg');
exec(args).then(() => console.log("Finished generating exe"));

But the promise is never resolved or rejected even after the output file is created.
My node version is v8.6.0
Running on Windows 10

Failed to reproduce. It always outputs Finished generating exe for me. Please provide args.

Not sure what I did wrong but it's working for me now. Withdrawing this issue.

I remember the issue now - it works ok when generating one binary, but I get an error when trying to generate more than one.
async function fred() {
    console.log('before win');
    try {
        await exec (['packages/prog_exercise_search/bin/www', '--target', 'win', '--output', 'fred.exe']);
    } catch (err) {
        console.log(err);
    }
    console.log('after win');
    console.log('before linux');
    try {
        await exec (['packages/prog_exercise_search/bin/www', '--target', 'linux', '--output', 'fred.linux']);
    } catch (err) {
        console.log(err);
    }
    console.log('after linux');
    console.log('before macos');
    try {
        await exec (['packages/prog_exercise_search/bin/www', '--target', 'macos', '--output', 'fred.macos']);
    } catch (err) {
        console.log(err);
    }
    console.log('after macos');
    process.exit(0);
}

I get the following output:
before win
><EMAIL_ADDRESS>after win
before linux
><EMAIL_ADDRESS>events.js:182
      throw er; // Unhandled 'error' event
      ^

Error: This socket has been ended by the other party
    at Socket.writeAfterFIN [as write] (net.js:354:12)
    at fabricate (C:\Users\tclgu\git_repos\tufan-cli-pkgr\node_modules\pkg\lib-es5\fabricator.js:88:15)
    at fabricateTwice (C:\Users\tclgu\git_repos\tufan-cli-pkgr\node_modules\pkg\lib-es5\fabricator.js:96:3)
    at MultiStream._queue (C:\Users\tclgu\git_repos\tufan-cli-pkgr\node_modules\pkg\lib-es5\producer.js:116:53)
    at MultiStream._next (C:\Users\tclgu\git_repos\tufan-cli-pkgr\node_modules\multistream\index.js:72:10)
    at Meter.onEnd (C:\Users\tclgu\git_repos\tufan-cli-pkgr\node_modules\multistream\index.js:119:10)
    at Object.onceWrapper (events.js:314:30)
    at emitNone (events.js:110:20)
    at Meter.emit (events.js:207:7)
    at endReadableNT (_stream_readable.js:1059:12)

Afterwards, fred.win and fred.linux exist but not fred.macos.


Create a cut-down layout from an existing layout

If I have an existing "full" layout (no --add filters), and I want to create a new layout with fewer components (for reduced disk space) while still using the existing layout as a source rather than re-downloading packages ... what is the command for that?
I tried specifying --installChannelUri as documented in "Use the command line" but apparently this is not valid with --layout.

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

ID: ff556dcf-4de1-8483-702a-73699062be1f
Version Independent ID: cba846ae-6af4-af49-b194-28b546169426
Content: Create a network-based installation of Visual Studio - Visual Studio
Content Source: docs/install/create-a-network-installation-of-visual-studio.md
Product: visual-studio-dev15
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@uecasm For installation support requests, please consider using either our Live Chat (English only) support option for installation issues, or reaching out to us and other Visual Studio developers through our Visual Studio conversation in the Gitter community.

This was a feature request, not a support request.

@uecasm Thanks for the further clarification. For information about that, please see our Suggest a feature page. (As an FYI, these options are listed in the "Get support" section of any installation-related page.)


[cxx-interop] Allow more C++ decls in public Swift interfaces

This is a follow-up to 8859b629.
Let's allow usages of C++ global variables, fields and constructors, as long as the types involved aren't fragile, e.g. if they are C-like structs.
This resolves compiler errors when trying to rebuild System.swiftmodule from its textual interface with Xcode 16.1.
rdar://140203932
rdar://141124318

@swift-ci please test

@swift-ci please test

@swift-ci please test Windows


Stop users from entering a fractional number of sentences in the settings

Stop users from entering a fractional number of sentences (like 3.3) in the settings.

Made some frontend changes to prevent this issue.


[Addition]: False location of miner Clever Merlot Snail

Hotspot b58 Addresses
11wX6NewfvdpLYNxgbrvCrLPhNKjTA97bNoXf5Rm48e5vXT9KD9
Discord Handle
bojansb
Reason(s)
Hi, I just wanted to report one miner that is not in asserted location, and that is Clever Merlot Snail with the address 11wX6NewfvdpLYNxgbrvCrLPhNKjTA97bNoXf5Rm48e5vXT9KD9
I dont belive that in this case there is bad intentions or fault behavior, I belive that miner owner don't know how to set a new location in app.
On 12.July 2022 there was a miner transfer from one owner to a new owner, and I belive a new owner don't know that he needs to change location of the miner.
Before the miner transfer miner indeed was in town Slavonski Brod in Croatia, but after the transfer, miner witness only signals from other town called Dakovo, which is about 35 km north-east.
I don' want any harm to miner owner, but this is not fair, and can someone please add this miner to the deny list.
Thank You.
Bojan

Issue resolved, owner reasserted correct location.


HDFS-16473. Make HDFS stat tool cross platform

Description of PR
The source files for hdfs_stat uses getopt for parsing the command line arguments. getopt is available only on Linux and thus, isn't cross platform. We need to replace getopt with boost::program_options to make this tool cross platform.
How was this patch tested?
hdfs_stat
This tool was testing locally with the Hadoop cluster setup in pseudo-distributed mode.
I ran the tool by providing the path to a file as the argument.
$ ./hdfs_stat ~/some-dir/some-data.txt

I verified the output against the information displayed in the UI.

I then ran the tool by providing the path to a directory as the argument.
$ ./hdfs_stat ~/some-dir

I verified the output against the information displayed in the UI.

I then ran the tool by providing a non-existent path as the argument.
$ ./hdfs_stat ~/some-non-existent-path

We note that an appropriate error message is reported in the above output.
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
38m 31s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 4 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
41m 39s

trunk passed

+1 :green_heart:
compile
3m 50s

trunk passed

+1 :green_heart:
mvnsite
0m 25s

trunk passed

+1 :green_heart:
shadedclient
65m 50s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 14s

the patch passed

+1 :green_heart:
compile
3m 42s

the patch passed

+1 :green_heart:
cc
3m 42s

the patch passed

+1 :green_heart:
golang
3m 42s

the patch passed

+1 :green_heart:
javac
3m 42s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 17s

the patch passed

+1 :green_heart:
shadedclient
19m 18s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
32m 46s

hadoop-hdfs-native-client in the patch passed.

+1 :green_heart:
asflicense
0m 39s

The patch does not generate ASF License warnings.

163m 40s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4145/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4145

Optional Tests
dupname asflicense compile cc mvnsite javac unit codespell golang

uname
Linux d1c9363bf8ba 4.15.0-169-generic #177-Ubuntu SMP Thu Feb 3 10:50:38 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / af8487991283df5dd5052dc094e181b544bc7409

Default Java
Red Hat, Inc.-1.8.0_322-b06

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4145/1/testReport/

Max. process+thread count
682 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-native-client U: hadoop-hdfs-project/hadoop-hdfs-native-client

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4145/1/console

versions
git=2.9.5 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
23m 55s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 4 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
27m 43s

trunk passed

+1 :green_heart:
compile
4m  1s

trunk passed

+1 :green_heart:
mvnsite
0m 42s

trunk passed

+1 :green_heart:
shadedclient
54m 57s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 22s

the patch passed

+1 :green_heart:
compile
3m 46s

the patch passed

+1 :green_heart:
cc
3m 46s

the patch passed

+1 :green_heart:
golang
3m 46s

the patch passed

+1 :green_heart:
javac
3m 46s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 24s

the patch passed

+1 :green_heart:
shadedclient
22m 48s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
37m 36s

hadoop-hdfs-native-client in the patch passed.

+1 :green_heart:
asflicense
0m 47s

The patch does not generate ASF License warnings.

147m  7s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4145/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4145

Optional Tests
dupname asflicense compile cc mvnsite javac unit codespell golang

uname
Linux 415ac826771a 4.15.0-169-generic #177-Ubuntu SMP Thu Feb 3 10:50:38 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / af8487991283df5dd5052dc094e181b544bc7409

Default Java
Red Hat, Inc.-1.8.0_312-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4145/1/testReport/

Max. process+thread count
691 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-native-client U: hadoop-hdfs-project/hadoop-hdfs-native-client

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4145/1/console

versions
git=2.27.0 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
12m 16s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 4 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
30m 21s

trunk passed

+1 :green_heart:
compile
3m 40s

trunk passed

+1 :green_heart:
mvnsite
0m 35s

trunk passed

+1 :green_heart:
shadedclient
64m 41s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 16s

the patch passed

+1 :green_heart:
compile
3m 37s

the patch passed

+1 :green_heart:
cc
3m 37s

the patch passed

+1 :green_heart:
golang
3m 37s

the patch passed

+1 :green_heart:
javac
3m 37s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 18s

the patch passed

+1 :green_heart:
shadedclient
28m 44s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
39m 26s

hadoop-hdfs-native-client in the patch passed.

+1 :green_heart:
asflicense
0m 33s

The patch does not generate ASF License warnings.

152m 23s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4145/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4145

Optional Tests
dupname asflicense compile cc mvnsite javac unit codespell golang

uname
Linux 2f698c13849e 4.15.0-169-generic #177-Ubuntu SMP Thu Feb 3 10:50:38 UTC 2022 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / af8487991283df5dd5052dc094e181b544bc7409

Default Java
Debian-11.0.14+9-post-Debian-1deb10u1

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4145/1/testReport/

Max. process+thread count
740 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-native-client U: hadoop-hdfs-project/hadoop-hdfs-native-client

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4145/1/console

versions
git=2.20.1 maven=3.6.0

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


problem with atomic.compare_and_swap

Reporting a bug

[x] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/main/CHANGE_LOG).
[x] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

I am trying to use atomic.compare_and_swap with float32-s. I get the following error:
E                   numba.core.errors.TypingError: Failed in cuda mode pipeline (step: nopython frontend)
E                   No implementation of function Function(<class 'numba.cuda.stubs.atomic.compare_and_swap'>) found for signature:
E
E                    >>> compare_and_swap(array(float32, 1d, C), float32, float32)
E
E                   There are 2 candidate implementations:
E                     - Of which 2 did not match due to:
E                     Overload of function 'compare_and_swap': File: numba/cuda/cudadecl.py: Line 431.
E                       With argument(s): '(array(float32, 1d, C), float32, float32)':
E                      No match.
E
E                   During: resolving callee type: Function(<class 'numba.cuda.stubs.atomic.compare_and_swap'>)
E                   During: typing of call at /home/shauncutts/src/factfiber.ai/mlops/ingolstadt/tests/cudautil/test_mathutil.py (153)
E
E
E                   File "tests/cudautil/test_mathutil.py", line 153:
E                   def _dummy_kernel() -> None:
E                       <source elided>
E                       zero = np.float32(0)
E                       cuda.atomic.compare_and_swap(array, zero, zero)  # type: ignore
E                       ^

Source code:
@cuda.jit  # type: ignore
def _dummy_kernel() -> None:
    array = cuda.shared.array(1, dtype=np.float32)  # type: ignore
    zero = np.float32(0)
    cuda.atomic.compare_and_swap(array, zero, zero)  # type: ignore

Thanks for the report - looks like the docs fail to mention that this is only supported for integer types: https://numba.readthedocs.io/en/latest/cuda/intrinsics.html?highlight=compare_and_swap
This mirrors CUDA C / C++: https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#atomiccas
Once I have completed the patch in https://github.com/numba/numba/issues/6026#issuecomment-1273285104, the following workaround would work:
@cuda.jit  # type: ignore
def _dummy_kernel() -> None:
    array = cuda.shared.array(1, dtype=np.float32)  # type: ignore
    zero = np.float32(0).view('i8')
    cuda.atomic.compare_and_swap(array, zero, zero)  # type: ignore

I'm not quite sure whether to create an implementation of atomic CAS for floats - @shaunc did you come across this for implementing a specific use case, or was it just something you noticed incidentally and kindly reported?

From triage meeting discussion: marking as a documentation issue.


Boot from a real physical drive?

I see it is generally possible in QEMU, just wondering if it is also possible in UTM? If yes, how?

In theory you can create a virtual disk referring to a physical device. therefor you would need to install QEMU via home-brew. Said disk should be able to be imported into UTM.
This would need to be tested however. For emu there are plenty of guides online.

Sandboxes apps (such as UTM) can't access physical drives.
You can either make a drive image as @Rastafabisch suggested, or share the device to your VM using qemu-nbd (but you can only access it after booting into an OS). Ping me if you'd like instructions on that.


fix: import wasm module from a higher directory fails in wrangler dev --local for a module format worker

This is a bit nuanced, but filing it while it's fresh.
This works -
cd packages/example-wasm-app
npx wrangler dev worker/module/index.js --local

This doesn't -
cd packages/example-wasm-app/worker/module
npx wrangler dev index.js --local

It gives this error -

I think it's because the actual wasm module is one directory higher, and that generates a so called "non-relative()d" path that the ignores module struggles with.

Annoying development, it now fails for both scenarios, but appears to work fine for the other two service worker examples. I'll dive into this later.

I prposed a fix for this in https://github.com/cloudflare/miniflare/pull/205


[feature request] display space ID for every active monitor

Would it be possible to display a space ID for each monitor?

Would love this as wel!

This would be fantastic if it's possible!

Yes! I have a 3 Monitor desk using 8 spaces (1-8) + 4 spaces (9-12) + 4 spaces (13-16) and one number on all screens is pretty useless for my needs.

Forked the repo and created a working prototype, which can be used.
Check out here
However, because of my non-existent swift skills, I think someone else would be more suitable in making a proper pr.
Here an example for 3 screens.

@alexus37 That looks good, but it also seems like it takes up a lot of room. (Consider the case of @DerWeiseScheuch, where there are 16 spaces.) It might be nice to have an option to expand it like this or not.

Or we could use fractions like ¼ and ⅜

But maybe that's a bit to small

So for me, this is fine to take that much space, since I come from the I3 world I am used to this. Also, in combination with keyboard shortcuts, this is good information to have (for me at least). I guess the right approach would be to allow multiple visualizations, which the user can configure, i.e. full detail vs compact.

Forked the repo and created a working prototype, which can be used. Check out here However, because of my non-existent swift skills, I think someone else would be more suitable in making a proper pr.
Here an example for 3 screens.

Any chance of pre compiling it and adding the binary to Git?

Yes, I can do that next week.

I am currently using 21 Desktop Spaces, spread between my MacBook and an external monitor.
Using fraction symbols to display #/# would limit me to 8 desktops per monitor, not what I would need. Similarly, having all 21 numbers in the Toolbar, just to have two of them highlighted, would be a serious waste of toolbar space, either this display or other toolbar icons would be regulary eclipsed by the current app's menus.
What's wrong with 2, 3, or 4 instances of the current, single number display?

I am using shortcuts eg. opt+ to switch between spaces therefore having more than 10 does not make sense to me. Additionally, I am also using shift+opt+ to move an application from one space to the other, therefore I want to know which spaces are on which screen and which one is visible. This works well for me, however, I don't want to state this is the perfect way.

So, I checked out https://github.com/Jaysce/Spaceman (which is a fork of Which space). It has the features I was looking .

@alexus37

Additionally, I am also using shift+opt+number to move an application from one space to the other

How do you do that?

How do you do that?

@pascalpp This is possible using (for example) BetterTouchTool, which is an exceptionally useful app.

I'm using yabai + skhd! In the config file, you have the power to define a command that moves the current window to another space. For example:
shift + lalt - 1 : yabai -m window --space 1


update apk in docker

we should do this for all the functions ?

we should do this for all the functions ?

Only alpine with node images are affected. Golang images (pure alpine) are not affected.


quilt_small.png is missing

https://github.com/QuiltMC/quilt-native-installer/blob/main/src/main.rs#L7
There is no file named quilt_small.png - but there is quilt.png.

Yep, just lost that during testing and renaming. That file isn't meant to exist, it's fixed now!


fix: aria-expanded toggle with true and false

Now the menu works well with keyboard tab key

Thanks @ChristianMurphy


Github Wrapped

I'm unable to access my Github wrapped

So sorry about that, it should be fixed now. Please try again and let me know if you run into any difficulties!


batch query is unstable

The cciss batch query seems unstable. I’ve tried a few subsets of this dataset, including removing ocean points, but it still crashes (except for points 101-200). @kdaust Wondering if you can troubleshoot this.
cciss_points.csv

Fixed in https://github.com/bcgov/CCISS_ShinyApp/commit/421b4a3f4a9121dffa5a26db5c5492545a46611d


Wrapper for MzidToTsvConverter for MS-GF+ mzid to tsv/csv conversion

A new and fast C# based mzid to tsv and subsequently to csv converter for MS-GF+ results. Was tested with MS-GF+ versions '2016_09_16' and '2017_01_27' and yields same results as the Java based version.
Please use (at least) version '2017/07/04' of the converter!
https://github.com/PNNL-Comp-Mass-Spec/Mzid-To-Tsv-Converter/releases/tag/v1.0_20170704

It is not used by default yet, it has to explicitly set in the params. We first should discuss this in the dev meeting.

Is it correct that the flags '-showDecoy' and '-unroll' are not used?
I guess the default is used then, but I would prefer to ecplicitely print out decoys and unroll the results.

It may be, that setting the flags are not working correctly at the moment. Nevertheless, so far as I tested it, the results are the same for the Java based and the C# based converter. So all should be as we did it before.
Anyways, I will check it again to make sure, it is correct.

Hmm, not sure why, but it doesn't work for me so far:

        -\-     msgfplus_C_mzid2csv_v2017_07_04 run initialized with BSA1_msgfplus_v2016_09_16.mzid (Wed Jul  5 13:37:25 2017)     -/-

[   run    ] Preparing engine
[   run    ] Starting engine
[ Citation ]
[  Please  ]
[  cite:   ] Kim S, Mischerikow N, Bandeira N, Navarro JD, Wich L, Mohammed S, Heck AJ, Pevzner PA. (2010) The Generating Function of CID, ETD, and CID/ETD Pairs of Tandem Mass Spectra: Applications to Database Search.
[  -----   ]
[ Citation ]
[ PREFLGHT ] Executing preflight sequence ...
[ prefligh ] Execution time 0.000 seconds
[ eXecutio ] Executing command list ...
Traceback (most recent call last):
  File ".\simple_example_search.py", line 95, in <module>
    main()
  File ".\simple_example_search.py", line 83, in main
    force      = False
  File "C:\Program Files\Python 3.5\lib\site-packages\ursgal\ucontroller.py", line 1823, in search
    force      = force,
  File "C:\Program Files\Python 3.5\lib\site-packages\ursgal\ucontroller.py", line 344, in convert_results_to_csv
    output_file_name = output_file_name
  File "C:\Program Files\Python 3.5\lib\site-packages\ursgal\ucontroller.py", line 2812, in execute_unode
    force, engine_name, answer
  File "C:\Program Files\Python 3.5\lib\site-packages\ursgal\ucontroller.py", line 2091, in run_unode_if_required
    json_path = json_path,
  File "C:\Program Files\Python 3.5\lib\site-packages\ursgal\unode.py", line 1352, in run
    report['execution'] = self._execute()
  File "C:\Program Files\Python 3.5\lib\site-packages\ursgal\unode.py", line 455, in _execute
    startupinfo=startupinfo,
  File "C:\Program Files\Python 3.5\lib\subprocess.py", line 950, in __init__
    restore_signals, start_new_session)
  File "C:\Program Files\Python 3.5\lib\subprocess.py", line 1220, in _execute_child
    startupinfo)
FileNotFoundError: [WinError 2] Das System kann die angegebene Datei nicht finden

I downloaded the newest zip package from the MSGF+ website and extracted is to the corresponding folder in Ursgal. Initializing the controller also shows me that the converter is available, so I guess it's not the engine itself that cannot be found ...

Sorry, i forgot to make the Mono call optional.  Thus  is not required under Windows.

Ok, right
By the way, according to the printout
unroll results: False
show decoy: False

Would be great if you could fix thoose things ;)
And then the next problem: the spec IDs are not correct (and therefore cannot be mapped to the RT). I think you mentioned this before ... so the version that they have on their site is not the most recent one?

I can not fix this, it is a problem in the converter itself, i think. But as i said, the result are the same as with the Java version. Have you checked that? However,  I will be in contact with the developers to clarify this.
Make sure to use this version of the converter (see above) to avoid wrong spec ids:
https://github.com/PNNL-Comp-Mass-Spec/Mzid-To-Tsv-Converter/releases/tag/v1.0_20170704

I'm sorry, I somehow missed the link the first time you posted it :/
I have tested setting unroll and show decoy to True (i.e. 0) and it shows True in the printout and the results are as expected (unrolled and with decoys when 0), so I would say it works ...
But what I actually meant with fixing it was the mono issue, so making the call optional like for msamanda.
Ok, now regarding checking it if they produce the same results:
well, the results directly after conversion are slightly different, the C version file has more lines (I think because it writes separate lines if a pep maps on different proteins), but after unify_csv the results are the same, so I think it's fine.
However, one more bug with the new version:
        -\-     msgfplus_C_mzid2csv_v2017_07_04 run initialized with StS_Hvo_wt_GluC_SN_Big12_20160929_msgfplus_v2017_01_27.mzid.gz (Thu Jul  6 09:38:44 2017)     -/-

[   run    ] Preparing engine
[   run    ] Starting engine
[ Citation ]
[  Please  ]
[  cite:   ] Kim S, Mischerikow N, Bandeira N, Navarro JD, Wich L, Mohammed S, Heck AJ, Pevzner PA. (2010) The Generating Function of CID, ETD, and CID/ETD Pairs of Tandem Mass Spectra: Applications to Database Search.
[  -----   ]
[ Citation ]
[ PREFLGHT ] Executing preflight sequence ...
[ prefligh ] Execution time 0.000 seconds
[ eXecutio ] Executing command list ...
Using options:
mzid path: "/Users/sschulze/ursgal/example_data/test/msgfplus_v2017_01_27/StS_Hvo_wt_GluC_SN_Big12_20160929_msgfplus_v2017_01_27.mzid.gz"
tsv path: "/Users/sschulze/ursgal/example_data/test/msgfplus_v2017_01_27/StS_Hvo_wt_GluC_SN_Big12_20160929_msgfplus_v2017_01_27.tsv"
unroll results: True
show decoy: True
single result per spectrum: False
Conversion failed: An item with the same key has already been added.
at System.ThrowHelper.ThrowArgumentException (System.ExceptionResource resource) [0x0000b] in <dbb16e0bacdc4a0f87478e401bc29b6c>:0
at System.Collections.Generic.Dictionary`2[TKey,TValue].Insert (TKey key, TValue value, System.Boolean add) [0x0008e] in <dbb16e0bacdc4a0f87478e401bc29b6c>:0
at System.Collections.Generic.Dictionary`2[TKey,TValue].Add (TKey key, TValue value) [0x00000] in <dbb16e0bacdc4a0f87478e401bc29b6c>:0
at PSI_Interface.IdentData.SimpleMZIdentMLReader.ReadPeptideEvidence (System.Xml.XmlReader reader) [0x000d3] in <c35e318121b24d90a706f5888e0514ce>:0
at PSI_Interface.IdentData.SimpleMZIdentMLReader.ReadSequenceCollection (System.Xml.XmlReader reader) [0x00098] in <c35e318121b24d90a706f5888e0514ce>:0
at PSI_Interface.IdentData.SimpleMZIdentMLReader.ReadMzIdentMl (System.Xml.XmlReader reader) [0x001f5] in <c35e318121b24d90a706f5888e0514ce>:0
at PSI_Interface.IdentData.SimpleMZIdentMLReader.Read (System.String path, System.Threading.CancellationToken cancelToken) [0x00079] in <c35e318121b24d90a706f5888e0514ce>:0
at MzidToTsvConverter.MzidToTsvConverter.ConvertToTsv (System.String mzidPath, System.String tsvPath, System.Boolean showDecoy, System.Boolean unrollResults, System.Boolean singleResult) [0x00005] in <aa4e31edbbde478ea61c353a0c4310ee>:0
at MzidToTsvConverter.MzidToTsvConverter.ConvertToTsv (MzidToTsvConverter.ConverterOptions options) [0x0001f] in <aa4e31edbbde478ea61c353a0c4310ee>:0
at MzidToTsvConverter.Program.Main (System.String[] args) [0x00026] in <aa4e31edbbde478ea61c353a0c4310ee>:0
Traceback (most recent call last):
  File "do_it_all_folder_wide_single_validation_semienzyme.py", line 167, in <module>
    target_decoy_database = sys.argv[3],
  File "do_it_all_folder_wide_single_validation_semienzyme.py", line 113, in main
    converted_result = uc.convert_results_to_csv(input_file=search_result)
  File "/Users/sschulze/ursgal/example_data/test/ursgal/ucontroller.py", line 344, in convert_results_to_csv
    output_file_name = output_file_name
  File "/Users/sschulze/ursgal/example_data/test/ursgal/ucontroller.py", line 2812, in execute_unode
    force, engine_name, answer
  File "/Users/sschulze/ursgal/example_data/test/ursgal/ucontroller.py", line 2091, in run_unode_if_required
    json_path = json_path,
  File "/Users/sschulze/ursgal/example_data/test/ursgal/unode.py", line 1313, in run
    report['execution'] = self._execute()
  File "/Users/sschulze/ursgal/example_data/test/ursgal/unode.py", line 485, in _execute
    '''.format( self.engine, os.path.relpath(self.exe), self.execute_return_code)
AssertionError:

msgfplus_C_mzid2csv_v2017_07_04 crashed!

  The executable
    ursgal/resources/platform_independent/arc_independent/msgfplus_C_mzid2csv_v2017_07_04/MzidToTsvConverter.exe
  terminated with Error code 42 .
  Inspect the printouts above for possible causes and verify that all input files are valid.

Since this seems to be an internal error again (and it works with the old version), we cannot do anything about it ... just wanted to mention it ;)

I have adjusted the flags, it should be correct now and works as expected. Thanks for testing Stefan!
I get the same error on some files. I have contacted the MS-GF team for a solution.
The mono issue should be also fixed now.
Cheers,
Johannes


Add UnKerballed Start 1.0.8 for 1.7.X from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of SpinkAkron, to add UnKerballed Start 1.0.8 for 1.7.X to CKAN.
Please direct questions about this pull request to SpinkAkron.
Mod details:
name = /mod/2255/UnKerballed%20Start%201.0.8%20for%201.7.X
author = SpinkAkron
abstract = UKS 1.0.7 combined with the mod config updates from UKS 1.1.0 for compatibility with KSP 1.7.X
license = CC BY-NC-SA 4.0
Homepage =
description =
This is the mod config updates from UKS 1.1.0 combined with UKS 1.0.7 to provide an updated version compatible with KSP 1.7.X

Removed from SpaceDock.


edit: change owner

#36
Change owner Except for CHANGELOG.md, swift file description, and project.pbxproj.

@stakada7
Thanks!


Empty path after page refresh (hash router)

The path seems to be empty after a full page reload when using RouteMode.Hash.
This works as expected when using RouteMode.Path

repro:
[<LitElement("my-app")>]
let MyApp () =
  let _ = LitElement.init (fun cfg -> cfg.useShadowDom <- false)
  let route = Hook.useRouter RouteMode.Hash

  html
    $"""

Hello, World!

    {match route with
     | [] -> html $"
Home
"
     | [ "test" ] -> html $"
Test
"
     | _ -> html $"
Not found
"}
    """

This should now be fixed in v1.0.0.


UI fails to render after last commit.

Issue resolved after re committing the code

Yes, resolved now.


Error: EACCES: permission denied, open '/.yarnrc' when running make bootstrap

When I run make bootstrap I get this error:
nateaune@Nathans-MBP-3 potsie % make bootstrap
Creating potsie_node_run ... done
yarn install v1.22.5
warning Skipping preferred cache folder "/.cache/yarn" because it is not writable.
warning Selected the next writable cache folder in the list, will be "/tmp/.yarn-cache-501".
[1/5] Validating package.json...
[2/5] Resolving packages...
success Already up-to-date.
Done in 2.37s.
Error: EACCES: permission denied, open '/.yarnrc'
ERROR: 1
make: *** [dependencies] Error 1

Julien says this is a bug and we need to define the HOME  environment variable to point to /tmp as it's done in most of our projects (see https://github.com/openfun/richie/blob/master/Makefile#L64 for example)

Yay! We are still working on providing a relevant test dataset for a tutorial. We'll keep you posted.


Possibility to configure include paths and other compiler options

How do you configure such things like include paths, compiler options and so on  for clangd, so that it can find your include files, use proper predefined symbols, required C++ standard, etc?
As an example, YouCompleteMe has a special config file for it.
And, BTW, does vim-clangd log any kind of errors/warning if something goes wrong, e.g. due to a wrong configuration? If so, where could you find those logs?

nice questions. let me explain one by one.
first of all, supporting include paths, compiler options is a quite high-want feature from me in a c++ developer's aspect. however clangd doesn't ship it. if you or someone else want this missing feature as well, please keep an eye on https://github.com/Chilledheart/vim-clangd/issues/4.
for the logs, you can find them at "~/.confiig/clangd/logs/vim-clangd.log".

Note that clangd also supports a compile_flags.txt, check the "Alternatives" Section in the docs.


Application customizer breaks on events pages in communication sites

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
When creating an application customizer that shows a header or footer, the expected behavior should be that it is loaded on every modern page.
Observed Behavior
It appears that on communication sites, when you create, edit, or display an event. The application customizer is not rendered. When opening the browser console, you get the following error:
sp-pages-assembly_en-us_18b206dbf5ea8a9da3221367da049861.js:547

Uncaught TypeError: Cannot read property 'getBoundingClientRect' of undefined
    at e._scrollEventHandler (sp-pages-assembly_en-us_18b206dbf5ea8a9da3221367da049861.js:547)
    at e (sp-pages-assembly_en-us_18b206dbf5ea8a9da3221367da049861.js:312)
    at u (sp-pages-assembly_en-us_18b206dbf5ea8a9da3221367da049861.js:312)
    at e.s (sp-pages-assembly_en-us_18b206dbf5ea8a9da3221367da049861.js:312)

When you refresh the page, the error is gone and your app. custom. appears again.
Here you can see the process of what happens:

Steps to Reproduce
I tried this by following the guide: https://docs.microsoft.com/en-us/sharepoint/dev/spfx/extensions/get-started/using-page-placeholder-with-extensions

@VesaJuvonen @estruyf any status update on resolving this bug? I'm running into this issue as well.

This issue is a duplicate of #1042
I'll close this issue so we can track it on a single issue.


Add Prettier as mandatory check in CI Pipeline

Prettier ensures that the TS code is formatted following the project specifications. It should be added as mandatory check when proposing a new PR.
Items:

[x] Add the check to a new Github check ./node_modules/.bin/prettier --check */**/*.{js,json,css}
[x] Document how to run the prettify over the project npm run prettify
[x] Document how to enable git commit pre-hook

Closing.


WIP Has any role from discord to with and without_role

As per issue #217
This only rewrites the existing decorators to use discords implementation of has_any_role.
Still need to review it on a running environment before I can remove the WIP with confidence

This isn't the approach I was envisioning. My idea was far simpler - to just import has_any_role from discord and literally call it. I'm not sure why we would copypaste their approach.
This only affects the decorators and should not be in conflict with the django rewrite, so there's no reason to close it.. but please justify your approach.

Hello @tagptroll1, any updates on this?

No follow-up since February, I'm closing this.


Is it possible to create generic perimeter

Is it possible to create generic perimeter like this
export default new BasePerimeter({
  govern: {
    'can': function (action) {
        return this.permissions.some(a => a.name === action);
    }
  },
});

Hey @mirhat,
this is unfortunately not possible. You have to explicitly specify what user can do:
export default new BasePerimeter({
  can: {
    create() {
      return this.permissions.some(a => a.name === action);
    }
  },
});

What would be your use-case for this?
You can still do something like:
export default new BasePerimeter({
  can: {
    proceed(action) {
      return this.permissions.some(a => a.name === action);
    }
  },
});

or possibly also something like (not tested though):
const listOfAvailablePermissions = ['permission1', 'permission1', '...'];

export default new BasePerimeter({
  can: {
    ...listOfAvailablePermissions.reduce(name => (function() {
        return this.permissions.some(a => a.name === action);
      })
    }, {})
  },
});

Does that answer you question?

Thank you, first solution is good enough.
export default new BasePerimeter({
  can: {
    proceed(action) {
      return this.permissions.some(a => a.name === action);
    }
  },
});


refactor(examples): consistent CIFAR-10 shapes

Loosely motivated and kicked off by https://elixirforum.com/t/help-figuring-out-reshape-operation-has-mismatched-element-counts-from-0-f32-32-0-8-64-to-16384-f32-32-512/58572.
Isn't an actual transpose make more sense here? Or am I missing something?
UPDATE: In fact this also gives me a jump in accuracy from 59-60% to 67-68%, same for others?

Thanks!


[bug] oc client missing on microshift arm bundle version 4.12.9

After starting the cluster when trying to setup the oc client we got
ec2-user@Mac-mini bin % eval $(crc oc-env)
ec2-user@Mac-mini bin % which oc
oc not found

Checking the clients folder oc client seems missing:
ec2-user@Mac-mini bin % ls -al /Users/ec2-user/.crc/bin/oc/
total 0
drwxr-x---  3 ec2-user  staff   96 Apr  5 08:24 .
drwxr-x---  6 ec2-user  staff  192 Apr  5 08:24 ..
lrwxr-xr-x  1 ec2-user  staff   67 Apr  5 08:24 podman -> /Users/ec2-user/.crc/cache/crc_microshift_vfkit_4.12.9_arm64/podman

Logs from crc setup does not show any issue:
ec2-user@Mac-mini ~ % crc setup
CRC is constantly improving and we would like to know more about usage (more details at https://developers.redhat.com/article/tool-data-collection)
Your preference can be changed manually if desired using 'crc config set consent-telemetry <yes/no>'
Would you like to contribute anonymous usage statistics? [y/N]: n
No worry, you can still enable telemetry manually with the command 'crc config set consent-telemetry yes'.
INFO Using bundle path /Users/ec2-user/.crc/cache/crc_microshift_vfkit_4.12.9_arm64.crcbundle
INFO Checking if running as non-root
INFO Checking if crc-admin-helper executable is cached
INFO Caching crc-admin-helper executable
INFO Using root access: Changing ownership of /Users/ec2-user/.crc/bin/crc-admin-helper-darwin
INFO Using root access: Setting suid for /Users/ec2-user/.crc/bin/crc-admin-helper-darwin
INFO Checking if running on a supported CPU architecture
INFO Checking minimum RAM requirements
INFO Checking if crc executable symlink exists
INFO Creating symlink for crc executable
INFO Checking if running emulated on a M1 CPU
INFO Checking if vfkit is installed
INFO Setting up virtualization with vfkit
INFO Checking if CRC bundle is extracted in '$HOME/.crc'
INFO Checking if /Users/ec2-user/.crc/cache/crc_microshift_vfkit_4.12.9_arm64.crcbundle exists
INFO Getting bundle for the CRC executable
INFO Downloading bundle: /Users/ec2-user/.crc/cache/crc_microshift_vfkit_4.12.9_arm64.crcbundle...
1.17 GiB / 1.17 GiB [-----------------------------------------------------------------------------------------------------------------------------------------------------------------------] 100.00% 11.35 MiB p/s
INFO Uncompressing /Users/ec2-user/.crc/cache/crc_microshift_vfkit_4.12.9_arm64.crcbundle
crc.img:  31.00 GiB / 31.00 GiB [-------------------------------------------------------------------------------------------------------------------------------------------------------------------------] 100.00%
INFO Checking if old launchd config for tray and/or daemon exists
INFO Checking if crc daemon plist file is present and loaded
INFO Adding crc daemon plist file and loading it
Your system is correctly setup for using CRC. Use 'crc start' to start the instance

Closed in favor of https://github.com/crc-org/crc/issues/3581


Update docs?

I just came across this typo in both version 5 & 6 docs (the word is):

it is strongly recommended is to use Python 2

http://click.pocoo.org/5/python3/#python-3-support
http://click.pocoo.org/6/python3/#python-3-support

But looking at the click repo itself, I noticed that this typo was already fixed by @jmorganc in #566 on Apr 27, 2016. It seems like the docs on click.pocoo.org are out of date?

existing docs don't get rebuilt

@davidism ,
Ok, it just seemed odd to merge #566 almost two years ago but never update the actual docs.


[Fix] version detection: support processor virtual filename

close #2948
cc @ljharb

What I've been playing with is:
diff --git a/lib/util/version.js b/lib/util/version.js
index c34b7c55..7f99dd96 100644
--- a/lib/util/version.js
+++ b/lib/util/version.js
@@ -7,6 +7,7 @@

 const resolve = require('resolve');
 const path = require('path');
+const fs = require('fs');
 const error = require('./error');

 let warnedForMissingVersion = false;
@@ -27,8 +28,20 @@ function detectReactVersion(context) {
     return cachedDetectedReactVersion;
   }

+  let basedir = process.cwd();
+  if (context) {
+    const filename = context.getFilename();
+    const fileDir = path.dirname(filename);
+    try {
+      const isFile = fs.statSync(filename).isFile();
+      const fileDirIsDir = fs.statSync(fileDir).isDirectory();
+      if (fileDirIsDir && isFile) {
+        basedir = fileDir;
+      }
+    } catch (e) { /**/ }
+  }
   try {
-    const reactPath = resolve.sync('react', {basedir: context ? path.dirname(context.getFilename()) : process.cwd()});
+    const reactPath = resolve.sync('react', {basedir});
     const react = require(reactPath); // eslint-disable-line global-require, import/no-dynamic-require
     cachedDetectedReactVersion = react.version;
     return cachedDetectedReactVersion;

This a flow version test too, so an extra util function is preferred.

Indeed; my implementation isn't ideal, i just meant the overall approach of using fs methods.

@ljharb Done!

@ljharb Although I disagree with your opinion, but these changes are just simple, so Done!

@ljharb Is there a schedule to release a patch version? I think it is a serious issue because the original version breaks.

I've just published v7.23.1.

Thanks, although the CHANGELOG seems incorrect.

That's quite correct. There's nothing unreleased at the moment.

I mean [7.23.1], see #2950

ah yes, thanks, a copypaste error.


Unclear how the command az synapse linked-service create works with a plain json file in a .azcli file in visual studio code

Related command
#az synapse linked-service create --workspace-name $synapseWorkspaceName --name "$synapseWorkspaceName-WorkspaceDefaultStorage" --file ????
Is your feature request related to a problem? Please describe.
Would be nicer to have a JSON example file is not working. On the internet no example. So I guess nobody is using it because it is too too complicated for incidental use.
Describe the solution you'd like
Simpler syntax or better example from a .azcli file where the json is in and the parameters can be modified on the fly so that 1 file holds
Describe alternatives you've considered
add manually ....:-)

Thank you for opening this issue, we will look into it.


Make -e compatible with -R to execute customized commands

Is your feature request related to a problem? Please describe.
No, it's just a feature that I think more people will like.
Describe the solution you'd like
I want to be able to combine a RAG with the shell execution feature. This way I can ask aichat to make shell commands tailored to my needs every time and avoid writing the instructions every time I want to use it.
Describe alternatives you've considered
From my understanding, the only change required is that we allow the -e argument be compatible with the RAG -R argument. I've used the debug function, created my own %shell% prompt and I got it to do what I want, it outputted just the command following my preferences. In this example I want the find operations use the fdfind inseat of the unix standard find. Unfortunately it just output the command, not integrated with the very nice interface the -e command provides.
Additional context
I think this change may enable more people to do more creative things with their operating systems. In my case as I have ADHD I make decisions about tools or ways to do things that I later forget. I've started using a "wisdom" folder with .txt files containing such decisions, and I've made aichat to use it as a RAG. Now, if I just manage to not only know but also format and use the commands I choose in the past, that will be perfect.
I've implemented the thing on my own using shell scripting, but any change is very painful. When I got acquainted with aichat I understood that it is a better base tool to build it. Unfortunately I don't know Rust and I'm far from learning it.

we will not support this feaure.
It is recommended to achieve tailored hell commands through the following two strategies.

Use your own %shell%. You can create a shell named %shell% in roles.yaml, and Aichat will use it with -e (override the built-in %shell% role).

Create an agent with RAG and the execute_command tool.


Version 5.5.28 doesn't work on Windows 7 32 bits

Hi!
here is my bug:
NuGetPackageExplorer.exe version <IP_ADDRESS>90 doesn't start on Windows 7 X86.
Type:

Bug

NPE version: 5.5.28
OS version: (Windows 7 - Microsoft Windows [version 6.1.7601] )
PROCESSOR_ARCHITECTURE=x86
PROCESSOR_IDENTIFIER=x86 Family 6 Model 15 Stepping 11, GenuineIntel
PROCESSOR_LEVEL=6
PROCESSOR_REVISION=0f0b
Installed from: Zip on GitHub
In case of a BUG:

What is the current result? version not compatible with your Windows system (X86)
What is the expected result? NuGetPackageExplorer opens ok
Are there any workarounds? no
Is there a version in which it did work? NuGetPackageExplorer-v5.0.230
Is there a package on Nuget.org which shows the bug? no

I have tested version 5.5.28 on a Windows 7 64 bits and it works fine

Closing as any Win7 support is incidental. Windows 7 is no longer supported by Microsoft.


Code Error Window on Startup

For some reason all of the sudden, I open Open Note Block Studio and It stops responding for a little bit so I'm over here thinking "hmmm... that's strange. It's probably just loading." A few seconds later a window pops up caller "Code Error" and so then I re installed it and it still pops up.

How can I fix this?
If you cant see it the error reads:

############################################################################################
ERROR in
action number 1
of Create Event
for object obj_controller:
Trying to draw non-existing sprite.
at gml_Script_wallpaper_init
############################################################################################
gml_Script_wallpaper_init (line -1)
gml_Script_change_theme
gml_Script_control_create
gml_Object_obj_controller_Create_0 (line -1)

I got 3.10 beta instead of the latest 3.9 and it fixed the issue.


Handle default image in templates instead of models

the default image for Category and Food model has been set in the model. Instead set the ImageField nullable and in the templates, check if the model has an image to show, else the default image has to be shown via static function

completed in d5acb19d59236d19852895ac2847802e288739d8


Upgrading the UBI version to 8.5

@AdamBrousseau Can you please review this change and merge it. Thanks !!

@AdamBrousseau Can you please merge this PR. Thanks !!


Update link location

Referenced file in the dotnet/standard repo was moved from docs to docs/planning. The #anchorTarget has survived.

This project is now archived.


Added a navbar template

Disclaimer
By sending us your contributions, you are agreeing that your contribution is made subject to the terms of our Contributor Ownership Statement

Thanks for the contribution @NandiyaLive ✨


Add clear button to filter fields #1673

Added a little script to make this post the search form automatically when the clear button is clicked.
I've also changed some htmx-triggers in the htmx-driven forms where search fields were used. They acted on the keyup event which I've changed to the input event since that also covers pasting text (docs).
FIXES #1673

It also submits the form in this situation:

the search input is empty
you type something
you clear it by clicking the clear icon

in this case there is no reason to submit anything as the input value hasn't changed

Chrome: works
Firefox: no x is shown
Safari: x is shown but clearing action does not work

Not all browsers behave the same, so it's not a big deal as this was an enhancement:
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/search#differences_between_search_and_text_types (open in chrome to test it on this website).
If we want this to behave the same in every browser, we will have to extend the native functionality.

@mietcls Implemented a fix that kinda works everywhere. It requires keeping track of the previous value and only submitting if that was not empty. That means the submit also immediately fires if you delete the last character in the field (with delete or backspace) which may not be what you want.
May be better to ask Emily to design a component for this with a button that we can simply put a click event on.

https://github.com/ugent-library/biblio-backoffice/pull/1687#pullrequestreview-2237504404
Fixed with b576b24


(NOTE: models are broken on Windows, unfortunately!)

We fixed that bug in translatelocally.

true - comment deleted and link added ...


Bug bash documentation on other languages

Take a day to review doc of another language :). Java of the service you know, or whatever

Done


Export path

Hi again!
In method "LibHaru_Save_File" Export_Path is validating if this var is not empty but path can be not valid like there is no folder in this path. This method returns true as PDF was created but if folder doesnt exist pdf will not be created.
There is a way to validate path throw UE btw, but i think it will be great to add path validation into method "LibHaru_Save_File"

Bool return value is not fair in this case. It return True even PDF is not created because of invalid path

No, I am agree with return value. I will change it. I say I just won't use unreal's api to check folders. Return value will be bool + string error error code based on HPDF_RESULT. If path is not avaible, it will return false + "4119"
I will look at how can I give a more meaningful outcode than integer based codes.

I agree with you in this case but in my case i should create path by myself so i will use unreal for this purpose. Thank you for fast answer

okay done and pushed.
If there is an error (not just paths but anything), it won't return true and there is an OutCode string output to say you what's going on.
This will generate you a hexademical value and you can check it from here.
https://github.com/libharu/libharu/wiki/Error-handling


adding OMG XSD files License info

FYI) In https://github.com/apache/incubator-kie-kogito-runtimes/pull/3714 , @pjfanning suggested to file a LEGAL JIRA to confirm if OMG license is safe to use. Filed this : https://issues.apache.org/jira/browse/LEGAL-690

rat check would pass with the change.
$ git diff
diff --git a/.rat-excludes b/.rat-excludes
index f7bad0bc28..5355198b63 100644
--- a/.rat-excludes
+++ b/.rat-excludes
@@ -21,7 +21,8 @@ TestingEmptyFile\.txt
 .*\.pmml_json
 .*\.drl_json
 .*\.dslr
-DMN.*\.xsd
+.*DMN.*\.xsd
+dmn.xsd
 DC\.xsd
 DI\.xsd
 KiePMML.*\.txt


promise.timeout() doesn't cancel the original promise after the timeout

With bluebird 2.x cancellable promises were being cancelled by timeout() however in 3.x that doen't happen even if cancellation is (globally) enabled. Is this intentional?

I hope not.

To reproduce:
var Promise = require('bluebird')
bb2/cancel-bug.js
var Promise = require('bluebird')
var assert = require('assert')

function assertAfter(t, str) {
  return Promise.delay(t)
    .cancellable()
    .then(_ => console.error(str))
}

var p1 = assertAfter(1000, "Not cancelled");
var p2 = p1.timeout(500).catch(_ => _)

vs
bb3/bug.js
var Promise = require('bluebird')
var assert = require('assert')

Promise.config({cancellation: true})

function assertAfter(t, str) {
  return Promise.delay(t)
    .then(_ => console.error(str))
}

var p1 = assertAfter(1000, "Not cancelled");
var p2 = p1.timeout(500).catch(_ => _)

node bb2/cancel-bug has no output, while node bb3/cancel-bug prints:
Not cancelled

Cancellation is not rejection anymore, so how would it even work?

@petkaantonov: cancelling p1 should prevent the _ => console.error(…) callback from occurring and cancel the Promise.delay promise, shouldn't it?

what does that have to do with timeout whose purpose is to reject with a cancellationerror?

@petkaantonov What I meant is that .timeout() used to not only reject things, but called .cancel() on the promise, resulting with rejections propagating upstream. Not sure if the current .timeout() does that anymore, but I'll look into it and maybe send a PR of what I expect the behaviour to be.

This is the PR above. Don't know if it makes sense to do it now as it may be considered a breaking change. Alternatively it may be considered a bugfix :grinning:

Example use case (assumes cancellation enabled)
function whilist(condition, body, value = undefined) {
  if (condition()) body().then(val => whilist(condition, body, val))
  else return Promise.resolve();
}

var waitButton = whilist(_ => !$('.button').length, _ => Promise.delay(100))
waitButton.timeout(10000, "Page failed to load"); // should end `whilist` loop

Is this in conflict with the new cancellation semantics?

No, I'd say it's totally in line with the new cancellation semantics. One might even consider cancelling with the TimeoutError instead of a CancellationError.
The only thing I'm not so sure about is whether parent.cancel() (in the PR) is enough. Conceptionally, you'd have to unsubscribe the handlers on parent (which were supposed to resolve the timeout promise that is now rejected) to make the cancellation work.

No, I'd say it's totally in line with the new cancellation semantics. One might even consider cancelling with the TimeoutError instead of a CancellationError.

You cannot cancel with anything, cancellation is not rejection anymore.

@petkaantonov you can reject the derived promise but also cancel the parent one. If cancelling the parent is not a good idea due to other promises being potentially derived from it, e.g.
This should not cancel p2 and p3 or prevent the logging (multiple derived promises):
var Promise = require('bluebird');
Promise.config({
  cancellation: true
});

var p1 = Promise.delay(100)
  .then(_ => console.log("p1"))

var p2 = p1.delay(200).then(_ => console.log("p2"))
var p3 = p1.delay(300).then(_ => console.log("p3"))

p1.timeout(50)

This should cancel the console.log and reject both p2 and p3
var Promise = require('bluebird');
Promise.config({
  cancellation: true
});

var p1 = Promise.delay(100)
  .then(_ => console.log("p1")).timeout(50)

var p2 = p1.delay(200).then(_ => console.log("p2"))
var p3 = p1.delay(300).then(_ => console.log("p3"))

then creating a parent/child pair within timeout then cancelling the parent but returning the child should fix that

@petkaantonov

You cannot cancel with anything, cancellation is not rejection anymore.

Well, it is to those callbacks that still observe cancellation, isn't it? I mean things like onCancel handlers, finally, and late subscribers. And I would think it's ok if those see a TimeoutError instead of a CancellationError.
@spion

If cancelling the parent is not a good idea due to other promises being potentially derived from it

That's fine, cancel() should automatically handle that.

@bergus it doesn't because the original promise gets cancelled, and therefore so will the other derived promises. Which is why I suggested creating a parent and a child (or rather, a child and a grand-child), cancelling the child and rejecting the grandchild.

Updated PR to address last point.

Ah, I see, that's what parent = this.then() is good for.


Would traits better replace the static class?

I am assuming that the Middleware class is basically a collection of functions, which are collected into a class instance?
$request = Middleware::setAttribute($request, self::KEY, $this->basePath);

// better as

use RequestAttributesHelperTrait;

$request = $this->setRequestAttribute($request, self::KEY, $this->basePath);

Using this as an example, it would be (IMO) better to have a set of traits to allow this functionality to be implanted into middlewares, rather than relying on a static class. Obviously backwards compatibility is an issue, so the static class can use the traits and still implement it's own methods.
When it comes to the factory method, I think an abstract factory class would be more apt, which can also have a helper trait to allow injecting itself into middleware instances that require it.
What do you think about an architectural change like this? Being that it is such a large change, it could be argued into a version release, but having a good backwards compatibility, it could be a feature release. I don't mind working on it, as I really like this codebase and want to use it, but I just can't bring myself to allow statics into my codebase :D

If I understand it correctly, you want to move the methods related with attributes to a trait. I think it can be fine, but there's a global constant used by these methods (Middleware::KEY) that should be defined anywhere and it's not possible to define constants in traits.
And about the factory method, can you write some example code?

I never thought about the Middleware::KEY :(

As you can see, the purpose of this constant is to define the key used to store the array that contains all attributes, in order to avoid conflicts with other middleware pieces.
Because it's a constant, that cannot be modified, two possible solutions come to my mind:

Use the value directly, instead a constant. For example:
$request->getAttribute('Psr7Middlewares\\Middleware');

Use a php magic constant with the same value always, for example __NAMESPACE__:
$request->getAttribute(__NAMESPACE__);

Of course, another solution is leave this as is :)

I agree with the idea of making smaller middleware block, can be very useful in more complex filters. Maybe move this constant into separate class? Like Middleware::IP_ADDRESS it will outer dependency but traits from the same package.

The idea I have is to lead to being able to allow third party middleware to be used by the same system, and extend your ideas by using the traits I'm on about; It wouldn't just be an attribute trait, but the other parts as well. Because of this the __NAMESPACE__ would probably be a bad idea. A private property perhaps?
The factory I had in mind would be a class containing an array of name => namespaced classdefinitions that would be used to create the middlewares from.. This would allow others to add their own to the factory too.
It's all just an idea, and I don't mind if it's rejected. I always feel uneasy about static calls though, that's all.

A private property is another option. It can be changeable by the middleware but that can be a feature (allow to create different namespaces).
If you feel strong with this idea, you can create a pull request and continue the discussion with some code in sight :)

Hi. I've implemented this in the new version (3.14)
There's the AttributeTrait to save attributes. It continue to rely on the static class due the constant KEY but, at least it is easy to create new middleware importing only traits and not other static things.
There's also the method Middleware::registerNamespace() to add other vendor namespaces to the factory.

Thanks! Sorry I didn't get any time to have a try at this. Clients have taken over my life recently.

No worries :)


Create instances by Laravel service container (using app()->make instead of new $className).

Describe the bug
Impossible to inject dependencies by constructor for classes inherited from Orchid classes (for example child class for ListLayout) .
Too few arguments to function App\Orchid\Layouts\Product\ProductListLayout::__construct(), 0 passed in /usr/share/nginx/html/vendor/orchid/platform/src/Screen/Layouts/Base.php on line 141 and exactly 1 expected (View: /usr/share/nginx/html/vendor/orchid/platform/resources/views/layouts/base.blade.php)

To Reproduce
Steps to reproduce the behavior:

Create class like this:

class ProductListLayout extends ListLayout
{
    const DATA_KEY = 'products';

    public function __construct(RouteNameProvider $routeNameProvider)
    {
        parent::__construct($routeNameProvider);
    }
}

Open page that used this layout.

Expected behavior
Page opened without errors. Constructor dependencies injected to class.
Additional context
In my case problem in https://github.com/orchidsoftware/platform/blob/master/src/Screen/Layouts/Base.php#L152
 $layout = ! is_object($layout) ? new $layout() : $layout;

Hi @likemusic
I think this is not a bug. But I fully support the idea of doing this!

@tabuna I love your so quick responses for any issues!
What about usage new in other places?

Can you help and make a list where this is really needed?

Constructor injection may be needed in any class, i think.

OK.  in that case I will see what can be done

I added creation for filters and for a relationship class.
Feel free to create a new issue if necessary.


Publish to npm

Would love to be able to npm install flowy to use with bundlers and unpkg.com
https://zellwk.com/blog/publish-to-npm/

@alyssaxuu lmk, I can work on this if you want.

Would be also good to have src and dist folders and some CI setup.
Could you need some asistance with this?

#37 is ready to merge to get this in place

@jerivas https://www.npmjs.com/package/flowy-engine has been published. v2 is in the works as a new NPM release in the future.

@joelvh
Hello, I did this:
npm i flowy-engine
but no luck

@AsimNet please check the error, your Node version and that you import the module, not the main entry. See also<EMAIL_ADDRESS>

could you please give me an example?
@DanielRuf

@AsimNet try import * as flowy from 'flowy-engine/flowy-engine.mjs.

same thing
ERROR in libs/author/src/lib/prototype/editor.component.ts:8:24 - error TS2307: Cannot find module 'flowy-engine/flowy-engine.mjs'.
@DanielRuf

https://github.com/joelvh/flowy-engine/blob/master/src/index.js

Did you install it? Please check the content of your node_modules/flowy-engine. The source repo has nothing to do with the code on npmjs / unpkg (where you can see what you should have in your node_modules).

same thing

You didn't specify the exact error before, you provided only a screenshot of a text with a red line under it which is not that helpful.
Please check your node_modules folder to see which files can be imported / required.

It works for me:
➜  flowy-test npm i flowy-engine
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.

+<EMAIL_ADDRESS>added 1 package from 3 contributors and audited 1 package in 2.019s
found 0 vulnerabilities

➜  flowy-test code .
➜  flowy-test cat index.mjs
import * as flowy from 'flowy-engine/flowy-engine.mjs';

console.log(flowy);%
➜  flowy-test node --version
v10.16.3
➜  flowy-test nodenv versions
  system
  6.17.1
  8.16.0
  8.17.0
  9.11.2
  10.16.0
* 10.16.3 (set by /Users/danielruf/.nodenv/version)
  12.13.0
  13.5.0
➜  flowy-test nodenv local 13.5.0
➜  flowy-test node index.mjs
(node:7408) ExperimentalWarning: The ESM module loader is experimental.
[Module] { default: [Function: e] }

Is the https://github.com/joelvh/flowy-engine "official" fork of npm feature?
@joelvh do you have any examples of the usage?

Yes, it is linked at https://www.npmjs.com/package/flowy-engine

@vicmosin it's outdated and I decided not to pursue the work I started back then because there wasn't a desire by the maintainer to bring my updates into the project. I'm effectively waiting to see how much more the project gets updated, and if there's a desire in the future to use more tooling and accept contributions such as ES6 and so on.
However, it is a usable version from that point in time. The README has directions to build and run the demo. And there's the demo build in the repository as well.
For what it's worth, there's a React implementation as well that works in its own branch. And I was ambitious to refactor the whole thing.


fix the input type deserialization issue

In autorest.csharp we do not have a InputNamespace converter - but we have one here, and we are asserting the auth to never be null, but it is not true, some project does not define their auth.

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


eliminate_flags changes name/package unexpectedly

In the released version of pyfoma, one creates a flag-eliminated fst with:
from pyfoma.eliminate_flags import eliminate_flags
efst = eliminate_flags(some_other_fst)

This has been changed to something more logical:
from pyfoma.algorithms import eliminate_flags
efst = eliminate_flags(some_other_fst)

Unfortunately, the old function has also changed its name, so the original code will break.  If there is no urgent reason to change pyfoma.eliminate_flags.eliminate_flags to pyfoma.eliminate_flags.eliminate_fst_flags then I suggest changing it back so older code will still work.

If you are concerned about namespace pollution in pyfoma.algorithms (since it does magical things) you can always just:
- from pyfoma.eliminate_flags import eliminate_fst_flags
+ from pyfoma.eliminate_flags import eliminate_flags as eliminate_fst_flags


Nested anonymous classes variable assignment fails code generation

Consider the following example code:
	// a basic abstract class for nesting example
	public static abstract class BasicAbstract {
		public abstract void doSomething();
	}

	public static class TestCls {
		private BasicAbstract inner;

		public void simpleAnonymous() {
            inner = new BasicAbstract() {
                @Override
                public void doSomething() {
                    inner = null;
                }
            };
		}
	}

Code gets generated correctly here. Now let's add another level of anonymous class:
	// a basic abstract class for nesting example
	public static abstract class BasicAbstract {
		public abstract void doSomething();
	}

	public static class TestCls {
		private BasicAbstract outer, inner;

		public void nestedAnonymous() {
			outer = new BasicAbstract() {
				@Override
				public void doSomething() {
					inner = new BasicAbstract() {
						@Override
						public void doSomething() {
							inner = null;
						}
					};
				}
			};
		}
	}

Code generation fails. You can see that inner = null doesn't get generated correctly. Instead, it gets a whole new copy of inner = new BasicAbstract() ... with a lot of recursion of the same.
As far as I can tell, this is a regression from version 0.6.0, where anonymous classes were never transplanted into their original place, but instead left as external class AnonymousClassN { ... } definitions.

The error was in synthetic method inliner.
Decompiled code with disabled MethodInlinerVisitor:
public class TestCls {
    private BasicAbstract inner;
    private BasicAbstract outer;

    public static abstract class BasicAbstract {
        public abstract void doSomething();
    }

    static /* synthetic */ BasicAbstract access$002(TestCls x0, BasicAbstract x1) {
        x0.inner = x1;
        return x1;
    }

    public void test() {
        this.outer = new BasicAbstract() {
            public void doSomething() {
                TestCls.access$002(TestCls.this, new BasicAbstract() {
                    public void doSomething() {
                        TestCls.access$002(TestCls.this, null);
                    }
                });
            }
        };
    }
}

And both calls of access$002 was incorrectly replaced with first call argument:
 TestCls.this.inner = new BasicAbstract() { ...

Once again, thank you for a blazing fast fix, it does indeed generate correct code for the sample above.
However, after this fix I have a problem, which I can't seem to isolate into a small test case. Take a look at https://github.com/danielesegato/android-support-library-archive/blob/master/v4/android-support-v4.jar and the decompile of android.support.v4.view.ViewPager class. The class has multiple constructors with different arguments with the constructor body largely repeated. The first constructor has the following statement:
        this.mEndScrollRunnable = new Runnable() {
            public void run() {
                ViewPager.this.setScrollState(ViewPager.SCROLL_STATE_IDLE);
                ViewPager.this.populate();
            }
        };

while all following constructors have the following:
        this.mEndScrollRunnable = /* anonymous class already generated */;

Okay, actually, the source for that class (https://github.com/danielesegato/android-support-library-archive/blob/master/v4/src/java/android/support/v4/view/ViewPager.java) shed light on what was going on.
Unlike the decompile, the original source initializes mEndScrollRunnable at declaration time, not in the constructor, which triggers the problem. Here is a test:
	public static class TestCls {

		public interface BasicAbstract {
			void doSomething();
		}

		private BasicAbstract x = new BasicAbstract() {
			@Override
			public void doSomething() {
				y = 1;
			}
		};
		private int y = 0;

		public TestCls() {
		}

		public TestCls(int z) {
		}
	}


Fix onAuditLog

Fixes #688
When it was changed so that created is parsed to a date in getAuditLog, it broke onAuditLog.
We need to reset the millisecond count to 0 because Roblox does something fascinating... and varies the value on each request.

works for me


Benchmark and Activation Functions

A collection of benchmark and activation functions.

Let's review this based on the latest PR that was merged.


[RaisedButton] Rounded corners are not rounded.

Hello. I encountered the following problem. Rounded corners on rised button do not appear rounded.

How can I fix this?

@olegshilov What browser are you seeing this in?

@jkruder

Chrome 43.0.2357.132 (64-bit)
Firefox 39.0
Safari 8.0.7 (10600.7.12)

@olegshilov It should be working...you're using the latest versions of the browsers. Are you applying any custom styling/overriding the theme?

@jkruder yes, I use a custom theme and override styles with style={{width: '100%'}}

Can you try your app without the style={width: '100%'}}? I just replicated on the demo site and it's very difficult to tell that there are rounded corners when the width is set to 100%. If you see rounded corners without the width we know that there is no issue with the stock buttons.

@jkruder I have another button without 100% width and it look the same.

The button is definitely rounded. The border-radius is only at 2px. I think it's just difficult to see the radius with the dark colors. If you zoom in on the image you just sent you can see the border radius.

@olegshilov How are you setting the button's background color?

@hai-cea Through a custom theme

Here's some of my research:
Your Colors
Retina

Non-Retina

Same button colors - Different background color
Retina

Non-Retina

I did notice that the buttons look fine on the theme customization page:
On Theme customization page
Retina

Non-Retina

After some digging I traced it back to a background color of white being set in the default theme:
https://github.com/callemall/material-ui/blob/master/src/styles/themes/light-theme.js#L98
I'll do a PR to fix.

@hai-cea Thanks.