Error during redisplay: (eval (funcall elfeed-search-header-function)) signaled (no-catch --cl-block-nil-- "239/239:53")

I'm getting this error when hitting RET in the feed list: instead of showing me the entry, nothing happens (except this error).
I problem a few days ago (I didn't record details) which stopped Elfeed from working at all. I guessed was caused by a corrupted ~/.elfeed/index, which I deleted, and then it did run ... but now I can't read any entries.
This is with Emacs compiled from source today, and the current Elfeed.

What's the value of elfeed-search-header-function (a variable)? That's a
strange error message. The thrown string looks like a piece of the
search buffer header, but that shouldn't be thrown. I'm guessing, you've
got a strange function set for the search header function.

Hmm ... its value is "elfeed-search--header."  Maybe it's connected to ivy/counsel/swiper, which I use, since ivy adds its own search functions. They all got upgraded to a new version, and maybe something in one of them fixed something. Any road up, it's back to normal, so I'll close this.
Thanks for all the work on Elfeed.


crud not working why?

in config file esay_panel
'actions' => [
        'product'
    ],

php artisan panel:crud product
There is no product in config file

The problem was resolved by executing this command:
php artisan optimize: clear

Could you please add some attachments?

I had the same issue
it is just an empty project with laravel 8 and I'm trying the package but I find this error, should I do something to register this config, or why does this error appears

it worked, it happens because the actions in easypanel config is empty
I should put the model name to be registered
// config/easy_panel.php
// enabled actions, If you want to create crud you must put action name here
    'actions' => ['post'],

@baselrabia
Yes, this error appears when you don't put crud file name in the config file.


Docs.MS CI Updates for Azure.Core.Amqp

CI Update for release of Azure.Core.Amqp

Docs Build status updates of commit d61eec8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

bundlepackages/azure-dotnet-preview.csv
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 1e2ac92:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

bundlepackages/azure-dotnet-preview.csv
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[WIP] Introduce --parser command-line argument to enable wbnf parsing

Proposal:
Introduce --parser command-line argument to enable wbnf parsing
To simplify the testing and transition to wbnf, the --parser flag can
switch between the existing Antlr parser and the under-development
wbnf parser.
Usage:
sysl validate example.sysl --parser wbnf
@anz-bank/sysl-developers

Right now there is no wbnf parser, so adding the scaffolding to switch the runtime parser seems to be jumping the gun a bit, and then, when there is a wbnf parser we would almost certainly want to simply switch the parser out without the users knowing anything changed, so we would want to then undo this sort of change.

With a perfect transition, yes. I'm foreseeing a desire to start consuming the wbnf parser before it's at feature parity. It may never get to feature parity. My assumption is also that there will be a desire to invoke the Antlr parser once the switch has been made to compare against unexpected quirks in existing behaviour.

The PR That I raised on the other hand allows us to build out the parser and get live feedback (via the textpb diff) comparing the antrl and wbnf parser outputs. I think concentrating on the wbnf parser implementation seperate from the existing sysl codebase is a much safer direction.

Comparing parser outputs is a good approach. This change doesn't preclude this. This change provides an integration point for early integration. Other than exposing a command-line argument that may be deprecated in the future, I see nothing unsafe about this approach.

Stale.


fix(karma-webpack): don't include the file extension (output.filename)

This PR contains a bug fix
https://github.com/webpack-contrib/karma-webpack/issues/322

@Havunen Please sign the CLA

The appveyor CI run failed due to the NPM DNS issue which should now be resolved. Could someone retrigger? :-)

How does this handle entry files with extensions other than .js? .ts, .coffee etc.

Yeah, its' very weird why [name].js doesn't work with karma-webpack. I was even under the impression the extension needs to present in config.output.filename... Maybe this is a bug somewhere else in the code


[Question] Is there an effort to build something similar to react native with preact?

Hello, It has recently come to my attention that react and other Facebook driven libraries have licenses that can become problematic in many projects https://github.com/facebook/react/issues/10191.
In this thread I learned about preact, and it looks like a near perfect drop in replacement for react on the web, however I couldn't help but notice that there doesn't appear to be native effort for mobile apps with preact. Is there a similar project that takes advantage of the preact libraries similar to how react native does?

oh, I see. I need to use something like cordova.

There are efforts underway to pair Preact up with NativeScript.  That combination produces something quite similar to React Native:
https://github.com/staydecent/nativescript-preact

Thanks! I really like the patterns in both react/preact and was hoping for exactly this in a native dev environment.

@developit I did some more research, and i found that there is a library that uses the same syntax as react but with weex as the platform framework
https://github.com/alibaba/rax

Well It seems that Facebook, changed their license again. This time it appears to be a good thing, 🦄


Unable to receive the commit message in GitHub's pull request.

I have forked this repository for my use.
I want to receive the commit message in GitHub's pull request.
Although I have commit messages, the commit message in GitHub's pull request always appears as "Tokens updated."
I understand the cause. refer: https://github.com/lukasoppermann/design-token-transformer/blob/main/.github/workflows/transform-tokens.yml#L46
I would like to set the commit message in the following section if a commit message is received through the request body.
Like this.
commit-message: ${{ github.event.client_payload.commitMessage || 'Tokens updated' }}

The setting

Added the PR.

@lukasoppermann
I'm not completely understanding, but what does "Merged" mean?
My own PR hasn't been merged yet.
If I miss something, sorry.

Sorry I am on the go and had internet trouble apparently. Now it should be merged.

Thank you!


OneDrive Rest: How to get know if subscribed item is deleted or moved?

I am integrating onedrive with REST, Using webhooks subscription https://docs.microsoft.com/en-us/onedrive/developer/rest-api/concepts/using-webhooks?view=odsp-graph-online with the end points  https://api.onedrive.com/v1.0 (not using graph API) to know the changes of the content, It tell me by webhooks callbackurl that particular subscribed content has been changed, So i sync the change, All goes well.
But if any subscribed webhook content is deleted or moved then i am not getting any callback from the webhook callbackurl. Therefore i am unable to detect that i have to sync the new change.
Please help me out.

Thanks for your feedback.
Does a delta API call indicate that items were deleted or moved?

thanks for reply, When i call delta API with the end point https://api.onedrive.com/v1.0/drives/{driveId}/root/delta
then i get invalidRequest with 400 http code bad request

When using the direct endpoint certain actions (including delta) must be prefixed.

Thanks @chackman  i try to resolve my problem with delta


feat (Topology/Instances/EReal): EReal.toReal tends to top at top

Add nhdsWithin_top and nhdsWithin_bot: the punctured neighbourhoods of top and bot in EReal are the map of atTop and atBot.
Add tendsto_toReal_atTop and tendsto_toReal_atBot.

bors merge


Add method to send check metadata

Motivation
https://github.com/DataDog/datadog-agent/pull/4234
https://github.com/DataDog/datadog-agent/pull/4158

 Integration Service Infos in flare

Closing and opening a new PR for easier discovery


Src cleanup

Method bodies in header files were moved into source files under the cppcache/src/ directory.
Some classes had their visibility set to public because their unit tests are in the integration test suite. We'll have to sort that later.

Please provide a JIRA that describes the purpose for this change. Describe in your PR/commit how these changes achieve the goals of the JIRA.

Can you please update the JIRA with the rational for making these changes? I assume it's and attempt to speed up build times on internal source files. If that is the case can you provide insight in the PR as to how much this change set improves time?

Performance benchmarks can be found here: https://docs.google.com/spreadsheets/d/1N50kDTOZsUWDTP2c5ejt9niGS-1al5Yox8O21FC5U-s/
In conclusion, the difference in runtime performance is insignificant and build times improved (which was the desired outcome).


meetings: add notes from sync on max and jonathan's conversation

Replaces #52. Sorry for all the confusion and all the commits/pull requests!

@MaxWofford can you look over this when you get a chance?

@zachlatta Should we standardize * vs - in markdown? Our files seem to be all doing their own thing.

Yeah, we should definitely have a style guide. I've gone ahead and opened #63 to address this. For now, don't worry about formatting.


WebSocket 501 error

Thanks for creating the WebSocket support!
When I run the wstest.html example on an ESP32:
websocket = new WebSocket(wsUri);
is throwing the following error:

WebSocket connection to 'ws://<IP_ADDRESS>/' failed: Error during WebSocket handshake: Unexpected response code: 501

Looks like there are 2 issues.

def _getConnUpgrade() checks if connection == 'upgrade'.  The problem is connection can hold more than 1 attribute such as 'upgrade, keep-alive'.  I changed == to in to fix:
if 'upgrade' in self._headers.get('Connection', '').lower()

A bigger problem is that uhashlib is not implemented.  MicroWebSocket.py imports sha1 from uhashlib.  I don't have a fix.  MicroPython lib contains a hashlib library.  It contains sha256, but not sha1.  Is it possible to use sha256 instead of sha1?

Ok for (1), connection can contain keep-alive, but this is strange in a ws request.
What's your browser ?
For (2), ok, SHA1 is mandatory yes...
It's implemented on pycom modules and on micropython standard.
Do you really not have this lib :
http://docs.micropython.org/en/latest/wipy/library/uhashlib.html ?

I was testing with Firefox:

'User-Agent': 'Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Firefox/52.0','Connection': 'keep-alive, Upgrade'

Unfortunately, uhashlib is not implemented:
import uhashlib

Traceback (most recent call last):
File "", line 1, in
ImportError: no module named 'uhashlib'

I posted a question to the MicroPython ESP32 developers and they responded that there is a hashlib library.  They provided the following example:
import binascii, hashlib
hash = hashlib.sha1('mango')
binascii.hexlify(hash.digest())

How about enclosing the uhashlib import in a try statement and then if an exception occurs try hashlib?

Ok, it's just "hashlib" and not "uhashlib", depends of implementation.
This is possible to load the existing module with :
try :
from uhashlib import sha1
except :
from hashlib import sha1
:)

uhashlib is it the only last problem for your esp32 ?
I could add this little change :)

Looks like uhashlib was an oversight.  A related pull request #178 was just submitted to the MicroPython ESP32 repo.

Oh yes! 👍


Add test logging helpers to Hosting

These are based off the ones in ServerTests. Now that I've seen them at work, I want to add these to Entropy because we're seeing some hanging tests, but I don't want copy-pasta so I figured I'd make it common in Hosting.
I do plan to make this even more low-impact by adding some xunit extensibility to make it "magic", but I don't want to do that until I've worked out the logging process on a few tests.
There's a particular spot in here I'm not so proud of, but very open to suggestions ;). See if you can spot it!
Note: At some point I plan to pull this to somewhere even more common (Common?) in order to be able to use it in all our tests. Also, I plan to use some xunit hooks to avoid some of the boilerplate.

:up: :date: (and one that adds new reviewable code) - ServerTests also had an HttpClient handler that logged requests which seemed very useful for testing. I moved it to this repo and added a way to ensure that tests can always get an HttpClient with LoggingHandler, pointed at the deployed site.


Update data.rst

correct field type from dipole_year to discovery_year in the documentation

Thanks a lot @Eben60 !


Merge in upstream changes

This is a bit big.. I don't have intentions of performing a merge right now


Don't Merge Yet: Use tahoe-sites for course access groups

Description: Use tahoe-sites for course access groups
JIRA: Link to JIRA ticket
Dependencies: dependencies on other outstanding PRs, issues, etc.
Merge deadline: List merge deadline (if any)
Installation instructions: List any non-trivial installation
instructions.
Testing instructions:

Open page A
Do thing B
Expect C to happen
If D happened instead - check failed.

Reviewers:

[ ] tag reviewer
[ ] tag reviewer

Merge checklist:

[ ] All reviewers approved
[ ] CI build is green
[ ] Version bumped
[ ] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Commits are squashed

Post merge:

[ ] Create a tag
[ ] Check new version is pushed to PyPI after tag-triggered build is
finished.
[ ] Delete working branch (if not needed anymore)

Author concerns: List any concerns about this PR - inelegant
solutions, hacks, quick-and-dirty implementations, concerns about
migrations, etc.

@shadinaif this seems to be ready, no? What's blocking a release of this code?

I just wanted this one be merged first https://github.com/appsembler/course-access-groups/pull/115 . Thank you for the reminder @OmarIthawi


Add SUNY Downstate Medical Center

http://www.downstate.edu/grad/

@jkl1337 Pull request merged. Thank you!


fix: revert all group id to app.revanced

There is a problem of using the upstream org.apktool:brut.j.dir because it depend on org.apktool:brut.j.util while our app.revanced:apktool-lib use app.revanced:brut.j.util. That bring back 'duplicate class issue'. I suggest publishing all 3 packages brut.j.dir, 'brut.j.common and brut.j.util to prevent future issue.

utiil looks more like a typo. I can not find any reference to utiil anywhere though. The modules dir and common can be used from upstream, there is no need to publish it. Instead of this PR, the typo utiil should be fixed wherever it is.

I just noticed, this was just a typo from you. It shouldbe enough to publish packages of all modules except for brut.j.util.

I just noticed, this was just a typo from you. It shouldbe enough to publish packages of all modules except for brut.j.util.

will it bring back https://github.com/revanced/revanced-manager/issues/648 issue?

It shouldn't. It should close the issue, since there shouldn't be any duplicates anymore.

This PR is just a suggestion and there are other way to do that better than a PR. So I will close it.

What about this? Shouldn't this suffice?

What about this? Shouldn't this suffice?

I don't know. In my opinion, brut.j.util need to be published too since https://github.com/revanced/Apktool/commit/a87ac3a46c55c0da0a7d2464529a9237d9aeac54 relate to https://github.com/revanced/revanced-manager/issues/648.

I reopen PR in case you prefer merging this changes instead of committing directly in repo.

@d4rkk3y Instead of brut.j.util, it shouldve been brut.j.common, since theres no commits in our fork to that module, so upstream can be used: fb847de351b48abde111cdbaff8f7c8ab500a657. To be 100% exact, the dependencies should be changed to use the upstream, instead of the project in this fork.

@d4rkk3y Instead of brut.j.util, it shouldve been brut.j.common, since theres no commits in our fork to that module, so upstream can be used: fb847de.

I agree. brut.j.common didn't has dependencies too. So it should not make any problem like this.


Fix CI error: Invalid workflow file: .github/workflows/build.yml#L36 The workflow is not valid. .github/workflows/build.yml (Line: 36, Col: 9): Unexpected value 'working-directory'

Invalid workflow file: .github/workflows/build.yml#L36
The workflow is not valid. .github/workflows/build.yml (Line: 36, Col: 9): Unexpected value 'working-directory'
https://github.com/casdoor/casdoor/actions/runs/3072488933

@gtn1024


new-command: spo report activitypages

implements #1245

Coverage remained the same at 100.0% when pulling 71486a8a34bfa7fef110700b7764a9abf28336a1 on plamber:report-sharepointactivitypages into f62e1ef8c7df7d30ab2e5898f036c4e999016f2b on pnp:dev.


Prettier dump on write mismatch (hex + diff)

Sample output:
running 1 test
test existing_pkg_iodump ... FAILED

failures:

---- existing_pkg_iodump stdout ----
thread 'existing_pkg_iodump' panicked at 'assertion failed: `(left == right)`
  left: `[104, 101, 108, 108, 111, 32, 119, 111, 114, 108, 100]`,
 right: `[203, 238, 82, 84, 0, 0, 0, 0, 4, 2, 0]`:
 hex written: [68, 65, 6c, 6c, 6f, 20, 77, 6f, 72, 6c, 64]
hex expected: [cb, ee, 52, 54, 00, 00, 00, 00, 04, 02, 00]
        DIFF:  ^^', /home/akavel/.cargo/git/checkouts/mock-io-4a9c6b3ff6c2f637/00444b0/src/lib.rs:235:21
note: Run with `RUST_BACKTRACE=1` for a backtrace.

#3 is merged, maybe merging master into here would fix CI?

Good idea, thanks! Done :heavy_check_mark:


Long-running tests

The unit tests, especially test_utils.py, are running very slowly. Now that we're using cached responses for the tests, there's no good reason for the long-running tests. I suspect the length of runtime is due to conservative wait time parameters left over from when we were using live API calls during testing. Reducing test runtime is probably as simple as changing these parameters.

I also think that long-running test workflows in Github Actions workflows are partly due to the install time of responses and its dependencies. This will be resolved if I can generate mocks programmatically rather than use mocks captured with responses. (I also would like to add a test that checks that API responses still match the spec I'm using for mock generation.


How to launch WAL script ?

Hello,
I'm trying to launch the following WAL script with version (0.8.1).
waltry.wal :
(define wait 0)
(define packets 0)
(whenever (rising clk)
          (when (&& req (! ack)) (inc wait))
          (when (&& req ack) (inc packets)))
(print (/ wait packets))

With VCD file provided in tutorial fdl.vcd .
The following command return nothing :
$ wal -l fdl.vcd -c waltry.wal

Even if I insert error in file nothing is returned.

Hi,
try wal -l fdl.vcd waltry.wal.

Thanks @LucasKl
In fact, the right command is with wal script as first argument :
$ wal waltry.wal -l fdl.vcd

>>>>> WAL Runtime error! <<<<<
 1: (define wait 0)
 2: (define packets 0)
 3: (whenever (rising clk)
 4:           (when (&& req (! ack)) (inc wait))
 5:           (when (&& req ack) (inc packets)))

The error is from code, if I try simplest following program It works :
(print "pof")

$ wal pof.wal -l fdl.vcd
pof

If script name is given at the end of command it doesn't work :
$ wal -l fdl.vcd pof.wal
File extension ".wal" not supported.
WAL 0.8.1
Exit to OS or terminate running evaluations with CTRL-C
t0(0) >->

Oh right, the order is important. Good that it works now. If something comes up, feel free to contact me. :)


Wrap docstrings and comments in HookedTransformer

Description
While using TransformerLens I often read HookedTransformer for documentation, but struggle with the lack of line breaks
in the comments. I'm updating the line breaks to align with IntelliJ's default right margin to fix this. I've tried to err on the side of leaving things as is when the lack of line breaks seems potentially beneficial, such as to avoid line breaks during an equation. Parts of the code use a much tighter line width than the IntelliJ default (80 chars) and I haven't touched that either.
I'm not pursuing a long term solution i.e. autoformatting. The reason is that black will not add line wraps for docstrings because it could affect structured data stored in the strings, so we would have to add a new formatting tool and assess the performance tradeoffs and edge cases that come with that, and I feel that this update gets us 60% of the value with 5% of the effort.
Type of change
Please delete options that are not relevant.

[x] Formatting

Checklist:

[N/A] I have commented my code, particularly in hard-to-understand areas
[N/A] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[N/A] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have not rewritten tests relating to key interfaces which would affect backward compatibility

My main concern/annoyance (and the reason lines don't currently have this, and that I expect people to use line wrap for) is that editing prose is a pain in the ass if you need to ensure it remains aligned (every line in a block needs to be re aligned, it's so annoying!). None of the Vs code extensions for doing this is right.
That said, this change seems net good, thanks! This just annoys me a lot, and there doesn't seem to be a good solution

Thanks @luciaquirke. I think you committed some unrelated changes that shouldn't be in the CI. Can you please edit the commit to remove:

[ ] .coverage.Lucias-MacBook-Air.local.67990.863551
[ ] .idea/workspace.xml

After that, I'll accept.

Thanks for the reviews @jbloomAus and @neelnanda-io!
I've removed the erroneous files and added .idea/ to the gitignore. Also thanks for reminding me about soft wrap, next time I won't have to raise a PR 😄

thanks, looks good!


[index] Provide distinct USRs for extension symbols

For indexing purposes we need to be able to treat extensions as distinct symbols.
rdar://30512293

@swift-ci smoke test and merge

Can you explain how this works? It looks like two extensions of the same type will still get the same USR.

Ah, never mind, I see printExtensionUSR is an existing thing.


Fix for deleting the last chip with backspace after it was the first element that received the focus.

This PR contains a fix for deleting the last chip with the backspace key.
Issue description:

Open the samples page of this project in Chrome
Click on the last chip in one of the examples
Hit now backspace
The chip gets removed but the now last chip does not receive the focus. Also the console shows an Angular error:
Uncaught TypeError: chipNavigate is not a function  at HTMLUnknownElement.Chips.scope.chips.handleKeyDown (angular-chips.js:222)  at defaultHandlerWrapper (angular.js:3540)  at HTMLUnknownElement.eventHandler (angular.js:3528)
This is caused by chipNavigate being undefined at: https://github.com/mohbasheer/angular-chips/blob/master/src/js/directives/chips.js#L219

Expected result: No angular error & the last chip should receive the focus
Fixed by:

Calling focusOnChip when no chipNavigate method was yet created.
Refactored focusOnChip() to only give focus to the last chip
Made sure the cases for arrow keys are still working ...

Hi @mohbasheer,
Just pushed a new commit:

When in the input field and you hit backspace or arrow key the focus is no longer moved to the chips.
Updated tests.

Let me know what you think and if you like me to squash commits also ...
Koen

it's working fine now. so Im closing this


LWM2M client registration failed

Hi Everyone,
I downloaded wakaama and build directory as described on wakaama github. But LWM2M client registration failed when I start the client. I can confirm the server is running, ready to accept connection. Below is the error message I always received.
**akil@akil-VirtualBox ~/fiware>** _./lwm2mclient -4 -h <IP_ADDRESS>_
Trying to bind LWM2M Client to port 56830
LWM2M Client "testlwm2mclient" started on port 56830
> Opening connection to server at <IP_ADDRESS>:5683
 -> State: STATE_REGISTERING
 -> State: STATE_REGISTERING
 -> State: STATE_REGISTERING
 -> State: STATE_REGISTERING
 -> State: STATE_REGISTERING
 -> State: STATE_REGISTERING
 -> State: STATE_BOOTSTRAP_REQUIRED
lwm2m_step() failed: 0xA3

Don't understand why this happen please.

Hi,
The usual suspect is the firewall, either on the server host or a network equipment on the way. Make sure UDP traffic on port 5683 is allowed.
Regards

Hi David,
Thank you for your response.  But a friend here could connect to the server on his machine. Nine report said error. I try instead use VirtualBox to run ubuntu (instead of my macos x) to see if this has to do with operating system. Same response.

Hi,
Can you tell your client details.
Like if it is a Linux computer, or a virtual machine or an embedded Board
like Raspberri Pi.
Thanks,
Amrit
On Thu, Oct 18, 2018 at 3:46 PM Akilu Rilwan Muhammad <
<EMAIL_ADDRESS>wrote:

Hi David,
Thank you for your response. But a friend here could connect to the server
on his machine. Nine report said error. I try instead use VirtualBox to run
ubuntu (instead of my macos x) to see if this has to do with operating
system. Same response.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/eclipse/wakaama/issues/395#issuecomment-430954857,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiWzfphUV16nEe_ExumofWxV1NVAVURTks5umFUPgaJpZM4XtELJ
.

Try running wireshark on the same box as your LwM2M Client to see if the packets are sent by the client and where, and to see if the client receives any reply from the server.
You can also activate traces in wakaama by defining LWM2M_WITH_LOGS to have more details on the error.

Hi, Can you tell your client details. Like if it is a Linux computer, or a virtual machine or an embedded Board like Raspberri Pi. Thanks, Amrit
…
On Thu, Oct 18, 2018 at 3:46 PM Akilu Rilwan Muhammad < @.***> wrote: Hi David, Thank you for your response. But a friend here could connect to the server on his machine. Nine report said error. I try instead use VirtualBox to run ubuntu (instead of my macos x) to see if this has to do with operating system. Same response. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#395 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AiWzfphUV16nEe_ExumofWxV1NVAVURTks5umFUPgaJpZM4XtELJ .

Hi, Can you tell your client details. Like if it is a Linux computer, or a virtual machine or an embedded Board like Raspberri Pi. Thanks, Amrit
…
On Thu, Oct 18, 2018 at 3:46 PM Akilu Rilwan Muhammad < @.***> wrote: Hi David, Thank you for your response. But a friend here could connect to the server on his machine. Nine report said error. I try instead use VirtualBox to run ubuntu (instead of my macos x) to see if this has to do with operating system. Same response. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#395 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AiWzfphUV16nEe_ExumofWxV1NVAVURTks5umFUPgaJpZM4XtELJ .

Hi Amrit,
I am running Ubuntu (18.04) as virtual machine.

Try running wireshark on the same box as your LwM2M Client to see if the packets are sent by the client and where, and to see if the client receives any reply from the server.
You can also activate traces in wakaama by defining LWM2M_WITH_LOGS to have more details on the error.

Hi David,
Can you please add more details how to do the stated approach?

Hi Akilu,
I also tried once and faced similar issues with virtual box.
In my case the server was installed in an independent Linux machine and
client was installed in a Virtual Box and Host PC being a Windows 7.
Then switched to Raspberry Pi for connecting.
The client through a virtual machine is not able to send packets out of the
host computer.
If the LWM2M server is also in your control, then I suggest running an
example udp server/client programme and see if you are able to connect.
https://www.geeksforgeeks.org/udp-server-client-implementation-c/
Let me know if you are able to connect.
Also you able to ping the server IP from client Virual Box and ping client
Virtual Box from server in the present scenario?
Thanks & Regards,
Amrit Pal Singh.
On Thu, Oct 18, 2018 at 5:45 PM Akilu Rilwan Muhammad <
<EMAIL_ADDRESS>wrote:

Try running wireshark on the same box as your LwM2M Client to see if the
packets are sent by the client and where, and to see if the client receives
any reply from the server.
You can also activate traces in wakaama by defining LWM2M_WITH_LOGS to
have more details on the error.
Hi David,
Can you please add more details how to do the stated approach?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/eclipse/wakaama/issues/395#issuecomment-430985699,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiWzfvqPW_lpzrfne_y_HaRZv1bIvxtBks5umHDPgaJpZM4XtELJ
.

Hi Akilu, I also tried once and faced similar issues with virtual box. In my case the server was installed in an independent Linux machine and client was installed in a Virtual Box and Host PC being a Windows 7. Then switched to Raspberry Pi for connecting. The client through a virtual machine is not able to send packets out of the host computer. If the LWM2M server is also in your control, then I suggest running an example udp server/client programme and see if you are able to connect. https://www.geeksforgeeks.org/udp-server-client-implementation-c/ Let me know if you are able to connect. Also you able to ping the server IP from client Virual Box and ping client Virtual Box from server in the present scenario? Thanks & Regards, Amrit Pal Singh.
…
On Thu, Oct 18, 2018 at 5:45 PM Akilu Rilwan Muhammad < @.***> wrote: Try running wireshark on the same box as your LwM2M Client to see if the packets are sent by the client and where, and to see if the client receives any reply from the server. You can also activate traces in wakaama by defining LWM2M_WITH_LOGS to have more details on the error. Hi David, Can you please add more details how to do the stated approach? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#395 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AiWzfvqPW_lpzrfne_y_HaRZv1bIvxtBks5umHDPgaJpZM4XtELJ .

Hi Amrit,
I doubt if the issue has to do with ubuntu the VirtualBox, because just now I tried my process from the host OS (macOS 10.14) I am getting the same error status...

Try running wireshark on the same box as your LwM2M Client to see if the packets are sent by the client and where, and to see if the client receives any reply from the server.
You can also activate traces in wakaama by defining LWM2M_WITH_LOGS to have more details on the error.

Try running wireshark on the same box as your LwM2M Client to see if the packets are sent by the client and where, and to see if the client receives any reply from the server.
You can also activate traces in wakaama by defining LWM2M_WITH_LOGS to have more details on the error.

Hi @dnav
See below the traces from the wireshark:
I install wireshark and I can confirm packets are being sent from cilent.
Frame 550: 100 bytes on wire (800 bits), 100 bytes captured (800 bits) on interface 0
Linux cooked capture
Internet Protocol Version 4, Src: <IP_ADDRESS>, Dst: <IP_ADDRESS>
Internet Control Message Protocol

Hi @arilwan,
Server port in your command should be 5683 if you did not change the default when launching the Server.
If packets are sent by the Client, then the next step is to check that the Server receives them. So you should run wireshark on the Server box. If packets are not received then your problem lies in the network configuration.
Regards,

Hi Akilu,
Try running the example udp client/server for the link that i gave.
Then the issue can be separate if it is a general UDP server/client
connection issue or a specific LWM2M server/client issue.
Thanks & Regards,
Amrit Pal Singh.
On Thu, Oct 18, 2018 at 9:09 PM David Navarro<EMAIL_ADDRESS>wrote:

Hi @arilwan https://github.com/arilwan,
Server port in your command should be 5683 if you did not change the
default when launching the Server.
If packets are sent by the Client, then the next step is to check that the
Server receives them. So you should run wireshark on the Server box. If
packets are not received then your problem lies in the network
configuration.
Regards,
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/eclipse/wakaama/issues/395#issuecomment-431057887,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiWzfv2wkJM_bu4bADLKNX4dEGLiYqXiks5umKCsgaJpZM4XtELJ
.

Hi Akilu,
you need to compile the code.
You can compile from either the client or server.
But the server binary has to run on server and client binary on client
machine.
You can run both first on localhost i.e both the server and client binary
on localhost, just to make sure that the binaries and the code is  working
fine.
And then run on separate machines.
Dont use virtual Box in this trial.
On Thu, Oct 18, 2018 at 9:56 PM Akilu Rilwan Muhammad <
<EMAIL_ADDRESS>wrote:

Hi Akilu, Try running the example udp client/server for the link that i
gave. Then the issue can be separate if it is a general UDP server/client
connection issue or a specific LWM2M server/client issue. Thanks & Regards,
Amrit Pal Singh.
… <#m_5177807824193391228_>
On Thu, Oct 18, 2018 at 9:09 PM David Navarro @.***> wrote: Hi
@arilwan https://github.com/arilwan https://github.com/arilwan, Server
port in your command should be 5683 if you did not change the default when
launching the Server. If packets are sent by the Client, then the next step
is to check that the Server receives them. So you should run wireshark on
the Server box. If packets are not received then your problem lies in the
network configuration. Regards, — You are receiving this because you
commented. Reply to this email directly, view it on GitHub <#395 (comment)
https://github.com/eclipse/wakaama/issues/395#issuecomment-431057887>,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiWzfv2wkJM_bu4bADLKNX4dEGLiYqXiks5umKCsgaJpZM4XtELJ
.
Hi Amrit,
Yes, that's a good suggestion. But I need a little guide (first step), how
to run the examples you sent. Do I need to copy any file to the remote
server (eg. UDPServer.c), or build environment with both files on my
localhost.
I will appreciate some guide, please. Thank you.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/eclipse/wakaama/issues/395#issuecomment-431074685,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiWzfiwuYcmXdbcWhB9ZGikX6NNm-L7cks5umKvHgaJpZM4XtELJ
.

Hi Akilu, you need to compile the code. You can compile from either the client or server. But the server binary has to run on server and client binary on client machine. You can run both first on localhost i.e both the server and client binary on localhost, just to make sure that the binaries and the code is working fine. And then run on separate machines. Dont use virtual Box in this trial. On Thu, Oct 18, 2018 at 9:56 PM Akilu Rilwan Muhammad <<EMAIL_ADDRESS>wrote:
…
Hi Akilu, Try running the example udp client/server for the link that i gave. Then the issue can be separate if it is a general UDP server/client connection issue or a specific LWM2M server/client issue. Thanks & Regards, Amrit Pal Singh. … <#m_5177807824193391228_> On Thu, Oct 18, 2018 at 9:09 PM David Navarro @.***> wrote: Hi @arilwan https://github.com/arilwan https://github.com/arilwan, Server port in your command should be 5683 if you did not change the default when launching the Server. If packets are sent by the Client, then the next step is to check that the Server receives them. So you should run wireshark on the Server box. If packets are not received then your problem lies in the network configuration. Regards, — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#395 (comment) <#395 (comment)>>, or mute the thread https://github.com/notifications/unsubscribe-auth/AiWzfv2wkJM_bu4bADLKNX4dEGLiYqXiks5umKCsgaJpZM4XtELJ . Hi Amrit, Yes, that's a good suggestion. But I need a little guide (first step), how to run the examples you sent. Do I need to copy any file to the remote server (eg. UDPServer.c), or build environment with both files on my localhost. I will appreciate some guide, please. Thank you. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#395 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AiWzfiwuYcmXdbcWhB9ZGikX6NNm-L7cks5umKvHgaJpZM4XtELJ .

Hi @apsk25
Huh!, this attempt failed too. I understand this has nothing to do with virtualbox. Huh!

I guess we can close this issue ?


Chapter one lesson one homework

praying to God i did the pull request right .

praying to God i did the pull request right .

Almost. The only issue is unnecessary readme and ide change.


React Project-Responsive Paragraph Word Counter

Hey @Codetoadeath can you please assign this issue to me?

Hey @Codetoadeath
I have starred the repository, implemented this word-counter website using react and also added description on how to run the project.
Finally, I have raise a Pull request, please review it when you get some time.


DBC22-1882: added empty state advisory and bulletin

DBC22-1882: added empty state advisory and bulletin

Please make the following adjustments:

Make sure you assign someone in the PR
This ticket refers to empty state as in the retrieved list is actually empty; we get 0 bulletins or advisories
The CMS requirements in the wireframes are no longer necessary. Please fill the static text from the "Example empty state text" section


kie-issues#2549: Decision Service cannot be recognized in columns of decision table

Closes: https://github.com/apache/incubator-kie-tools/issues/2549
It closes the ticket related to the colorize and autocomplete only.
https://github.com/user-attachments/assets/0059f31e-8eee-4426-8c3a-1b58ccc0a4df

@jomarko @tiagobento Changes applied! Ready for you.


Remove version: Tableau.Reader version 24.1.1062

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://downloads.tableau.com/tssoftware/TableauReader-64bit-2024-1-2.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 896)

Close with reason: Package still available.;


Add an option to MS Windows Binary to launch without console window

Godot version:
3.0.2
OS/device including version:
Microsoft Windows 10
It would be nice to have an option to launch the engine without an attached console window on Windows OS.
Just little thing, but it makes switching between Godot and other applications one click more, which adds up when switching often. It also gets confusing when having more then one Godot instance open.

The editor is best used with a console window because some errors aren't displayed in the editor window – those are usually bugs, but can also be a result of user error.
Note that release-mode Windows export templates will not spawn a console window (debug-mode Windows export templates will spawn one though).

I've actually been thinking about this a while ago. I agree with @Calinou that the console is quite useful, but only on Windows the console is shown, so cross-platform behavior is not quite consistent. If you need the messages you could open via command prompt, like you need to do with Linux or macOS.

@Calinou commented
The editor is best used with a console window because some errors aren't displayed in the editor window

log > temp file > disk

Okay... But I can hide it in a Blender.

WinAPI has 2 methods:
ShowWindow() and GetConsoleWindow().
Thanks to them, you can change the visibility of the command line.
For example like this done in Blender:

It seems to me that it will be quite easy to add such functionality to Godot, but I do not have the opportunity :(

I downloaded the source code and added a code to toggle the console window on F12 key press.

@dedraks: mind making a PR?

Please do, @dedraks

It seemed like no one was working on this at the moment, so I created #24448 which basically mirrors Blender's behaviour. Feedback would be appreciated.

I just googled to see if this was possible and it was added a couple minutes ago. Very surreal and very awesome!

Meanwhile I've been working around this since godot 2 by opening one command line and running several instances of godot from that same command line.
Would there be any caveats to doing this? Any conflicts or anything?

The console being useful is not really the point. If you become advanced enough to need the console, it should be clear where it is. It is bad UX to have a console launch along with the main application and sit with no option to hide/unhide like any other (useful) UI element. It being a window outside of the main Godot window makes this even more in eed of this toggle. Some of us have multiple projects open and its, pardon my french, a bit of a shitshow with the consoles getting in the way as well. This feature should be opt-in and off by default. Voted in favor or this. (and if it can be put into the main app along with Debugger etc on the bottom, even better)

Guys, it's implemented already...

It's 2020, I am currently starting work on a Godot "assistant application" which enhances Godot, I have had issues with the console window too, and after researching about Godot's console online, it looks like it is still a problem. So I will add a custom plugin for Godot that will allow different methods of displaying any log or console text. I am testting different methods, including adding a console display to Godot to remove the 2nd console window.

@HonorsoftEntertainment You can hide the console window using Editor > Toggle Console Window.

@HonorsoftEntertainment As of Godot 3.2, you can hide the console window using Editor > Toggle Console Window.
Yes, but isn't that just for builds? I read that the console window is always shown in the Godot Editor. I have read complaints about the old-style console windows, and I have a few too. I think a better console output is needed for Godot, maybe with improvements added like a proper text copy paste system which console simply don't have.

@HonorsoftEntertainment The option I mentioned is available in the editor.

It doesn't work with Windows 11's new terminal, you should use Godot 3.5.


OkHttp sending 404 if the first request fails

I have the following scenario:

I have an app that request images from a hosting server. If the image does not exist in the server, it returns with a 404 (this is correct)

I upload the image to the server

Exit app and reload the app, the image exist in the server, I can view it if I use the browser but OkHttp keep saying 404 NOT FOUND (see log)

But If I clean the cache, the image is correctly downloaded from the internet.

Note: I am using picasso
Log
10-12 14:29:04.003 10793-10793/ni.com.spiralni.help D/Picasso: Main        created      [R7] Request{https://firebasestorage.googleapis.com/v0/b/helpapp-d0762.appspot.com/o/images%2Fborradme.jpg?alt=media resize(200,200) centerCrop}
10-12 14:29:04.005 10793-13185/ni.com.spiralni.help D/Picasso: Dispatcher  enqueued     [R7]+1ms
10-12 14:29:04.005 10793-13197/ni.com.spiralni.help D/Picasso: Hunter      executing    [R7]+1ms
10-12 14:29:24.761 10793-10793/ni.com.spiralni.help I/Choreographer: Skipped 58 frames!  The application may be doing too much work on its main thread.
10-12 14:29:33.839 10793-13197/ni.com.spiralni.help I/OkHttpClient: Sending request https://firebasestorage.googleapis.com/v0/b/helpapp-d0762.appspot.com/o/images%2Fborradme.jpg?alt=media on null
10-12 14:29:34.109 10793-13197/ni.com.spiralni.help W/art: Verification of void com.squareup.okhttp.internal.http.HttpEngine.<init>(com.squareup.okhttp.OkHttpClient, com.squareup.okhttp.Request, boolean, boolean, boolean, com.squareup.okhttp.Connection, com.squareup.okhttp.internal.http.RouteSelector, com.squareup.okhttp.internal.http.RetryableSink, com.squareup.okhttp.Response) took 152.613ms
10-12 14:29:34.319 10793-13197/ni.com.spiralni.help W/art: Verification of com.squareup.okhttp.Response com.squareup.okhttp.internal.http.HttpEngine.cacheWritingResponse(com.squareup.okhttp.internal.http.CacheRequest, com.squareup.okhttp.Response) took 153.197ms
10-12 14:29:34.622 10793-13197/ni.com.spiralni.help W/art: Verification of void com.squareup.okhttp.internal.http.HttpEngine.maybeCache() took 106.882ms
10-12 14:30:09.736 10793-13197/ni.com.spiralni.help W/art: Verification of com.squareup.okhttp.internal.Platform com.squareup.okhttp.internal.Platform.findPlatform() took 195.866ms
10-12 14:30:21.732 10793-13197/ni.com.spiralni.help I/OkHttpClient: Received response for https://firebasestorage.googleapis.com/v0/b/helpapp-d0762.appspot.com/o/images%2Fborradme.jpg?alt=media in 49058.7ms
                                                                    OkHttp-Selected-Protocol: h2
                                                                    x-guploader-uploadid: AEnB2Ur7reXY0kFd1ufrx6RFVxGZk8u40TKGZ3_4SUYtsUvzJe6E1HnI8nPqRczAi7XEuKmIYvSdHgOAU5d_dWVukO-L_EpfRDLCsr3JNE6dNUXkJWs9SQc
                                                                    content-type: application/xml; charset=UTF-8
                                                                    content-length: 127
                                                                    date: Wed, 12 Oct 2016 18:52:01 GMT
                                                                    expires: Wed, 12 Oct 2016 18:52:01 GMT
                                                                    server: UploadServer
                                                                    alt-svc: quic=":443"; ma=2592000; v="36,35,34,33,32"
                                                                    OkHttp-Sent-Millis:<PHONE_NUMBER>795
                                                                    OkHttp-Received-Millis:<PHONE_NUMBER>229
                                                                    Cache-Control: max-age=864000
10-12 14:30:21.829 10793-13197/ni.com.spiralni.help I/OkHttpClient: Received response code 404

It can be useful to cache 404s (http://highscalability.com/blog/2010/3/26/strategy-caching-404s-saved-the-onion-66-on-server-time.html).
If you don't want this behavior, change the cache header on your server (not sure if Firebase gives you such options).
Worst case scenario, you could modify the response Cache-Control header client-side (perhaps in an in interceptor).

@NightlyNexus , thanks for the article, it was a lifesaver 👍 .
It seems that firebase allows to override the caching behaviour for 404 resources. (Docs https://www.firebase.com/docs/hosting/guide/full-config.html)
Just for curiosity, how can I achieve this by creating the interceptor you mentioned?

Perhaps something like this.
  @Override public Response intercept(Chain chain) throws IOException {
    Response response = chain.proceed(chain.request());
    if (response.code() == 404) {
      response = response.newBuilder().removeHeader("Cache-Control").build();
    }
    return response;
  }
}```

Glad you could change it on Firebase's side!

Thanks @NightlyNexus !


Is ConceptualSearch applicable for ElasticSearch?

Hi, thanks for this great example of combining word2vec with Lucene / Solr. How transferable is your code to Elastic Search? I ask only because AWS has an ElasticSearch Service which would be convenient to use.

Sorry for the late reply, I just saw this. I am not that familiar with ES,  but I am assuming it uses the same file format as it's also Lucene based so it should work. The plugins I have developed won't work out of the box, but they are not needed for the clustering version output, which is the easiest to get going with anyways. Let me know if you have any other questions

Please note - If anyone has any questions, please include @simonhughes22 in the issue otherwise I won't get an email and may miss updates if I haven't checked in a while.

If you just use the synonym file output by the clustering approach, it would work fine in ES


Fixed "gradle.properties" "mod_url" and "mod_issues_url"

Someone needed some sleep when creating this file :/

Nah, was just planning to move this to that namespace if it ever get's to a released state :P
But I guess for now this is more correct.


default map option in new ui

good morning, this note is to request that there be an option or way to select a global default map in the new ui, yes, I know that users could easily change it by selecting another, but many of us use aerial or custom maps, and the truth is that between the fact that the map is what is most noticeable in the app and that most of the users are kinda "stupid", they always end up entering a new pc and seeing a completely blank map which discourages them , thank you very much for your attention to this case

Seems to be similar to https://github.com/traccar/traccar-web/issues/1006.

Seems to be similar to #1006.
Greetings, yes, basically, another option would be that all the map layers are active when starting, and that the user does not have to enter preferences to activate those that are normally used, is that easier?

Is this already solved?
There is a server attribute "Active maps". The problem is that it doesn't show the list of maps. You have to type as free String.


Feat standalone

Thank you so much for the pull request @ronnahom96! I noticed this is your first pull request and I wanted to say welcome to the Cesium community!
The Pull Request Guidelines is a handy reference for making sure your PR gets accepted quickly, so make sure to skim that.

:x: Missing CONTRIBUTORS.md entry.

Please add yourself to the contributors file!

:x: Missing CLA.

Please send in a Contributor License Agreement (CLA) and comment back here to let us know to check this!

:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

Hi @ronnahom96, could you please let us know the goal of this PR?

@ronnahom96 I'm going to close this PR for now. If you would like someone to review your contribution, I would suggest writing up a summary of the changes in your PR description. Thanks!


pw: command not found

Problem
When  source ./scripts/bootstrap.sh,it shows that pw: command not found
`  BOOTSTRAP! Bootstrap may take a few minutes; please be patient.
Downloading and installing packages into local source directory:
Setting up CIPD package manager...done (3.0s)
Setting up Python environment.....done (5.7s)
Setting up pw packages............skipped (0.1s)
Setting up Host tools.............done (0.1s)
Activating environment (setting environment variables):
Setting environment variables for CIPD package manager...done
Setting environment variables for Python environment.....done
Setting environment variables for pw packages............skipped
Setting environment variables for Host tools.............done
Checking the environment:
pw: command not found
Error during bootstrap--see messages above.

`

Problem
When source ./scripts/bootstrap.sh,it shows that pw: command not found
` BOOTSTRAP! Bootstrap may take a few minutes; please be patient.
Downloading and installing packages into local source directory:
Setting up CIPD package manager...done (3.0s)
Setting up Python environment.....done (5.7s)
Setting up pw packages............skipped (0.1s)
Setting up Host tools.............done (0.1s)
Activating environment (setting environment variables):
Setting environment variables for CIPD package manager...done
Setting environment variables for Python environment.....done
Setting environment variables for pw packages............skipped
Setting environment variables for Host tools.............done
Checking the environment:
pw: command not found
Error during bootstrap--see messages above.

`

I have met the same issue. Have you fixed it?

Delete .environment dir,and retry,will be ok

Delete .environment dir,and retry,will be ok

Delete .environment dir,and retry,will be ok

Did not fix the error on macOS 13.0 Beta (22A5321d).

Delete .environment dir,and retry,will be ok

Did not fix the error on macOS 13.0 Beta (22A5321d).

Tried with a clean repo per #10766 and it works now.

Delete .environment dir,and retry.
This problem still exists.

same issue, still stuck

still exists. Suggestions above of deleting  .environment dir a nd trying again works

I had the same issue here.
Suggestions above of deleting .environment dir and trying again works²
Environment:
Ubuntu 22.04.3 LTS - Virtual Machine
Project CHIP Version: <IP_ADDRESS>


Add effectiveHorizontalAlignment direction default case

issue message
Switch covers known cases, but 'UIUserInterfaceLayoutDirection' may have additional unknown values, possibly added in future versions

Thanks for your contribution! However, I won't merge the PR because

I find it highly unlikely that there will be additional values to UIUserInterfaceLayoutDirection in the future. The world's languages probably won't change for a couple of centuries and so their layout directions won't either...

If a future UIKit version would introduce a new value for UIUserInterfaceLayoutDirection, we would have to deal with that new value and implement it accordingly, so it's important that the code shows a compiler error at that point to makes us aware of the issue.

The following argument would not hold any longer:
 // It's safe to force-unwrap as `.leading` and `.trailing` are covered
 // above and the `trivialMapping` dictionary contains all other keys.


Make typescript target es6 and add babel plugin in rollup.config.js. …

…this should makes the tree shaking work

I couldn't get test to pass locally but the tree shaking is working
for example, searching for OnBehalfOfClient in the built lib/msal-browser/dist/index.es.js should yield no results if treeshaking works

If folks would like to help improve this PR and get tests working, that's very welcome
I followed some of the instructions from this issue to make this PR: https://github.com/rollup/rollup/issues/2807#issuecomment-484798155

Quick question: how does this impact IE support?

I also don't understand why the build process generates a bundle for the "module" field in package.json, it just doesn't make sense to me, but perhaps i'm missing something.
Bundlers rely on import's/export's to tree shake effectively (same as rollup), so I expect the "module" field to link to an index file that then imports the relevant modules, which in-turn imports other modules. The consuming build then will bundle and tree-shake.
I expect this library to publish ES modules (as in, seperate files with imports/exports) to allow my consuming build to tree-shake.

@badsyntax Yea this is where I also need to learn more about, I'm very new to how js libraries should export modules
@jasonnutter I'm not sure but I the babel plugin that is added should respect the .babelrc config file https://www.npmjs.com/package/@rollup/plugin-babel
Let me read further to see what the current babelrc file is configured to do, sorry, very new at this :)

after some reading, I found that it looks like for tree-shaking to work for a webpack project, the package shouldn't be bundled (in one file)
https://github.com/webpack/webpack/issues/9337#issuecomment-524966201
So we need to use preserveModules, I think this is what you were saying @badsyntax ?

this commit so far is not correct in enabling tree-shaking for the consumer, it is actually treeshaking the msal-common dependencies inside msal-browser, which wont do anything since the consumer will pull msal-common and msal-browser seperately

I think I may have gotten it to work, here is folder structure of the build in msal-browser and msal-common now that I've preservedModules: true in the rollup configs
msal-browser

msal-common

My project that uses msal-browser has decreased its bundle size about 150kB from this tree-shaking
TODO: uncomment out the other builds (cjs, minified build, etc)

This is awesome work @thoo1! Over the weekend I will try your branch in my project to check the size reductions and verify the approach.

folder structure after latest changes (to enable cjs and umd builds too)
msal-browser:

msal-common:

I also don't understand why the build process generates a bundle for the "module" field in package.json, it just doesn't make sense to me, but perhaps i'm missing something.
Bundlers rely on import's/export's to tree shake effectively (same as rollup), so I expect the "module" field to link to an index file that then imports the relevant modules, which in-turn imports other modules. The consuming build then will bundle and tree-shake.
I expect this library to publish ES modules (as in, seperate files with imports/exports) to allow my consuming build to tree-shake.

We produce single-file builds for our CDN hosted version. cc: @pkanher617

Also interested in this feature. in our case msal takes 2/5 of bundle size which is a lot.
@jasonnutter could you please tell us what do we miss here to have this merged?

Apologies, been busy and haven't had a chance to follow up here.
This brings up another question: could this be solved if we moved msal-browser and msal-common to tsdx (which we use for msal-react and msal-node)? We were already planning to do that in the next major version of MSAL Browser, but we could do that work now if it will address this issue.
cc @pkanher617

In the meantime, I'll pull down this branch and see how it looks. Thanks for taking the time @thoo1!


CSV Reporting support

Hi,
I can't find anything about the CSV reporting support that metrics implements (https://dropwizard.github.io/metrics/3.1.0/manual/core/#man-core-reporters-csv).
Is there a future CSV support for this plugin ?

Strange… I wonder why I didn't include that…

Fixed in 3.1.1


how to get to a std python data structure

is not in docs
now we get something like
[[], [50-bare_string: id], [], [0-assignment: =], [], [90-string: "len"], [], [91-newline:
], [], [50-bare_string: name], [], [0-assignment: =], [], [90-string: "ms1_lenoir"], [], [91-newline:
], [], [50-bare_string: descr], [], [0-assignment: =], [], [90-string: ""], [], [91-newline:
], [91-newline:
]]
not very nice
why do you keep the newlines ...
I thought the goal of this project was to go from any python data structure & serialize in a consistent way
which means this info does not have to be remembered
I am surprised that something like
data=contoml.loads(C)
does not result in a clean python dict of lists....

the easiest fix is prob to create a method which can dump such a data structure

The reason for all of this is so we can keep the original structure of the toml file including comments and new lines.
If you want python dict/list structure use TOMLFile.primitive
example

In [20]: tom = contoml.loads("""
[apple]
color = green
type = jonagold
""")
In [21]: tom.primitive
Out[21]: {'apple': {'color': 'green', 'type': 'jonagold'}}

As @grimpy elaborated, the types constructed by the contoml are suped up equivalents of Python's containers and primitives. You can index and update them like regular Python values and they'll do their best to maintain their original textual structure when serialized back.
Perhaps the __repr__() definitions of those types aren't very clear, but they're supposed to show the content of those objects which are the lexcial tokens that made up this value.
The constructed top-level value is an instance of TOMLFile and its docstrings reveal the property that can be used to go from this data structure to a plain Python container:
 |  ----------------------------------------------------------------------
 |  Data descriptors defined here:
 |
 |  primitive
 |      Returns a primitive object representation for this container (which is a dict).
 |
 |      WARNING: The returned container does not contain any markup or formatting metadata.

Please close this issue if this answers your question.

the idea was NOT to keep enter's, spaces, ...
the full goal of this module is to always generate same serialization independent of what people put in like in HRD

tom.primitive
works (-:
great, will have to document this

I've added an example to the README.md file displayed on the main page which mentions that exact use case.


Create home page for the game and choosing a language to play with

Expected Behavior
when entering the rebus game a home page will welcome the user containing a logo, the name of the game and a nav bar at the top of the page where you can choose a language: English, French, Arabic, Spanish etc..
After choosing the language you will be directed to the game
Actual Behavior
Right now when you enter the game you directly can start playing in one language which is English
Steps to Reproduce the Problem

Create a home page with nav bar containing the languages to add
start creating  the pieces of the game in the first added language
Start creating  the pieces of the game in the second added language
etc..

Can I work on this issue?

i want to work on it plz guide me

Can we work together in this issue?????

@shrushti22 can we work together in this issue ?????


feat: ignore buffers that are of the type nofile, prompt, terminal

This fixes:

#1
A part of #2. For Neogit with diffview integration enabled, this errors out when quitting a diffview diff.

Hey, @stevenxxiu!
I saw you PR! But I haven't time to test it and merge it. I have some amends that I'd like to do. I will add them on top of your PR and then we'll merge it.  I think I will resolve it in one or two weeks. Sorry

No worries, hope you enjoyed your holiday! You can add comments and I can update my PR too if you wish.

I wrapped duplicated code pieces into functions. This made init.lua module more concise and readable. And with new wrap_fn.lua, I will be able to fallback to the previous implementation in case of any setbacks with this one. @stevenxxiu, now you can check whether old problems trouble you and if new ones emerged!

Thanks for the refactor.
Not sure if this is related, but when I go in diffview, focus on some file contents, then exit, I get the error:
E5108: Error executing lua vim/_editor.lua:0: nvim_exec2()..BufHidden Autocommands for "*": Vim(append):Error executing lua callback: Vim:E957: Invalid window number
stack traceback:
        [C]: in function 'getmatches'
        ...y/highlight-whitespace/lua/highlight-whitespace/core.lua:102: in function 'fn'
        ...ighlight-whitespace/lua/highlight-whitespace/wrap_fn.lua:11: in function 'callback'
        ...y/highlight-whitespace/lua/highlight-whitespace/init.lua:22: in function <...y/highlight-whitespace/lua/highlight-whitespace/init.lua:21>
        [C]: in function 'nvim_exec2'
        vim/_editor.lua: in function 'cmd'
        ...re/nvim/lazy/neogit/lua/neogit/integrations/diffview.lua:20: in function 'close'
        [string ":lua"]:1: in main chunk
stack traceback:
        [C]: in function 'nvim_exec2'
        vim/_editor.lua: in function 'cmd'
        ...re/nvim/lazy/neogit/lua/neogit/integrations/diffview.lua:20: in function 'close'
        [string ":lua"]:1: in main chunk

Hey!
I haven't tried it with Neogit. But I will take a look. Maybe I'll switch to Neogit myself in the end. Had you met the same error before I refactored your changes. Or maybe the changes caused this behaviour? @stevenxxiu

I can't quite remember. Looking at your changes I don't think it causes it, but I never looked at this issue before. I can take a look later.
Oh and it's Neogit with diffview that they support, to view side-by-side diffs.

Hi, the refactor commit indeed causes the bug. I'm not sure what the cause is.

I found another way to replicate the issue. Run :checkhealth, then close the window. We get:
Error detected while processing BufHidden Autocommands for "*":
Error executing lua callback: Vim:E957: Invalid window number
stack traceback:
        [C]: in function 'getmatches'
                ...y/highlight-whitespace/lua/highlight-whitespace/core.lua:103: in function 'save_matches_to_cache_and_clear'
                        ...y/highlight-whitespace/lua/highlight-whitespace/init.lua:55: in function <...y/highlight-whitespace/lua/highlight-whitespace/init.lua:51>

I made a fix to the issue that appears to work. Not sure if this fix makes sense.

I reviewed the function you referred to. I think there were a few issues with it:

There was just one check (whether a window where a buffer had been loaded before it got hidden was floating or not) and it controlled just clearmatches function and not touched caching results of getmatches
clearmatches was no parameters passed to it - what might cause issues (e.g., I don't know which window is current when this function is called)

function M.save_matches_to_cache_and_clear(args)
  local bnr = (args or {}).buf or api.nvim_get_current_buf()
  local wid = fn.bufwinid(bnr)

  if
    not api.nvim_buf_get_option(bnr, "modifiable")
    or api.nvim_win_get_config(wid).relative ~= ""
  then
    return
  end

  fn.clearmatches(wid)
  local bname = api.nvim_buf_get_name(bnr)
  M.buffer_cached_matches[bname] = fn.getmatches(wid)
  M.win_group_match[bname] = nil
end

About reproducibility with checkhealth. I wasn't able to reproduce it. Likely because we have different settings/ different set of plugins and so on. But let's check if the snippet above solves your problems. You can first try it as it is. And if it does not help, you can add pcall wrapping
@stevenxxiu

I tried your new function, and there's a new error:
E5108: Error executing lua vim/_editor.lua:0: nvim_exec2()..BufHidden Autocommands for "*": Vim(append):Error executing lua callback: ...y/highlight-whitespace/lua/highlight-whitespace/core.lua:104: Invalid window id: -1
stack traceback:
        [C]: in function 'nvim_win_get_config'
        ...y/highlight-whitespace/lua/highlight-whitespace/core.lua:104: in function 'fn'
        ...ighlight-whitespace/lua/highlight-whitespace/wrap_fn.lua:11: in function 'callback'
        ...y/highlight-whitespace/lua/highlight-whitespace/init.lua:22: in function <...y/highlight-whitespace/lua/highlight-whitespace/init.lua:21>
        [C]: in function 'nvim_exec2'
        vim/_editor.lua: in function 'cmd'
        ...re/nvim/lazy/neogit/lua/neogit/integrations/diffview.lua:20: in function 'close'
        [string ":lua"]:1: in main chunk
stack traceback:
        [C]: in function 'nvim_exec2'
        vim/_editor.lua: in function 'cmd'
        ...re/nvim/lazy/neogit/lua/neogit/integrations/diffview.lua:20: in function 'close'
        [string ":lua"]:1: in main chunk

The :checkhealth issue could be due to my bufferline plugin.

Let's check this snippet :)
function M.save_matches_to_cache_and_clear(args)
  local bnr = (args or {}).buf or api.nvim_get_current_buf()
  local wid = fn.bufwinid(bnr)

  if
    not api.nvim_get_option_value("modifiable", { buf = bnr })
    or api.nvim_win_get_config(wid).relative ~= ""
  then
    return
  end

  local ok, matches = pcall(fn.getmatches, wid)
  if ok then
    local bname = api.nvim_buf_get_name(bnr)
    M.buffer_cached_matches[bname] = matches
    M.win_group_match[bname] = nil
  end
end

Btw, which LSP do you use for lua code?
@stevenxxiu

The error is in the call to nvim_win_get_config(). So if you just want a pcall(), I think this works:
function M.save_matches_to_cache_and_clear(args)
  local bnr = (args or {}).buf or api.nvim_get_current_buf()
  local wid = fn.bufwinid(bnr)

  if not api.nvim_get_option_value("modifiable", { buf = bnr }) then
    return
  end

  local ok, config = pcall(api.nvim_win_get_config, wid)
  if not ok then
    return
  end

  if config.relative ~= "" then
    return
  end

  local bname = api.nvim_buf_get_name(bnr)
  M.buffer_cached_matches[bname] = matches
  M.win_group_match[bname] = nil
end

I use lazydev for my LSP.

The error is in the call to nvim_win_get_config(). So if you just want a pcall(), I think this works:
function M.save_matches_to_cache_and_clear(args)
  local bnr = (args or {}).buf or api.nvim_get_current_buf()
  local wid = fn.bufwinid(bnr)

  if not api.nvim_get_option_value("modifiable", { buf = bnr }) then
    return
  end

  local ok, config = pcall(api.nvim_win_get_config, wid)
  if not ok or config.relative ~= "" then
    return
  end

  local matches = fn.getmatches(wid)
  local bname = api.nvim_buf_get_name(bnr)
  M.buffer_cached_matches[bname] = matches
  M.win_group_match[bname] = nil
end

I use lazydev to set up my LSP.

Hmm, if this error is about non-existing window id, maybe it is better than just to check the returned value from fn.bufwinid?
function M.save_matches_to_cache_and_clear(args)
  local bnr = (args or {}).buf or api.nvim_get_current_buf()
  local wid = fn.bufwinid(bnr)

  if
    wid < 0
    or api.nvim_win_get_config(wid).relative ~= ""
    or not api.nvim_get_option_value("modifiable", { buf = bnr }) then
    return
  end

  local bname = api.nvim_buf_get_name(bnr)
  M.buffer_cached_matches[bname] = fn.getmatches(wid)
  M.win_group_match[bname] = nil
end

@stevenxxiu

Hmm, if this error is about non-existing window id, maybe it is better than just to check the returned value from fn.bufwinid?

Indeed that works! I didn't know about this. Thanks for telling me.

@stevenxxiu
I'm glad I was able to help you remotely, without even reproducing the problem locally, on my machine
What other issues (related to this PR) do you have? I guess the last snippet solved some of your problems. Are we close to merging PR? If yes, you can update the code (by adding also the last snippet), push the commit, and I will merge the PR

I copied over your last snippet and made a commit.
I can't see any other issues for now.


Fix error_handler return type declaration

Hi,
Small patch to fix error handler function return type.
From php.net documentation, the error_handler should return a bool and not a void (https://www.php.net/manual/en/function.set-error-handler.php).

Thanks


[TF-TRT] Fix AddN for const layer inputs in non implicit batch cases.

This fixes a crash when AddN elementwise operation is executed with constant layer as input(s) to the AddN node. In this case the Const nodes are not recognized as tensors and in those cases the batch dimension is always removed. This causes a mismatch of input tensor sizes causing the crash/native segment fallback. Only when implicit batch mode is used, the batch dimension must be removed to match the other input tensor sizes.

Replacing PR: #58474
@DEKHTIARJonathan  Please review

@poulsbo I'm not able to "approve the PR" and for some reason "Bixia" got auto-assigned.
Though the TP Job seems to have flagged the PR as one of TF-TRT PRs: https://github.com/tensorflow/tensorflow/actions/runs/3699422368/jobs/6266773452
In addition, there's an interesting message:
First-time contributors need a maintainer to approve running workflows. [Learn more.](https://docs.github.com/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks) 2 successful, 2 expected, and 1 pending checks
It's not the first time that @pavanimajety is submitting a PR or contributes to TF.

@poulsbo Anything blocking ? It's most minimalist PR I could find. Wondering why it's blocked.

@DEKHTIARJonathan Trying to move this forward on my side.


lambda_allow_npcdmg cvar not disabling NPC damage

Important NPCs always take damage from players regardless of whether lambda_allow_npcdmg is enabled or not. Taking out the attackerIsPlayer == true check fixes this (at the cost of making them completely invincible) so it seems to not be recognizing the source of damage correctly.
Additionally, temporarily essential NPCs like the one just before the manhacks on d1_canals_03 are also not affected by the cvar at all. There's currently no check for this so consider this part to be a feature request.

Thanks for the report, pretty sure it worked at some point. Some NPCs are also marked no longer important after finishing the objective, like in d1_canals_03 the NPC can be killed once the door opened without a mission fail. I will look into this.


[Beta]: docs(cn): sharing-state-between-components

@qinhua our ci failed.
please check it with yarn dev or other possible commands on your local machine to fix it.

It's weird that we need modify the tsx file.

It's weird that we need modify the tsx file.

I found the title level has been modified after compare the MD file with main branch, and that will produce some errors in Navigation.tsx.


WIP Rewrite callbacks to use async/await and Promises

Fixes a number of issues with callbacks in the current version (and a bug with ConnectionId/connectionId casing)
This is untested – I just edited in GitHub's browser 🙀
Please don't merge unless you've tested this.
I should be able to test this later today – does the CF template spin up all resources required to test this?
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks @mhart. The SAM template does include all the base needed. The only real work is create the websocket API and link the functions to it. I've just tested with 5 clients and they receive/resolve equally to what was expected.

Awesome, glad it works!
You might want to add a note in the README that this doesn't setup the websocket API or link the functions or anything – it sounds at the moment like it does everything you need.


reorganize documentation site

closes #67
To run locally:
In one terminal tab:
sbt ";project docs ;makeMicrosite"
In another terminal tab:
cd modules/docs/target/site/ && jekyll serve
In the browser go to http://<IP_ADDRESS>:4000/

Seems like a lot of things break when testing on my local.

Click Guides

Click "User Guide" on the left nav

It expands, but then the rest of the links disappear, and the top "Guides" link changes to "Main Menu"

Click "API Documentation"

In the left nav bar click "User Guide"

Same issue as above

Seems like in general the "Guides" navigation is broken.
Other notes:

I would remove Contributing and Developing all together from the main menu and docs site as they are repository / project specific.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6b36ea9). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #66   +/-   ##
=========================================
  Coverage          ?   93.76%
=========================================
  Files             ?      103
  Lines             ?     4765
  Branches          ?       75
=========================================
  Hits              ?     4468
  Misses            ?      297
  Partials          ?        0

Continue to review full report at Codecov.

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


feat: add validation for property sort order

Description of your changes

Fixes #
I have:

[x] Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

Need to classify as user error https://github.com/Azure/fleet/blob/28e5a083bab5d437888b281cf0cf1c76a32ee0f1/pkg/scheduler/framework/plugins/clusteraffinity/types.go#L269


add div and span with classes to the photo view

This is just a small tweak to the photo view to make styling easier.

Many thanks for the contribution, really appreciate it!
One of the philosophies of node-gallery is to keep the core as small & simple as possible. In general, I think the base templates don't do a great job of semantically describing their content in an easy-to-style manner, so agree things could be much better!
I've added a few comments suggesting alternatives, hopefully it doesn't seem pedantic. Please feel free to disagree, or if you're onboard with some of the alternatives I've suggested I'd be happy to merge this :-)
If you can, you might squash your commits into one? If not, don't worry!
Cheers
Cian

Hi Cian,
Thanks for your feedback! I'll rework and flatten my patch a little this weekend, bearing in mind your comments and with an eye to consistency and semantic presentation.
Best,
Adam

Closing to rebase and re-submit.

Published +<EMAIL_ADDRESS>


Added new PokeApi.NET version to the wrappers list, fixed typos in docsv2.md again

With the rebased version, as @phalt  requested (See #106).

@nimaje broken link in the table of content (was referring to e instead of é).

:+1:


Add a Constrained Pick and Place Objective

An alternative to the existing 3 Waypoints Pick and Place Objective, with constraints on the gripper.
note: this has only been added to the picknik_ur_mock_hw_config robot configuration.

@eholum I just merged main into this branch if you'd like to test again, otherwise feel free to merge!


OpenJS Foundation Sustainability Collaboration Space Meeting 2024-12-23

Time
UTC Mon 23-Dec-2024 17:30 (05:30 PM):

Timezone
Date/Time

US / Pacific
Mon 23-Dec-2024 09:30 (09:30 AM)

US / Mountain
Mon 23-Dec-2024 10:30 (10:30 AM)

US / Central
Mon 23-Dec-2024 11:30 (11:30 AM)

US / Eastern
Mon 23-Dec-2024 12:30 (12:30 PM)

EU / Western
Mon 23-Dec-2024 17:30 (05:30 PM)

EU / Central
Mon 23-Dec-2024 18:30 (06:30 PM)

EU / Eastern
Mon 23-Dec-2024 19:30 (07:30 PM)

Moscow
Mon 23-Dec-2024 20:30 (08:30 PM)

Chennai
Mon 23-Dec-2024 23:00 (11:00 PM)

Hangzhou
Tue 24-Dec-2024 01:30 (01:30 AM)

Tokyo
Tue 24-Dec-2024 02:30 (02:30 AM)

Sydney
Tue 24-Dec-2024 04:30 (04:30 AM)

Or in your local time:

https://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+Sustainability Collaboration Space+Meeting+2024-12-23&iso=20241223T1730
or https://www.wolframalpha.com/input/?i=05PM+UTC%2C+Dec+23%2C+2024+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1b3vnFdqg-vonhMar_5GGOjaB_1mMyKiCrQkheb82uQQ/edit

Agenda
Extracted from sustainability-agenda labelled issues and pull requests from the openjs-foundation org prior to the meeting.
openjs-foundation/sustainability-collab-space

Find a better meeting time #16
OpenJS + Open Source Pledge #12
Guide on Resource Allocation for JS Ecosystem Projects Post-Funding #9
Invite projects to share their experiences with us #6
Document Sponsorship Platforms and Strategies for Maintainers #5
Create a blog post to announce the collab space  #3

Invited

Ecosystem Report Collab Space Members: @openjs-foundation/sustainability-collaboration-space

Observers/Guests
Notes
The agenda comes from issues labelled with sustainability-agenda across all of the repositories in the openjs-foundation org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting
link for participants: https://zoom-lfx.platform.linuxfoundation.org/meeting/97342926119?password=39c25254-e7a3-4830-be8e-d54d63234bfb

Invitees
Please use the following emoji reactions in this post to indicate your
availability.

:+1: - Attending
:-1: - Not attending
:confused: - Not sure yet

I believe the remaining meetings in 2024 were cancelled? cc @UlisesGascon to confirm.


Firefox addon

It seems that there is no firefox addon anymore or link is wrong...

@OrionRandD whoops, sorry for that. Here it is and I fixed the readme as well.

Thx a lot. I've installed it and now everything is working fine... :)

README link appears broken again for Firefox.

@mathias ah, I updated only one of two links out of two last time, sorry!
Thanks for letting me know, fixed!

Does this still work in firefox? Link doesn't appear to go anywhere.

Sigh! Apparently it was temporary hidden by Mozilla because of some boilerplate evals generated by Webpack.
I'll try to fix this asap.

@nathanbraun ok, I think it's fixed now! Hopefully it works for you.


pull request for e2e test at 12/06/2016 11:26:02

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Class attributes not set properly within methods

First of all, I should mention I am running from web - not sure if it is on newest version. If not, may be this problem is fixed - if so, just directly close the issue.
Scenario: I've tried creating a class with a setter and getter. Afer running the class, unfortunately, the result is not correct from the transpiler, but correct from an SAP system.
Code:
class car definition.
  public section.
    methods:
      set_wheels importing i_wheels type i,
      get_wheels returning value(result) type i.

  private section.
    data:
      wheels type i.
endclass.

class car implementation.
  method set_wheels.
    me->wheels = i_wheels.
  endmethod.

  method get_wheels.
    result = me->wheels.
  endmethod.
endclass.

data mercedes type ref to car.
create object mercedes.
mercedes->set_wheels( 5 ).
write: mercedes->get_wheels( ).

ABAP transpiler returns 0. SAP system returns 5. The code is the same with only exception  I ran it in the SAP system with a start-of-selection before the mercedes data definition. Still, even if I add start-of-selection to the transpiler, the result is the same (0).

Hi,
There are some problems with code on the top level, try wrapping the code into a FORM,

root cause = https://github.com/abaplint/abaplint/issues/808

Hey Lars,
Thanks for your quick reply. Indeed, the issue is apparently not the actual setting of parameters, as demonstrated in your example.
An even simple work-around I've thought of is to simply use a main method as we would in Java. If we call the same code wrapped in the (static) main method, it's all good:
class car definition.
  public section.
    class-methods:
      main.

    methods:
      set_wheels importing i_wheels type i,
      get_wheels returning value(result) type i.

  private section.
    data:
      wheels type i.
endclass.

class car implementation.
  method set_wheels.
    me->wheels = i_wheels.
  endmethod.

  method get_wheels.
    result = me->wheels.
  endmethod.

  method main.
    data mercedes type ref to car.
    create object mercedes.
    mercedes->set_wheels( 5 ).
    write: mercedes->get_wheels( ).
  endmethod.
endclass.

car=>main(  ).

its a bug... just haven't fixed it yet, as I guess code will typically be wrapped into a FORM or METHOD

try again, latest version, abaplint bug fixed and deps updated

just tested with my original example + by changing the wheels attribute to public and directly assigning to it, both confirmed to print correct output (5)


Missing module name change

Sorry for the long wait and also thanks a lot for this quick win @lisbonjoker.


Node ip connectivity issue

I am facing 1 issue, i have a 3 node docker cluster, and 1 container running on 3rd node
When i access that container and run curl command with ip of Node1 with container port, its give me reply, when i run curl with node2 ip with container port, its giving reply, but when i run curl with Node 3 IP and port where that container is running, it not giving any response
Same issue when there is a docker single node cluster, after accessing container, when i hit curl command with node ip and container port, its not giving any response
But from outside its working fine, i am only not able to get reply, when i curl with the ip where that container is running
I have setup DIRD as systemctl service
Can you please help me on this

Hi,
Have you check this issue, how can i resolve this issue in my environment

Thanks for using DIRD. We're going to be slow replying this week due to annual leave in the team.
To start with though, we need to know how to reproduce your cluster setup. Please provide full details of how to reproduce, including of the cluster nodes, their ingress network IPs (typically in the 10.0.0.x network) the service you're running (with docker service create) and the DIRD commands you're running on each node of your cluster.

I have created a docker cluster with ip of <IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>
I have setup DIRD with with systemctl, i have not created service of that in docker
I have setup a nginx docker container, if my container is running in <IP_ADDRESS>, then i take access of that container by going to <IP_ADDRESS> VM and docker exec -it nginx /bin/sh
When i curl <IP_ADDRESS>:80 from inside that container, it gives me reply
When i curl <IP_ADDRESS>:80 from inside that container, it gives me reply
But When i curl <IP_ADDRESS>:80 from inside that container, it takes too long to respond and didn't give any response
I have created docker swarm using custom network
docker swarm init --default-addr-pool <IP_ADDRESS>/16 --default-addr-pool-mask-length 16
And created a network using this
docker network create --driver=overlay --attachable --subnet <IP_ADDRESS>/16 shared_network
And i am attaching a nginx container to shared_network

Thanks for this. I'm afraid your test case doesn't seem like a good fit for DIRD's supported functionality at first glance.
DIRD is intended to facilitate direct routing (instead of default NAT routing) for incoming public traffic addressed to a Docker service (and not attached containers) via the swarm nodes' public IPs.
In your example, you have created a swarm but no service; attaching the nginx container to the overlay network isn't a currently supported mode of operation for DIRD. If you wish, you could try using docker service create to create an nginx service instead, that publishes port 80. That should certainly work.
Also in your example, it seems you are running curl from inside a container. Doing that isn't supported, as the point of DIRD is to directly route public traffic. Once you've launched an nginx service, you could certainly try running curl directly either (a) from outside the cluster (addressing any cluster node's public IP) or (b) from any cluster node (but not from within a container).
Please also make sure that your DIRD install command that you're running from systemd specifies the  ingress network IPs of all three nodes. For avoidance of doubt, these IPs are not the <IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS> IPs, but should be IPs of each node on the <IP_ADDRESS> overlay network.
Please also make sure DIRD is running with the same command on all three nodes, and with the --preexisting option.
I hope that helps.


expose Headers::new and Extensions::new

Other users than http-rs might want to use these types, but they cannot currently be constructed outside of this crate

needs a rebase for clippy

needs a rebase for clippy

I'm okay with this change, but think we should document that this is most likely not what you want to do. Someone coming from hyperium/http may incorrectly assume that Headers instances need to manually be created. Instead this really only serves as an intermediate type that is meant to be accessed through Request, Response, etc. Much like a trait is, but slightly different.
Just exposing this without guidance may cause confusion, and we should try and evade that if possible. (Though it'd be correct to point out that our current guidance on these types isn't great either, and folks may already be confused. But ideally we shouldn't add to that 😅 )

I'm okay with this change, but think we should document that this is most likely not what you want to do. Someone coming from hyperium/http may incorrectly assume that Headers instances need to manually be created. Instead this really only serves as an intermediate type that is meant to be accessed through Request, Response, etc. Much like a trait is, but slightly different.
Just exposing this without guidance may cause confusion, and we should try and evade that if possible. (Though it'd be correct to point out that our current guidance on these types isn't great either, and folks may already be confused. But ideally we shouldn't add to that 😅 )

Oh woah, the impl actually worked! Added a test for it too:
#[cfg(test)]
mod test {
    use super::*;

    #[test]
    fn header_from_strings() {
        let strings = ("Content-Length", "12");
        assert_eq!(strings.header_name(), "Content-Length");
        assert_eq!(strings.header_value(), "12");
    }
}

Oh woah, the impl actually worked! Added a test for it too:
#[cfg(test)]
mod test {
    use super::*;

    #[test]
    fn header_from_strings() {
        let strings = ("Content-Length", "12");
        assert_eq!(strings.header_name(), "Content-Length");
        assert_eq!(strings.header_value(), "12");
    }
}

I think this could be useful if you could set the headers when constructing a request, at least that would make per-client headers in Surf config a tad nicer under the hood.

I think this could be useful if you could set the headers when constructing a request, at least that would make per-client headers in Surf config a tad nicer under the hood.

why not using Middleware?

Unimportant implementation detail, but feature wise, convenience.


Error when run in ios

when i run corodva app in ios using by xcode , it show error in Other Sources > main.m , Thread 1: signal SIGABRT .
we only install cordova-admob-free plugin

looks like a change in ads sdk
https://developers.google.com/ad-manager/mobile-ads-sdk/ios/quick-start#update_your_infoplist
Update your Info.plist
Declare that your app is an Ad Manager app by adding the GADIsAdManagerApp key with a boolean value YES to your app's Info.plist. You can make this change programmatically:
GADIsAdManagerApp

Or, edit it in the property list editor:
Warning: This step is required as of Google Mobile Ads SDK version 7.42.0. Failure to add add this Info.plist entry results in a crash with the message: "The Google Mobile Ads SDK was initialized incorrectly."

as a workaround if you don't want to add it manually , you can add this to your config.xml file so that every time your project will be updated automatically:
<platform name="ios">         <config-file parent="GADIsAdManagerApp" platform="ios" target="*-Info.plist">             <true />         </config-file>     </platform>
I guess this now needs to be added to this plugin itself so that config file updated by the plugin


Find a good use of i2gw.InputResources or deprecate it

What would you like to be added:
We need to revisit the decision to use i2gw.InputResources struct or deprecate it altogether.
Why this is needed:
Currently this struct only holds a list of Ingresses, I couldn't think of a reason we will need other resources as input resources.
I think this was added originally to be able to support CRD propagation to providers but since we move to a "fetch and store" per provider logic we need to revisit whether this is needed.
Another sub task that comes with it is changing ToGatewayAPI function signature  on the Provider interface as the input resources are no longer in use after https://github.com/kubernetes-sigs/ingress2gateway/pull/112

[ ] Find a good use of i2gw.InputResources or deprecate it
[ ] Remove input resources from ToGatewayAPI function.

I can't think of a reason either now that we're fetching the resources at the provider level. Let's go ahead and remove it to keep things clean.
/assign

Hi, I'm interested in this issue.
/assign

I feel the same way about i2gw.InputResources, we could just pass the list of Ingresses instead of wrapping it with i2gw.InputResources again.


Populate tt.extern_elementwise conversion with trigonometric and hyperbolic functions

Populate conversion pattern implementations with recently added functions to MLIR math dialect

inverse trigonometric functions (asin, acos, atan)
hyperbolic functions (sinh, cosh, tanh)
inverse hyperbolic functions (asinh, acosh, atanh)

LGTM! Let's merge this in and once this lands we can update all the lit tests at once to remove libdevice

OK

@aobolensk Just need you to update with latest main and we're good to go.

@aobolensk Just need you to update with latest main and we're good to go.

Done


nixos/mpd: conditionally provision required directories with StateDirectory

Motivation for this change

current state isn't very reproducible when stateDir deviates from the expected value (/var/lib/mpd)... also we don't have the ability to understand what the filesystem looks in order to make the right choices

maybe the user has network mounted drives we don't know about
maybe the user is using btrfs subvolumes
maybe something else you never considered

modules make too many assumptions about stateDir, resulting in various permissions issues:

https://github.com/NixOS/nixpkgs/issues/80827
https://github.com/NixOS/nixpkgs/issues/81910
https://discourse.nixos.org/t/nixpkgs-policy-as-for-systemd-prestart-setup-scripts-vs-systemd-tmpfiles/5839/7

when given arbitrary paths systemd.tmpfiles.rules generally doesn't behave as most users would expect

[ "d /a foo foo" "d /a/b/c foo foo" ] produces an error (rightfully so) because /a/b would be owned by root:root

in conclusion: it is great when NixOS "just works" because of defaults, but in the case of user customization we should push some of the work onto the user for maximum flexibility

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

OK I think I got convinced that StateDirectory is better for the simple cases of NixOS' modules. Quoting systemd.exec(1):

tmpfiles.d(5) provides functionality that overlaps with these options. Using these options is recommended, because the lifetime of the directories is tied directly to the lifetime of the unit, and it is not necessary to ensure that the tmpfiles.d configuration is executed before the unit is started.

And I also think it won't affect existing users that do nothing besides using the defaults. It didn't even affect me which I was using tmpfiles rules on these directories.


Gift Entry: Editing Batch Info Clears Batch Number field if it’s been added to a Page Layout

Issue Summary:
If a customer has enabled the automatic Batch Number feature for Gift Entry, the value may be cleared from an NPSP Data Import Batch record when clicking the Edit Batch Info button and saving. This seems to occur if the Batch Number field has been added to a Page Layout for the NPSP Data Import Batch object.
Root Cause
In the handleSave() method of geBatchWizard.js, we set the value of dataImportBatchCreateDefaults via the getRecordCreateDefaults LWC method. (https://developer.salesforce.com/docs/component-library/documentation/en/lwc/reference_wire_adapters_create_record_values). This returns object metadata associated with the default Page Layout. If a customer has added the Batch Number field (npsp__Batch_Number__c) to the page layout, we retrieve it and it is reset to a default value of null, despite being set to a value previously.
Steps to Repeat:

Enable Advanced Mapping and Gift Entry in NPSP Settings.
Save and Activate Batch Numbering (Display Format = BTC-{00000}; Starting Number = 1)
Go to Setup —> Object Manager —> NPSP Data Import Batch —> Page Layout —> NPSP Data Import Batch. Remove the Batch Number field if it’s currently on the Page Layout. Save.
Go to the Gift Entry Tab.
On the Batches Tab, click the cog and Select Fields To Display. Add the Batch Number field.
Click the New Batch button.

Choose the default template and click Next.
Name = Test Batch 1. Click Next.
Leave the defaults on the Set Default Values screen. Click Save.
Go back to the Gift Entry Tab and confirm that your batch has a Batch Number in the batches table (should be BTC-00001).
Click on the Batch again.
Click the Edit Batch Info button. Click Next and Save (no need to change anything).
Go back to the Gift Entry Tab and confirm that the Batch Number is still present and is still BTC-00001.

Now add the Batch Number field to the NPSP Data Import Batch Page Layout in Setup —> Object Manager —> NPSP Data Import Batch —> Page Layouts.
Go back to the Gift Entry Tab.
Click on your Batch and click the Edit Batch Info button. Click Next and Save (no need to change anything).
Go back to the Gift Entry Tab an confirm that your Batch Number has now vanished.

Workaround:
Remove the Batch Number field from your Page Layouts on the NPSP Data Import Batch object.

W-8909629