Too many times of Rejoining group lead to Maximum call stack size exceeded

In the beginning, it is confuse me when i open my laptop the error down here was always in the console
RangeError: Maximum call stack size exceeded

But, just now , when i rejoin the wifi network, the error reappear again.
After couple times testing, I am sure that the rejoin would cause the error.
There are some console info.
2016-03-04T08:53:59.064Z INFO group-name-56d94c8a737bb0fb808ade52 Joined group group-name generationId 3 as group-name-56d94c8a737bb0fb808ade52-2fc5cefc-c9b4-41d6-91e0-438075a26395
2016-03-04T08:53:59.382Z ERROR group-name-56d94c8a737bb0fb808ade55 Sending heartbeat failed:  KafkaError: UnknownMemberId: Returned from group requests (offset commits/fetches, heartbeats, etc) when the memberId is not in the current generation.
{ [KafkaError: Returned from group requests (offset commits/fetches, heartbeats, etc) when the memberId is not in the current generation.]
  name: 'KafkaError',
  code: 'UnknownMemberId',
  message: 'Returned from group requests (offset commits/fetches, heartbeats, etc) when the memberId is not in the current generation.' }
    at Object.exports.byCode (/path/to/program/node_modules/no-kafka/lib/errors.js:84:12)
    at _Reader.Protocol.define.read (/path/to/program/node_modules/no-kafka/lib/protocol/common.js:87:23)
    at _Reader._read (/path/to/program/node_modules/bin-protocol/lib/reader.js:41:18)
    at apply (/path/to/program/node_modules/bin-protocol/node_modules/lodash/lodash.js:419:27)
    at _Reader.wrapper [as ErrorCode] (/path/to/program/node_modules/bin-protocol/node_modules/lodash/lodash.js:4499:16)
    at _Reader.Protocol.define.read (/path/to/program/node_modules/no-kafka/lib/protocol/group_membership.js:275:14)
    at _Reader._read (/path/to/program/node_modules/bin-protocol/lib/reader.js:41:18)
    at apply (/path/to/program/node_modules/bin-protocol/node_modules/lodash/lodash.js:418:27)
    at _Reader.wrapper [as HeartbeatResponse] (/path/to/program/node_modules/bin-protocol/node_modules/lodash/lodash.js:4499:16)
    at /path/to/program/node_modules/no-kafka/lib/client.js:541:61
From previous event:
    at /path/to/program/node_modules/no-kafka/lib/client.js:540:40
    at processImmediate [as _immediateCallback] (timers.js:383:17)
From previous event:
    at Client.heartbeatRequest (/path/to/program/node_modules/no-kafka/lib/client.js:531:48)
    at GroupConsumer._heartbeat (/path/to/program/node_modules/no-kafka/lib/group_consumer.js:201:24)
    at /path/to/program/node_modules/no-kafka/lib/group_consumer.js:209:25
    at Timer.listOnTimeout (timers.js:92:15)
[at apply (/path/to/program/node_modules/no-kafka/node_modules/lodash/lodash.js:419:27)]
2016-03-04T08:54:00.060Z INFO group-name-56d94c8a737bb0fb808ade52 Rejoining group on RebalanceInProgress
/path/to/program/node_modules/bluebird/js/release/promise.js:533
Promise.prototype._settlePromise = function(promise, handler, receiver, value) {
                                           ^

RangeError: Maximum call stack size exceeded
    at Promise._settlePromise (/path/to/program/node_modules/bluebird/js/release/promise.js:533:44)
    at Promise._settlePromise0 (/path/to/program/node_modules/bluebird/js/release/promise.js:605:10)
    at Promise._settlePromises (/path/to/program/node_modules/bluebird/js/release/promise.js:684:18)
    at Promise._fulfill (/path/to/program/node_modules/bluebird/js/release/promise.js:629:18)
    at Promise._resolveCallback (/path/to/program/node_modules/bluebird/js/release/promise.js:424:57)
    at Promise._settlePromiseFromHandler (/path/to/program/node_modules/bluebird/js/release/promise.js:515:17)

Doesn't know it is related about he newest change log of<EMAIL_ADDRESS>which have said something about stack overflow.

It happens for me as well, but it only happens after laptop hibernation. I'll try updated bluebird.

It still happens with 3.3.3
I can hardly replicate it when I close my laptop while some group consumers are running and then when I open the lid some of the consumers may fail with call stack error in bluebird.
I'm not able to replicate it when I stop/start kafka server, even for 300 times.
For now I think it happens when the system clock is skewed.

Can you try https://github.com/oleksiyk/kafka/commit/f597761de17d92a9ac14ca501e4fa0f3ceea998a ?

@oleksiyk doesn't use yet, but should solve the problem.


Support MERGE SQL command

What is MERGE?
"A relational database management system uses SQL MERGE (also called upsert) statements to INSERT new records or UPDATE existing records depending on whether condition matches. It was officially introduced in the SQL:2003 standard, and expanded in the SQL:2008 standard."
Some DBMSs are currently supporting the MERGE command:
PostgresSQL: https://www.postgresql.org/docs/current/sql-merge.html
SQL Server: https://learn.microsoft.com/en-us/sql/t-sql/statements/merge-transact-sql
Oracle: https://docs.oracle.com/database/121/SQLRF/statements_9017.htm
Example from PostgresSQL:
"Attempt to insert a new stock item along with the quantity of stock. If the item already exists, instead update the stock count of the existing item. Don't allow entries that have zero stock.".
MERGE INTO wines w
USING wine_stock_changes s
ON s.winename = w.winename
WHEN NOT MATCHED AND s.stock_delta > 0 THEN
  INSERT VALUES(s.winename, s.stock_delta)
WHEN MATCHED AND w.stock + s.stock_delta > 0 THEN
  UPDATE SET stock = w.stock + s.stock_delta
WHEN MATCHED THEN
  DELETE;

This one is like the mothership of mutation queries ain't it? 😆

Now that mssql dialect is in the core, this query is the only way, I know of, to perform upserts in this engine, outside of if-else in what seems to be stored procedures.


Discovery/Connection impossible

When I used this demo one year ago (with Kitkat and Lollipop) everything was ok, but not now.
I'm using a Nexus 5 (6.0.1) and Nexus 6P ( N preview 1). From Settings app I can see and connect two devices.
When I start the discovery on these two devices (from this demo app) I can't see any devices and obviously I can't start the connection procedure. Why?
It's a bug on Nexus 6p android n preview 1 related to bluetooth? Or this demo is broken on Android 6.x and N?

The Android M and N use new permission system , so you need to modify the app to request permission from the user.
Check here for more

I modified very quickly this example to add runtime permissions and It seems that you are right.
Obviously I didn't request Bluetooth as runtime permissions, but COARSE and FINE LOCATION to be able to establish a connection via bluetooth. This is absolutely weird, that to be able to use this app I must use two other permissions not strictly related to bluetooth. I read on the web the reasons...and ok, I accept that :) But, I think that this example requires an urgent update, because it's useless as it is.

I forked this repo and pushed two commits, the first to add runtime permissions and the second one to fix a bug existing also in this repo (when you click on "No devices have been paired" the app will crash because there is an exception that must be catched).
If someone is interested I can create a pull request.
I'm looking forward to receiving your reply soon.
Thank you,
Stefano

Ok i created the pull request... if you are not interest simply ignore me :).

Anyone having  issues with the crash for clicking on no devices paired you can use this so that it doesn't crash and it opens bluetooth settings to pair a new device:
/**
     * The on-click listener for all devices in the ListViews
     */
    private AdapterView.OnItemClickListener mDeviceClickListener
            = new AdapterView.OnItemClickListener() {
        public void onItemClick(AdapterView<?> av, View v, int arg2, long arg3) {
            // Cancel discovery because it's costly and we're about to connect
            mBtAdapter.cancelDiscovery();

            // Get the device MAC address, which is the last 17 chars in the View
            String info = ((TextView) v).getText().toString();
            if(((TextView) v).getText().toString().matches(getResources().getText(R.string.none_found).toString())){
                Intent intentOpenBluetoothSettings = new Intent();
                intentOpenBluetoothSettings.setAction(android.provider.Settings.ACTION_BLUETOOTH_SETTINGS);
                startActivity(intentOpenBluetoothSettings);
            }
            else {
                String address = info.substring(info.length() - 17);
                // Create the result Intent and include the MAC address
                Intent intent = new Intent();
                intent.putExtra(EXTRA_DEVICE_ADDRESS, address);
                setResult(Activity.RESULT_OK, intent);
                // Set result and finish this Activity
            }

            finish();
        }
    };


Fix missed version bump for samples

This should fix the Travis-CI build.

One of y'all want to review this easy one?

LGTM


App crashes in older iphones with no nfc

I added the package to my app, it works fine from iphone 7 and up but it crashes the app at startup in iphones older then iphone7 which don't possess nfc.

Hi!
I am not able to reproduce this as I only have a iPhone 8 which does support NFC.
Even running the example app in the iOS simulator (iPhone 7 and 8) doesn't cause it to crash.

Hello everyone,
you can't reproduce the NFC related crash on a simulator,
the same way you can't use the camera on the simulator.
The crash happens on any device with a iOS version lower than 13.
I hope this helps.
BR

Well I don't have any devices running iOS <13 so I can't test it at all.
If any of you do, please send the crash stack trace or similar.

Hello,
the thing is there is no stack. The app crashes instantly.
The only thing you can see is a flash of the app being opend and closed in the same instant.
Had a similar issue with another dependency that was failing the app in release mode.
The only thing you see in the console after installing the app is :
"Connection closed before full header was received"
I'm sorry I don't have more info to share.
BR

Perhaps it is because the CoreNFC framework is not weakly linked?

It's an issue with the CoreNFC framework not being available on older devices. I was testing on my iPhone 5 running iOS 10.3.3 and got this error:
dyld: Library not loaded: /System/Library/Frameworks/CoreNFC.framework/CoreNFC
  Referenced from: /var/containers/Bundle/Application/41338648-F28E-4864-900B-BB90F9DE5F4D/Runner.app/Runner
  Reason: image not found

After a bunch of searching through StackOverflow and some trial and error, I was able to solve this. Add the CoreNFC.framework in General -> Frameworks, Libraries, and Embedded Content. Set the Embed value to "Do Not Embed". Then, head over to Build Phases -> Link Binary With Libraries, and find (or add) CoreNFC.framework and mark the status as "Optional".
I also had to clean and rebuild about 100 times. But I can confirm it works when built for physical devices (on an iPhone 5, and on an iPhone 8).
I'd suggest adding this to the FAQ or setup, unless there's some magic way you can tell the plugin to set it up like this. The plugin works great by wrapping everything around "NFC.isNDEFSupported", so you wouldn't expect a hard iOS 13 and up dependency.

I just merged PR #28 which changed the Podspec file to have the CoreNFC framework be a weak framework instead.
@rohnsht does this fix the issue for you?

Can any of you confirm if this is fixed or not with #28?
dependencies:
  nfc_in_flutter:
    git:
      url: git://github.com/semlette/nfc_in_flutter.git
      ref: 4c1b73f5d024fac7d25ac909f5d47796a53e5e3e

I am going to assume #28 fixed it.
I am closing the issue, but will reopen it again if it didn't actually fix it.

@martyfuhry how did you even manage to be able to offer this app for iPhone 6s and older devices? We are struggling to make it available on older phones since it always say iPhone 7 and never in the store. Cant even find a setting to specify devices... only iOS versions...

@martyfuhry how did you even manage to be able to offer this app for iPhone 6s and older devices? We are struggling to make it available on older phones since it always say iPhone 7 and never in the store. Cant even find a setting to specify devices... only iOS versions...

I struggled. And now I've switched to nfc_manager. I think this project has been abandoned and I haven't had any trouble with nfc_manager.


0x72 not working as fn-key

I tried to assign fn key functionality to key 0x72 ("help" says Karabiner Event Viewer), but it does not work for my wasd ISO 124 keyboard on macOS Sierra 10.12.6 (can't use f-keys as F1-F12 while pressing fn/help).
"simple_modifications": {"help": "fn"}
Use all F1, F2, etc. as standard keys is not activated.
Thank you for any help!

Changing 0x72 to anything doesn't seem to work. I tried to assign it to volume_increment but the event viewer still records it as help. (I'm also using a WASD keyboard.)

Had this problem too, use "insert" instead of "help" on the simple mapping which actually maps to 0x72.

Sure enough, that does the trick. Thanks @bencart!


correct user initialize format

@cpreid

🚀 Deployed on https://65551ba91c002d4edf5bea51--cozy-fox-0defec.netlify.app


Add new Factuality test

Follow this approach: Llama 2 vs GPT.
The idea is to give the LLM model two texts, one which is factual and accurate and one that is not, and the model should say which one is the factual one.
Check the link to see if the model data is available (prompt and texts), otherwise create an issue to build this kind of data in future.

Factuality test Data: Dataset
Factuality test prompts used for experiment: Prompts used
Evaluation Notebook : Notebook


Apple Music not working on macOS Sonoma

The menu item doesn't detect when I play music from Music on macOS Sonoma.

Could you provide a video of it not showing, the application that you're streaming from, and some more explanations when it happened?

this doesn't seem right for now, so if you have any issues, please give more details and I'll be reopening this, thanks.


Update README.md to reflect an Ubuntu PPA

I figured that since @amitojsingh366 would like to disable updating on Linux (https://github.com/benawad/dogehouse/pull/1690), then an Ubuntu repository would be critical. I couldn't figure out for the life of me how to use launchpad.net unfortunately, however I did figure out how to self-host my own PPA very easily with my RPi at home. So I updated the README to include this new way to install dogehouse

Also, if anyone has control over the dogehouse.tv domain, someone could make a ppa.dogehouse.tv sub-domain and makes an A record pointing to <IP_ADDRESS>. That way the ppa would look more legitamite than just ppa.palomagit.ml


SetTimer does work in side step

Describe the bug
setInterval() does not seem to be working from steps.
Same code when I tried without gauge works .  I have created a project to demo this. the project actually has to print "hello world"  every 1 sec for a duration of 5 seconds . But it does not wait for setInterval. If we execute the code without
Here is the project -https://github.com/bimalviswam/gaugeTimer
What command(s) did you run when you found the bug?
to run with gauge - Run "npm install" and then "gauge run specs"
to run without gauge - cd into the project and then  "node ./tests/withOutGauge.js"

### Output:

setInterval function is not respected and gauge completes the execution without any out put

#### Gauge (Output of `gauge -v`)

Gauge version: 1.0.6

Plugins
-------
html-report (4.0.8)
js (2.3.6)
json-report (0.3.2)
screenshot (0.0.1)

#### Node.js/Java/Python/.Net/Ruby version
node - 12.14.4
#### Operating System information
Mac Mojave
Version 10.14.6

#### IDE information
VsCode 1.46.1

answered here: https://spectrum.chat/gauge/general/setinterval-from-step-is-not-respected~7ee1391b-9288-41d0-a340-cd2a0792e67e?m=MTU5NDgzNjMzNjcwOA==


Initial PR to gather feedback.

Modify ByteBufferReadableStreamChannel to take in a list of ByteBuffer (#394).
Extending ByteBufferReadableStreamChannel to take in a List of ByteBuffer.

Updating existing constructor to call into the new constructor.
Updating class member ByteBuffer buffer to List<ByteBuffer> buffers
Updating readInto() to process a List of ByteBuffer

TODO

add/update tests.

Addressed the following

Marked size as final
Added documentaion to computeSize, using buffer.remaning()
Removed lock mechanism
Removed explicit null checks in writeContent and computeSize
Removed appending null as last block

@moontails will review this today or tomorrow. Sorry for the delay

@vgkholla no problem and thank you!
I have also added a testcase for the corner cases involving the creation of ByteBufferReadableStreamChannel.
Since we are effectively calling the second constructor from the first, I was thinking the existing tests would do as they are now run on a list of ByteBuffer. What do you think?

@moontails The existing tests would run on the new constructor but they wouldn't test list specific use cases - for e.g., only the last ByteBuffer in a list should go in with isLast as true. Since you never have more than one item in the list in the current test cases, there is no opportunity to test the code paths that occur when isLast is false.

@vgkholla I apologize for the delay in making the requested changes. Have split the original test into

ByteBufferReadableStreamChannelTest that contains only the corner cases tests
ByteBufferReadableStreamChannelParameterizedTest that contains the other test cases to test against different ways of creating a ByteBufferReadableStreamChannel

Hey - I wish to take a quick look, could you hold off merging for now?
On May 2, 2017 10:05 PM, "Gopalakrishna Holla"<EMAIL_ADDRESS>wrote:

@vgkholla approved this pull request.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/linkedin/ambry/pull/492#pullrequestreview-35940248,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYzQ17o8atNHbuDXVdq4WawOsLqlEeuks5r2ArtgaJpZM4KyKMD
.

Hi @pnarayanan did you get a chance to review?

@moontails this looks mostly good to me, once you address these last comments this can be merged.
Please also rebase this PR with the master branch.

Closing this due to inactivity. If we decide to take this change up again, a new PR may be opened using this one as a reference.


Dynamodb expressions parser

This PR improves condition expression support for dynamodb2. It should hopefully fix #1819, #1587, and #1711.
The essential changes are:

Update the parsing code for filter expressions to an OOP design that is hopefully more readable and reliable
Add some test cases for condition expressions that were previously not implemented
Support nested attributes in List and Map types.
Switch Expected to use the code in comparisons
Switch ConditionExpression to use code in comparisons as well

The remaining TODO items are

[x] Finish adding test cases
[ ] Finish documenting
[x] Update uses of super() to support python 2
[ ] DRY up the code for DynamoType and AttributeValue

Thanks for doing this work @mjstevens777!  I'd be happy to pull it down and test it with some use cases I have.  Is it in a state where I can do that?

Coverage decreased (-0.3%) to 92.669% when pulling c2c32a7731f561d773b50cd4f6c6fd945c718a7d on mjstevens777:feat/dynamodb-expressions into 66dafc6679e979c8ad102c0a782d9731179945c6 on spulec:master.

Coverage decreased (-0.3%) to 92.669% when pulling c2c32a7731f561d773b50cd4f6c6fd945c718a7d on mjstevens777:feat/dynamodb-expressions into 66dafc6679e979c8ad102c0a782d9731179945c6 on spulec:master.

@killthrush It should be ready to use. The tests are now passing for both python2 and python3.

Cool.  So I got this to work with several use cases.  There was on particular one related to attribute names that I couldn't get to work - this case should raise an exception if I run it more than once but it doesn't:
new_state = 'starting'
result = table.update_item(Key={'job_id': job_id},
                           UpdateExpression='SET #new_state=:new_state',
                           ConditionExpression='attribute_not_exists(#new_state)',
                           ExpressionAttributeNames={
                               '#new_state': new_state,
                           },
                           ExpressionAttributeValues={
                               ':new_state': 123,
                           },
                           ReturnValues='ALL_NEW')

If I do this, then the ConditionExpression does work as expected and the mock raises an exception:
new_state = 'starting'
result = table.update_item(Key={'job_id': job_id},
                           UpdateExpression='SET #new_state=:new_state',
                           ConditionExpression=f'attribute_not_exists({new_state})',
                           ExpressionAttributeNames={
                               '#new_state': new_state,
                           },
                           ExpressionAttributeValues={
                               ':new_state': 123,
                           },
                           ReturnValues='ALL_NEW')

I'm pretty sure that attribute names should be honored in these expressions, is that your understanding as well?

@killthrush It looks like there were no tests for update_item with ConditionExpression, so that was a helpful thing to point out. But it looks like the tests are correctly throwing errors with update_item, attribute_not_exists, and ExpressionAttributeNames, so I'm not sure what the issue would be.

🤔 Interesting.  It's still failing for me over here with 6ae3ac6.  I'll fork your repo and see if I can reproduce it there.

OK, reproduced in the test fixture with the following:
    # Test nonexistent top-level attribute.
    with assert_raises(client.exceptions.ConditionalCheckFailedException):
        table.update_item(Key={
            'forum_name': 'the-key',
            'subject': '123'
            },
            UpdateExpression='set #new_state=:new_state, #ttl=:ttl',
            ConditionExpression=f'attribute_not_exists(#new_state)',
            ExpressionAttributeNames={'#new_state': 'foobar', '#ttl': 'ttl'},
            ExpressionAttributeValues={':new_state': 'some-value', ':ttl': Decimal('12345.67')},
            ReturnValues='ALL_NEW'
        )

@mjstevens777 are you still working on this or is it fair game for me to pick it up?

I’m not really working on it. That would be great, thanks!

Closing in favor of https://github.com/spulec/moto/pull/2266


Update master branch

@mtaillefumier this is what I mean.  I'm puzzled. 🧐🤔

The last commit dates back from Oct. 5th in master. I also have this commit locally. So anything before that should not be showing.

Exactly! I think the issue is with squash commits in develop, but I can't prove it. I started a new main branch based on current develop. Should be good enough. I will try to keep main updated more frequently. It all goes well, we will keep it and remove master.


Error run the openfast_win32orwin64_debug compiled by VS2017

Dear everyone,
I have recompile the OpenFAST (Debug,win32) with VS2017 and succeed without any errors. But when I open the OpenFAST_win32_debug, the  error happened shows :'Can not find the mkl_sequential.dll '  ,but when i run the release version, there are no problems ,It seems like  only the debug version have the problem .  Could you help me solve the problem?
Best regards.

Did you run this from cmd.exe or from the Intel Parallel Studio command prompt?

when i use the cmd.exe ,the debug version shows the same error like above ,But when i use the Intel Parallel Studio command prompt ,if the openfast windows version and  Parallel Studio version is same ,running is normal . if the version is not identical (openfast-WIN32 with IPS IA-64) ,the error 0xc000007b showed.

This sounds like the expected behavior. If you're expecting to run OpenFAST compiled with the 64-bit Intel tools, you should use the 64-bit Parallel Studio command prompt. Same for 32 bit.

I have understand.But I have another question .If I copy the openfast_debug
.exe to another computer without the IPS, I have to run the openfast with the cmd.exe . Then how to solve the problem about  MKL?

Actually, @bjonkman doesnt the VS project included with OpenFAST statically link the intel libraries? Is it required to run in an Intel cmd or install the redistributable libraries?

It does look like the VS project is using statically linked libraries, so I'm not exactly sure why you would have that issue with the debug version of OpenFAST:

I personally do use dynamic libraries for the debug version, though, because there is a bug with using static debug libraries and math kernel libraries in the version of Intel Fortran I am using. Maybe you have a similar issue?
Regardless, if you want to use the dynamic libraries on a different machine, you'll have to find the version of the Intel Redistributable libraries that corresponds with the compiler version you are using and install that wherever you want to run it from the cmd.exe prompt.

Thank you for your reply, It helps me  a lot


🚧Setup CLA Assistant

Complete The Item Below

[X] I have updated the title without removing the 🚧 emoji.

Description
Setup (CLA Assistant )for the project.
This will help manage the signing of the license agreements using PRs for all contributors.
Acceptance Criteria

[ ] CLA assistant is set up
[ ] Create issues to set up CLA Assistant for projects below:

[ ] CASL
[ ] VersionMiner
[ ] BranchValidator

ToDo Items

[X] Change type labels added to this issue.  Refer to the Change Type Labels section below.
[X] Priority label added to this issue.  Refer to the Priority Type Labels section below.
[X] Issue linked to the correct project (if applicable).
[ ] Issue linked to the correct milestone (if applicable).
[ ] Draft pull request created and linked to this issue (only required with code changes).

Issue Dependencies
No response
Related Work
No response
Additional Information:
Change Type Labels

Change Type
Label

Bug Fixes
🐛bug

Breaking Changes
🧨breaking changes

New Feature
✨new feature

Workflow Changes
workflow

Code Doc Changes
🗒documentation/code

Product Doc Changes
📝documentation/product

Priority Type Labels

Priority Type
Label

Low Priority
low priority

Medium Priority
medium priority

High Priority
high priority

Code of Conduct

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

@CalvinWilkinson  Please research CLA Assistant to collect more info for other issues to be created

Simple simple researching revieled that having a CLA is not required.
For more info, refer to this from Ben Balter who is the Chief of Staff for Security at GitHub.


Allow only valid email address with @ in email subscription

I saw a case where email field contains a string without "@" and a cron job was failing due to it. Check if we verify email address correctly.

I would like to work on this bug. I started off with cloning pf, but then stalled on where to start. While I'm comfortable with normal python, I'm new to django & google app engine. Can you please provide me some pointers to  how to start? Is there any bootstrap & configure script to start / set-up?

Hello @kspviswa ,
It's great to hear that you are interested in development of Person Finder. Here is a GettingStarted doc:
https://github.com/google/personfinder/wiki/GettingStarted
Let me know if anything is unclear.

@kspviswa  I would also like to contribute, The situations same on this end comfortable with python, I was hoping maybe we can work togather.

@Aloknayan Welcome to pf. To get started please look @ gettingstarted link as suggessted by @gimite . BTW I've started working on this issue. I would suggest you to work on some other issue, to prevent our efforts being duplicated.
@gimite Is there any way, to assign the issues to ourselves ? Something liking picking ?

@kspviswa You can just write your interest here as you have already done.

Thank you very much kspviswa@ for the great contribution. The change is going to be merged to the default branch and pushed at the next server deployment (hopefully very soon).

@skywhale Thanks for your helpful reviews and motivation. @gimite Thanks for pointing to quick start link, which enabled be to start quickly .
BTW could you help me in selecting next bug? I'm eager to contribute more :)

@skywhale  @gimite Can this bug be closed?

Actually the original bug says that I once found a record with an invalid mail address in the data store in the server. Since you pointed out that there was already a validation in the server side, it shouldn't have happened. I will close this bug after verifying that there is no other data flow to put the address without validation.

It's possible to enter an invalid email address for the found person (email_of_found_person). I couldn't find any code that tries to send emails to that address, so I'm not sure it could have caused a cron job to fail, but it's been 2.5 years, so maybe there used to be code that needed it to be valid.

It seems like a deliberate decision was made to only validate email addresses when they're used for subscriptions (the email validation code is within the subscription class). I'll check with Pete or Nobu to see if there's any product reason not to validate non-subscription emails.

I'm not sure it could have caused a cron job to fail, but it's been 2.5 years

As far as I know, we only used subscription email to send emails.

I think I figured this out: when records are deleted, messages are sent to associated emails (app/delete.py, line 74). Those emails are validated client-side, but not server-side.


Tile requests for GLCs to portable server should return 404 if no tile and outside of layer KML region

Note that this issue is to change the behavior of portable server.  There is a separate ticket to change the behavior of GEE server at #1578 .
Describe the bug
Tile requests to a layer in a GLC will return a tile even if there is no packet at the requested location.  This causes imagery layers to be obscured in the GEE map viewer.  The user can deselect the obscuring layer in the map viewer to see the other imagery layer.
I don't know the details but the server must be manufacturing a tile when no tile exists for the request.  I suspect it's using a lower resolution tile and then extracting a section of that tile to satisfy the higher resolution request.  If instead, the server did not return a tile then that might allow other imagery layers to show in the map viewer.  To be honest I don't know what the map viewer would do with a 404 return from a tile request.
This new behavior should only happen if the request is outside the layer's KML region.  If the request is for a tile that touches or is contained in the layer's KML region then the existing behavior should continue to be used.
Additionally, this new behavior should be specific to GLCs.
To Reproduce
Build GLMs of different areas.  I used the SanFran tutorial data and USGS Orlando imagery.  Assemble the two into a GLC and view the GLC in the GEE map viewer.  It depends on the order of the layers but with both of them enabled one of them will be obscured.  Deselect the imagery layer that is visible and the other will become visible.
Expected behavior
Both imagery layers should be visible in the map viewer without having to deselect one of them.
Desktop

Server OS: Ubuntu 16
Client OS: Windows 10
Browser chrome
Version 5.3.1

Note that the same behavior can be observed with GLCs containing multiple globes.  While testing a GLC containing two GLBs, one of the San Francisco tutorial imagery and the second of European imagery, I found that in the Earth Client I had to turn off the European imagery to be able to see the San Francisco imagery.


getsize is deprecated and should be removed

I am trying to run
geogif.gif(alt_ds.sla).compute()
So make a gif of an Xarray dataset locally but get the following error:
RuntimeError: getsize is deprecated and should be removed.
Why might that be?

Thanks for reporting @yutik-nn. Looks like this method was removed in newer versions of Pillow.
geogif=0.1.5 has just been released fixing this issue.


How to train this model?hurry up!

Looking forward  to the training codes, too.

We have released the training code of X-Decoder today : )

Thanks so much for the interests, and sorry for the delay.


Font Awesome (FA) icons in experience.html

The "Other IT" section has 5 FA icons. These don't look right on mobile (the fourth one is flush against right side and the 5th one is kicked down to the next row).
Either drop to 3 or create 2 rows of 3.

Closed by #40


index.rst: entry point arguments: arrays are not allowed inside structs

It's an odd restriction.  Just an oversight?

From  "Arguments and Return Types", in the bullet point for "Resources":

Slices must only hold scalars, vectors, matrices, or structs containing any of these types. Nested structs are allowed.

The array case is missing.

Yeah, we should treat them as resources, not stage in/out variables.

We should write somewhere what happens if the WHLSL shader says the array has n elements but the API binds an array that has < n elements

Do you mean this at the outer level?  That is, where an API-side array is being bound to a slice that is an argument to an entry point?
Or do you mean an array inside a struct that is an argument to an entry point?   In this case I'm not sure how the API call can validate the array size because it's buried inside other data.


unsaved changes dialog

When you click on a different note and you have unsaved changes the following popup shows up:

This also has some dire consequences. There should be 3 buttons:
Yes, No (discard curent changes), Cancel
Cancel must not be used for No. Cancel is used for cancelling the current action. In this case it means that the dialog would disappear and the current note is still in memory and edited, but not saved.
No means the current changes are discarded.
Yes means the changes are saved.

P.S.: sorry to open so many issues.

Fine with me! Easier to track things with many different issues than with only one big one...

Have added three buttons with long write-up for such cases:
Options:
Save own note to different name
Save own note and overwrite file system changes
Load changed note and discard own changes

Hope thats clear enough now :-)

I think you might have mixed up #24 and #25.
This issue still persists. There's no new dialog with 3 options. I still see the same dialog as in the screenshot above https://github.com/ThomasDaheim/ownNoteEditor/issues/25#issue-124001687

You're right - I shouldn't commit so late in the day :-) Will have a look at it again

Hi @tessus,
unfortunately, I haven't found any way to do "Cancel" properly. It would need to stop e.g. changes tabs or notes when clicking on them. But I haven't found a way to get the related event BEFORE the focus has changed. So with "Cancel" you would end up still editing your note but the UI would e.g. show a different group with a totally different list of notes...


booted bots can't join the battle

I started robocode-tankroyale-gui-0.22.0.jar on Windows 10.
Selected Config Menu -> "Bot Root Directory" and added the path to the sample bots
Selected Server Menu -> Start Server
Selected Server Menu -> Select Server -> Test  -> "server is running"
Selected Battle Menu -> Start Battle -> Selected some bots from "Bot Directories (local only)" and clicked on Boot-Button.
Selected Bots were shown on "Booted Bots (local only)" list
"Joined Bots (local/remote)" list stays empty. So I can't add a bot to "Selected Bots(local/remote)" list and the "Start Battle" Button remains disabled.

What wents wrong?

@alex-314214:
Thank you for sharing a description of what you do. 👍
It looks like you did it the correct way, but I am unsure if you selected the joined bots to the battle by pressing one of the Add buttons? When you have selected which bots that must battle (in the Selected Bots (battle participants)) you should be able to press the Start Battle button.
Please notice that you will need at least two bots in the battle before the Start Battle button gets enabled.

In order to see what is going on, we need to start the bot from a terminal window.
So you should cd your way into the bots directory, e.g. the Corners bot, and write:
java -cp ../lib/* Corners.java

I expect that the Corners bot will run or write an error to standard out.
You can do the same thing for any Java bot, so if Corners works, but not your own, you should try out running your bot instead from the terminal the same way.
Please note that a server must be started in advantage.
Please note that you will run into this error:
Disconnected from: ws://localhost:7654, status code: 1008, reason: Invalid secret

This is because the server is protected with an server key that the bot needs to provide in order to connect to the server.
You can find the key in the generated server.properties file located at the directory where the GUI is started from.
It will look similar to this:
bots-secrets=ZKfl1TOEvabzxqm6YRC0og
controller-secrets=QlgOL13TnWrdQoqnKcIEpA

Now, before calling the java -cp ../lib/* Corners.java command, you should set the SERVER_SECRET environment variable to the secret you find in the server.properties file under bots-secrets.
if you use PowerShell:
$env:SERVER_SECRET="ZKfl1TOEvabzxqm6YRC0og"

if you instead use Command  Prompt:
set SERVER_SECRET=ZKfl1TOEvabzxqm6YRC0og

Now you should be able to connect to the server with java -cp ../lib/* Corners.java.
What it written out to the console in your case, when you try to start the bot?

Thank you for your hints. But I found a solution on https://github.com/robo-code/robocode/releases that works.

Anybody else encountering this issue, and what are your findings (written out by the bot(s))?

One observation on my side is, that this happens sometime when running on Linux. Hence, this needs more investigation.

Closing it for now.

I think I'm experiencing the same issue. When attempting to run Corners, I encounter the following error: "Error: Could not find or load main class Corners.java".

@TheFlameFish : Okay. I have now re-opened the issue.
Can you provide you with some details about your system, Java version and Robocode Tank Royale version? 🙂
Thanks in advance.

Never mind, sorry, I got it working. I have Java 8 installed for a different program and apparently, it was being used as the default Java version. Changing the environment variables to have version 22 be the default fixed it.

@TheFlameFish: Thanks for the heads up.
I think I will build-in a Java version check, so an error will be thrown if the Java version is too old. Then it might get easier to figure out, that the JVM is just too old. 🙂


Added Latest Homepage screenshot in the README.md

After Fixing the issue, tag the issue by # in your Pull request like below example.
Added Admin Login Page fixing issue no #1 .
Find the teslaclublive.png in the root folder and replace it with the latest cropped screenshot of the home page having same name to update the image in the README.md.
teslaclublive.png

yes sir


Books search form

dev notes

[x] create books/forms.py
[x] move shared classes and their tests from people/forms.py to common/
[x] create a books form that subclasses FacetForm
[x] add form methods to the ItemList view
[x] make ItemList view subclass FormMixin, AjaxTemplateMixin, FacetJSONMixin
[x] add a test_get_queryset() test for the new view
[x] update item_list.html template based on member_list.html
[x] update books-search.ts based on members-search.ts

Verified that this is working on the QA site.


Investigate Using VS Code Proper

Why are we using code-server.
We can also just use native:
wget https://az764295.vo.msecnd.net/stable/$(curl https://update.code.visualstudio.com/api/commits/stable/server-linux-x64-web | cut -d '"' -f 2)/vscode-server-linux-x64-web.tar.gz

It's not clear that code-server gets us anything specifically.

https://github.com/microsoft/vscode/issues/135856


Update project description and add access log documentation

https://skywalking.apache.org/docs/skywalking-rover/next/en/setup/overview/#skywalking-oap This part should be updated as well.

And Configuration and Use Cases should be updated too.
Meanwhile, I can only see deploy on Linux, where is on K8s? The access log requires k8s, right?

Yes, I will add a new Kubernetes deploy documentation.

I think from now on, running on k8s is the first and most important use case, until we are going to monitor VM service through eBPF(but not now).
The previous use cases seem to be a little advanced features only.


Fyi: large log causing error

Was getting a datatable error at seatgroups/about

This seems to be caused by the following call
https://seat.tgrad.space/seatgroups/logs/logs?draw=1&columns[0][data]=created_at&columns[0][name]=&columns[0][searchable]=true&columns[0][orderable]=true&columns[0][search][value]=&columns[0][search][regex]=false&columns[1][data]=event&columns[1][name]=event&columns[1][searchable]=true&columns[1][orderable]=true&columns[1][search][value]=&columns[1][search][regex]=false&columns[2][data]=message&columns[2][name]=&columns[2][searchable]=true&columns[2][orderable]=true&columns[2][search][value]=&columns[2][search][regex]=false&order[0][column]=0&order[0][dir]=desc&start=0&length=25&search[value]=&search[regex]=false&_=1536582211801
which resulted in a memory limit:
[2018-09-10 12:58:46] local.ERROR: Allowed memory size of 134217728 bytes exhausted (tried to allocate 4096 bytes) {"userId":395763831,"exception":"[object] (Symfony\Component\Debug\Exception\FatalErrorException(code: 1): Allowed memory size of 134217728 bytes exhausted (tried to allocate 4096 bytes) at /var/www/seat.tgrad.space/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php:183)
[stacktrace]
#0 {main}
"}
Fixed my issue with

3
5
15:00:12
DELETE FROM seat.seatgroup_logs
132952 row(s) affected
1.140 sec

Just be aware that a limit might be needed on the log history call
Cheers

Thank you for reporting this issue.
This is an odd one: I explicitly use serverSide rendering for this table:
https://github.com/herpaderpaldent/seat-groups/blob/master/src/resources/views/logs/list.blade.php#L17
This is why it shouldn't happen. I can try to implement smth but not really sure if it will be fixed by that. Need to exchange with people more experienced then me about this one.
Have you seen the delete button, which should be there for you to purge the logs too?

Also, the message field storing the log is apparently too small.
PDOException: SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'message' at row 1 in /var/www/seat/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php:142

Also, the message field storing the log is apparently too small.

Damn, i didn't really think that this could happen. I used string and thought that would be enough. To what should i alter the field?

Ah yeah I get these indeed as well, I would convert the column to TEXT

I tried to fix this issue with v.1.3.0. let me know if you persist to have issues.


How to call Java Standard Libraries in JPHP Like in PeachPie (.NET)?

I can do this in PeachPie (PHP Compiler for .NET) which is basically calling .NET standard libraries.
<?php
use \System\Console;
use \System\IO\StreamReader;
use \System\Net\WebRequest;

function main()
{
    $uri = $_SERVER['argv'][1];

    $http = WebRequest::Create($uri);
    $response = $http->GetResponse();
    $stream = new StreamReader( $response->GetResponseStream() );

    Console::WriteLine( "HTTP Response:\n" . $stream->ReadToEnd() );
}

main();

How to do call Java standard libraries like the one above? I've tried this one.
<?php

function main() {
    \System\out::println("Hello World\n");
}

main();

But it gives me.
$ jppm start
-> linux
-> app:run
-> install

Fatal error: Uncaught Error: Class 'System\out' not found in res://index.php on line 4, position 12
Stack Trace:
#0 <internal>() called at [res://index.php:1]
#1 main() called at [res://index.php:7]
#2 {main}
  thrown in res://index.php on line 4

Maybe because in Java out is a field of System and you should access it like a field.
Console is a class of System package in .net


Error after update xcode.

Module compiled with Swift 5.1.3 cannot be imported by the Swift 5.2 compiler

Are you using Carthage? You have to rebuild your dependencies with the new Swift compiler 🙂

ok, It's success. Thank you.


DMP export update

Now PDF export works with a complex KM - 41 pages were generated within 1 second (built & run in Docker).
I also find out that there is some problem with Indications and Metrics in report, that might be solved also in this PR (?):

Answered and Unanswered indications are both 0
The same metrics are there multiple times for one chapter (for example three times repeated)

Found were not problems:

Answered and Unanswered indications are both 0 because none of the questions is mandatory due to phase
The same metrics are there multiple times for one chapter (for example three times repeated) = caused by repeated migrations on local instance


Add consolidation of Flux stdlib dropdowns [food for thought]

Describe the issue here.
Idk how pervasive this is but I find the list of dropdowns for the Flux stddlib docs difficult to navigate.  It's a little long and the pattern seems just slightly inconsistent. Given that I'm already familiar with our docs and Flux, I can't imagine our users (especially new ones) have it any easier.

Seems to me the endpoints like Pagerduty, Slack, and Pushbullet could be conslidated, as well as all the InfluxDB ones?
The changes to the InfluxDB-specific docs with "Write Data" and "Query Data" made a lot of sense.  I feel like something along those lines could be applied here.

Thanks @samhld, the Flux docs are about to undergo a bit of an overhaul, so good timing.

@samhld I'm working on the restructure and have decided to structure the standard lib docs to match the structure of the standard library itself. Thoughts?

@samhld I'm working on the restructure and have decided to structure the standard lib docs to match the structure of the standard library itself. Thoughts?

@sanderson I assume you mean following this structure?
If so, I would probably say the same thing about it.  An example of what I would think is a net positive consolidation would a structure grouping http, json, csv, array, sql, kafka, socket, etc. like this:

I/O

Inputs

http
socket
json
csv
array
sql
kafka

Outputs

http
socket
json
csv
array
sql
kafka

To a new user, all of that would look like:

I/O

There are probably a lot of other ways organize that (i.e., you could pull out http and socket into a category called network or something)
You could also store pagerduty, slack, victorops, etc., into monitor or alerting or something.

@sanderson I assume you mean following this structure?
If so, I would probably say the same thing about it.  An example of what I would think is a net positive consolidation would a structure grouping http, json, csv, array, sql, kafka, socket, etc. like this:

I/O

Inputs

http
socket
json
csv
array
sql
kafka

Outputs

http
socket
json
csv
array
sql
kafka

To a new user, all of that would look like:

I/O

There are probably a lot of other ways organize that (i.e., you could pull out http and socket into a category called network or something)
You could also store pagerduty, slack, victorops, etc., into monitor or alerting or something.


feat: Xray compliant ids

Need some feedback here.  @fbogsany @ahayworth @robertlaurin  This general implementation works on my local test environment.
Questions:
Seems like this is in the wrong place? Where should I put this "extension" for AWS XRay?  Perhaps I should include it in the xray propagator gem?  Seems like people should use that gem in conjunction with this.
Remaining Tasks:

Decide where this should live
Write test(s)
Update some documentation

This is all in reference to:
https://github.com/open-telemetry/opentelemetry-ruby/issues/832

Seems like this is in the wrong place? Where should I put this "extension" for AWS XRay? Perhaps I should include it in the xray propagator gem? Seems like people should use that gem in conjunction with this.

Yes - I would agree that it shouldn't go here. I think your suggestion of placing it alongside the xray propagator somehow would make the most sense. Then as part of this PR we could also update the xray propagator docs to reference the extra installation step. 😄

IIRC AWS has included this sort of thing in their OpenTelemetry distributions. AFAIK they don't yet have a distribution for OpenTelemetry Ruby. IDK whether they regard XRay propagation and XRay trace ID generation as going hand-in-hand - technically, you could use a different trace context propagator with XRay, and potentially only use the XRay propagator when talking to AWS services.
All that aside, unless someone from AWS has a strong opinion (and I don't know who to ping about this), sticking it in the XRay propagator gem seems reasonable to start with.

Looking at where Java put it:
https://github.com/open-telemetry/opentelemetry-java/blob/main/sdk-extensions/aws/src/main/java/io/opentelemetry/sdk/extension/aws/trace/AwsXrayIdGenerator.java
If we follow their example it seems it should go into some type of SDK extension, I dont see a similar pattern for something like this in the current repo?  It's not a big deal to me one way or another.  🤷
I'm good with moving it into the propagator.  Seems like a good place for now.  The propagator doesn't automatically instrument so you can still use a different propagator and the xray id generation functionality independently.
I'll do this now.

Alright, updated.

@fbogsany Thanks for the feedback, total noob here with ruby.  I'll respond to each comment, but they should all be resolved.

Updated.


Final pass on public API javadoc

The goal is to do one final pass on public Javadoc before 3.0.0 is released.

Remove author tags
Correct typos, improve grammar, ...

Other ideas around standard L&F are welcome.

Modules concerned are api module and the builders packages.


[BUG] user account info is not autofilling consistently

first and last name only work if the overview page is your entry point (i.e. if you refresh)
email and library card number seem to not work at all

Can't Reproduce: v3.0-alpha
No idea when this got fixed, but I think it's all good now.


how to use this model step by step ?

Hi @jiangjiaxi96, we added in the README a usage example on the Omniglot dataset and the link to the Omniglot and Mini-Imagenet dataset (embedding included). If this doesn't clear your doubts please let me know.
Best,
Ste

Thank you so much for your help!!!! But, I got some bugs when I choose the Omniglot dataset. Have you met those bugs before? Is it possible that the related package version is wrong?would you plz tell me the requirement.txt?

On Oct 9, 2020, at 9:04 PM, stefano vincenzi<EMAIL_ADDRESS>wrote:
Hi @jiangjiaxi96 https://github.com/jiangjiaxi96, we added in the README a usage example on the Omniglot dataset and the link to the Omniglot and Mini-Imagenet dataset (embedding included). If this doesn't clear your doubts please let me know.
Best,
Ste
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/alessiabertugli/FUSION-ME/issues/2#issuecomment-706168611, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANBQOKP7KXX5RAH234CBDLDSJ4C6NANCNFSM4SJXWCNA.

No description provided.

111

No description provided.

Hello, I am just a beginner at this. Could you tell me what to link to the data_floder? Because there's many data_floders in that function. Thanks!!

Are you talking about data_folder in get_embeddings function in datasets/utils.py? If yes, you have to link the precomputed embeddings (you can find them in the link provided in the README file under "Embeddings"), according to the type of embeddings and the dataset you want to use.

Thanks for reply! I do talking about the data_folder in get_embeddings function in datasets/utils.py. What you mean is that these data_floder are supposed to link to other project's precomputed embeddings? As the perior knowledge I guess? So, because I use the omniglot data set, I should replace data_folder = 'path_to_bigan_encodings'by the path to the "baseline.sh" in the file that I download form your README file. Forgive me for my stupidity，I am a undergraduate and my task is to run a rcent project of few-shot learing and to make a demo.
Thank you again!!! That's so nice for you to help me! Wish you a good time!
------------------ 原始邮件 ------------------
发件人:                                                                                "alessiabertugli/FUSION-ME"<EMAIL_ADDRESS>发送时间: 2020年10月12日(星期一) 晚上8:01
收件人<EMAIL_ADDRESS>抄送: "1111a"<1259840828<EMAIL_ADDRESS>主题: Re: [alessiabertugli/FUSION-ME] how to use this model step by step ? (#2)
Are you talking about data_folder in get_embeddings function in datasets/utils.py? If yes, you have to link the precomputed embeddings (you can find them in the link provided in the README file under "Embeddings"), according to the type of embeddings and the dataset you want to use.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

We use precomputed embeddings obtained with other methods (Deep Cluster, ACAI, BiGAN) to cluster data and assign them pseudo-labels. To facile the use of our model, we give you our precomputed embeddings that you can find at the link in the README. You have to download these files, put them in a folder (for example you can create a new folder called "data" contains sub-folders such "acai_encodings", "bigan_encodings" and "deepcluster_encodings") and finally you have to set data_folder = your_absolute_path+"data/acai_encodings/" for example to use Omniglot dataset with acai embeddings.

@jiangjiaxi96
requirements.txt
I use this environment also for other projects, so not all the packages are necessary for this repo.


Object mode or no-jit at all for unknown objects

Hi,
I'm following up on #3907, regarding the deprecation of object mode. I almost never use object mode, but I could imagine a case like this:
import pandas as pd
import numba as nb

@nb.jit
def double(x):
    return x*2

double(0.5)

double(np.ones(3))

double(pd.DataFrame(np.ones(3)))

That example behaves as expected, all three lines produce a result. However, I think that it works by falling back on object mode. If object-mode fallback is removed, what would happen to a case like that?
I understand that there's an opportunity to simplify the pipeline by removing the fallback, but would it be possible to leave some kind of multiple-dispatch on unknown types that runs object-mode or simply interpreted code?
Unlike the current case, I don't think it needs to look line-by-line inside the function to know whether to fall back. For example, whereas today the following works via object-mode fallback
def something_non_jitable(x):
    return x

@nb.jit
def foo(x):
    return something_non_jitable(x)

foo(0.5)

in the future it would not, because it's not a problem of input parameters but of code failing. So in the future foo(0.5) would fail as it fails today with njit but double(pd.DataFrame(np.ones(3))) would not fail (as it does with njit) but would fallback to object code or interpreted code.
Something to consider. Thanks for your work on Numba.

Thanks for writing this up, I'm going to raise it at the core developer meeting today.

Make sure we also note this comment https://github.com/numba/numba/issues/3907#issuecomment-500765025

Yeah, this is a good point.  Things we are trying to discourage with this change:

Automatically switching between modes of very different performance.
Applying Numba to functions (accidentally) that it can't compile.  (A common pitfall for new users.)
Top-level usage of object mode: Object mode has several limitations, and we got ourselves into trouble by trying to reuse results from the compiler pipeline between nopython mode and object mode when we fall back.
Magical thinking about what object mode can achieve (😄): Object mode suffers from being a compilation mode that seldom speeds up code, but exists for other reasons.  New users tend to assume that if compilation succeeds, it must be faster.

That said, I agree that there is a potential use for Numba in situations where the author doesn't have total control over the types coming in, and would like some kind of best-effort compilation attempt.  As you point out, this still needs assistance from the Numba dispatcher, in order to recognize argument type combinations where nopython mode will fail, and immediately dispatch to the interpreter without attempting to compile.
One possible solution (not committing to this, but just brainstorming) would be a new option interpreter_fallback=True to @jit that did the following:

Compute type signature of input arguments and look for matches in the dispatch table.  Call if match found.
If no match in table and any of the arguments type to pyobject (Numba's "I don't know" type), then insert and call a dispatch entry to the interpreter version of the function.
Otherwise, if compilation is still allowed on the function (i.e. the function wasn't declared with an explicit set of type signature.), compile in nopython mode.

If compilation succeeds, insert compiled function into dispatch table and call.
If compilation fails, insert dispatch entry to interpreter version of the function.

Finally, if you get to this point, compilation is closed on the function and no matches have been found, so insert an entry into the dispatch table calling the interpreter version of the function.

This would be considered an advanced option that should only be used if the user understands the implications, and has the advantage of avoiding object mode while also allowing dispatch between compiled and interpreted implementations.
I'll bring this up at the developer meeting and see what people think.  We're very hesitant to add new options to an already option-heavy @jit decorator.

@seibert @stuartarchibald thanks for considering it. If @jithas too many options, what about a second (compounded) decorator? Something like
@jit
@with_interpreted_fallback
def foo()

or even make room for future expansion
@jit(parallel, etc)
@with_advanced_options(interpreted_fallback=True, <other advanced otions>)
def foo()

The conclusion from today's discussion is that we're convinced by the arguments from you and @godaygo, and want to fuse the above proposal with this one: https://github.com/numba/numba/issues/3907#issuecomment-500765025.
@jit will get an optional fallback argument which is None by default (no fallback, raise an exception if compilation fails).  If fallback is a special constant (numba.INTERPRETER), the fallback will be the decorated function, but executed in the Python interpreter.  If fallback is a callable, the dispatcher will call that function (again, in the interpreter) when fallback is required, and that function will take the place of the decorated function.  (i.e. the callable can return something back to the top-level caller)  The rules for when to take the fallback option will be the same as I describe above.
In retrospect, this would have been a much better and more explicit way to handle our object-mode logic, and it also allows library authors more freedom to use Numba for some cases, and do something entirely different when it doesn't work.
I'm not sure on the ETA for this, but we'll definitely get it into Numba at least one release before we drop object-mode fallback.
Thanks for talking through this.  These discussions are very helpful for us.  :)

glad to be of help!


Is this storage engine faster then Async Storage?

Hi! I am asking because I know AsyncStorage is slow. Is this WAY faster?

@wmonecke Overall. Yes, Realm is faster because they are mapping the data using in-memory, using B+ trees to avoid unnecessary copying.
You can check this link below for more details.
https://www.simform.com/react-native-database-selection-guide/

Awesome thanks! @jayzyaj


Android app: Untranslatable strings

The second value of settings_notifications_min_priority_summary_x_or_higher is not available for translation. The values comes from toPriorityString via minPriority?.summaryProvider.

Good catch. Will be easy to fix too.

Done in https://github.com/binwiederhier/ntfy-android/commit/f35c598ae8e58d4f7b772b83bbba2fdc911e27b6


New Feature: add IORef module

When will this feature be available? Working on unit tests as proposed by @gcanti in in this discussion.


Syncing client with server

Greetings!
Are there any working examples showing how to sync data between server and clients?
Thanks in advance!

Not yet, I will come up with some examples showcasing a few use cases soon enough.

I'm reading both client and server code. Where do I state the time interval between syncs?

There's no time interval, it's not polling. It uses WebSocket, so the server actually sends updates to the client at any time during the connection.

Oh, ok! :+1:
So, that raises the question: every CRUD action by a client will be send to all other clients?

By default, yes. However, the server can modify or omit the change before sending anything to clients via the change function.
If you have a cluster of servers, it gets a little tricky because then each server has to coordinate changes among each other to reach the right clients. It is possible to do but isn't abstracted by Fortune.js, at least not yet (I think it should take advantage of multiple cores by default).


CREATE3 version

Can we have a version for CREATE3?
With CREATE3, the resulting address of a deployed contract is no longer dependent on contract bytecode.

I don't believe there is such a thing as a CREATE3 opcode like there is for CREATE or CREATE2.  If you are referring to the pattern of deploying via a proxy contract as described at https://github.com/transmissions11/solmate/blob/main/src/utils/CREATE3.sol, then that is solving a separate problem from this and is unrelated.
You are certainly welcome to use something like that if that is what you are looking for!  There are also people who have published factories for making deployment via that technique easier like https://github.com/ZeframLou/create3-factory/blob/main/src/CREATE3Factory.sol.


type annotations needed for Box<_>

I get this error when trying to build the app from arch AUR
https://aur.archlinux.org/packages/iamb
rustup: 1.27.1
rustc: 1.80.1
error[E0282]: type annotations needed for `Box<_>`
  --> /home/alireza/.cargo/registry/src/index.crates.io-6f17d22bba15001f/time-0.3.29/src/format_description/parse/mod.rs:83:9
   |
83 |     let items = format_items
   |         ^^^^^
...
86 |     Ok(items.into())
   |              ---- type must be known at this point
   |
help: consider giving `items` an explicit type, where the placeholders `_` are specified
   |
83 |     let items: Box<_> = format_items
   |              ++++++++

   Compiling imbl v2.0.3
   Compiling chacha20poly1305 v0.10.1
   Compiling vodozemac v0.5.1
   Compiling image v0.24.7
For more information about this error, try `rustc --explain E0282`.
error: could not compile `time` (lib) due to 1 previous error
warning: build failed, waiting for other jobs to finish...
==> ERROR: A failure occurred in build().
    Aborting...
error: failed to build 'iamb-0.0.9-1':
error: packages failed to build: iamb-0.0.9-1

sorry for my mistake. the error is from another lib, and I built iamb myself now, and it built successfully. so the problem must be from the PKGBUILD.

The problem unfortunately comes from a specific version of one the dependencies in the lockfile for v0.0.9 when building with Rust v 1.80 (the latest release). Building from main (which it sounds like is what you ended up doing) or with a Rust version of 1.79 should work.
I'm working towards cutting a new v0.0.10 release which will mean that crates.io then contains a release that builds with 1.80.


Sampleapp603

This is the same as earlier pr just with less commits and better git structure

Coverage increased (+0.1%) to 92.54% when pulling bf3315d2e256ac4c49013b5dc6de77ee091da828 on wnordmann:sampleapp603 into 8e3d62db991732cd665b8fe296271fabf15f2459 on boundlessgeo:ultimateredux.


feat-add-loader

loader feature added.
I test it locally and it working properly

@GagandeepSingh0115 I don't know why, but it shows 40 files changed. Can you look into it? It's pretty same code that's already in repo, so why it's coming like that!

ok, give me 30 min.


Make parenting example clearer on parenting rules for newcomers

For new users (me included!) coming from engines like Unity, it's easy to not be aware that a parent must also have a GlobalTransform along the obvious Transform.
They then ask themselves why their hierarchy doesn't work properly when using a single Transform instead of a PbrBundle like in the example.
Objective

Make parenting basics/rules more obvious to newcomers

Solution

Add a rotating object whose parent is an "empty'": a minimal parent entity that's just a transform in the world.

Closing as part of backlog cleanup, likely to have significant bit rot after two years. @dtaralla, if you feel this would still be a good change please feel free to open another PR. Cheers!


Fix nonce computation

Fixes #1430 .
Changes proposed in this PR:

implemented nonce endpoint
modified sign_message function
transformed UNIX timestamp in milisecs

The nonce endpoint is just for... orientation. The nonce should not be formed via the endpoint, but using the nonce logic we have defined (needs to be larger than last time).

The nonce endpoint is just for... orientation. The nonce should not be formed via the endpoint, but using the nonce logic we have defined (needs to be larger than last time).

I have converted to miliseconds format because by default timestamp() function converts to UNIX seconds.

Does this supersede the rest of #1307?

Does this supersede the rest of #1307?

It is an important aspect and this PR can be related to #1307 issue.


feat: pass config to parse

Passing config to parse, so we change lang or abortEarly when needed

Thank you for this PR. Could you please add the usage explanation to the README as well?

@Songkeys Ignore this PR, i stumbled upon the v.config method provided by valibot, which does the same thing for me and should for everyone else


Some problems with the fs‘ writeSync function

node version: 10.14.0
npm or yarn version: yarn v6.4.1
OS/version/architecture: windows 10  x64
In the program, the writeSync method of the fs module is called multiple times. If the power-off operation is performed midway, part of the file content is empty, but the file size exists. Is this because the partial cache is not flushed to the disk?

I'm sorry, but unless you can provide more details you might be looking for https://github.com/nodejs/node.


Add --pid flag for staying in the host's pid namespace

Replaces #9339
This adds an integration test for the --pid host mode

kewl LGTM, I bet next will come --pid=container ;)

This has been LGTM'ed in the other PR, merging.

I just shed of tear of joy when I noticed this PR was merged.  I'll be a sobbing baby when --pid=container lands.

Is there a PR open for --pod=container ?
On Jan 15, 2015 9:39 AM, "Darren"<EMAIL_ADDRESS>wrote:

I just shed of tear of joy when I noticed this PR was merged. I'll be a
sobbing baby when --pid=container lands.
--
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10080#issuecomment-70127018.

I was making a joke that it would be next, but no one has made it yet
On Thursday, January 15, 2015, Tim Hockin<EMAIL_ADDRESS>wrote:

Is there a PR open for --pod=container ?
On Jan 15, 2015 9:39 AM, "Darren"<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

I just shed of tear of joy when I noticed this PR was merged. I'll be a
sobbing baby when --pid=container lands.
--
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10080#issuecomment-70127018.

—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10080#issuecomment-70172175.

Consider this a request?
On Jan 15, 2015 3:15 PM, "Jessie Frazelle"<EMAIL_ADDRESS>wrote:

I was making a joke that it would be next, but no one has made it yet
On Thursday, January 15, 2015, Tim Hockin<EMAIL_ADDRESS>wrote:

Is there a PR open for --pod=container ?
On Jan 15, 2015 9:39 AM, "Darren"<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

I just shed of tear of joy when I noticed this PR was merged. I'll be
a
sobbing baby when --pid=container lands.
--
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10080#issuecomment-70127018.

--
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10080#issuecomment-70172175.

 Could you please create a new issue describing this? It'd make it easier to track, and a decision to implement it (or not) can be made there.
(Sorry for meddling in this, @jfrazelle :smile: trying to keep my head sane following things)

People looking for --pid=container; @thockin opened a new issue for that here: https://github.com/docker/docker/issues/10163 (thanks!)

I cannot seem to figure out how start a container with --pid=host using the API, Does this command line option an API analogue?


Fix friends of rust website  https://www.rust-lang.org/en-US/friends.html

When I navigate from https://www.rust-lang.org/en-US/faq.html to https://www.rust-lang.org/en-US/friends.html - the link seems to be broken.
The same URL has been used in quite a few projects.
Specific focus:
A growing number of organizations!

cc @steveklabnik


Fixed pointing runs

Adds simulation of the fixed pointing (in alt/az) runs.
Changes:

FixedPointingDataRun: new class implementing the fixed pointing simulation;
FixedObsGenerator: new class generating runs based on the requested pointing and exposure;
refactoring to accommodate both fixed pointing and regular "sky" runs.

@mstrzys , any comments on this PR?

@mstrzys , I've fixed the merge conflicts; this MR is ready to merge if you approve.


Noxious is broken on atom-shell v0.23.0

See: https://github.com/atom/atom-shell/issues/1410

We have abandoned the ursa module and its native dependencies and replaced it with the JavaScript forge module.


There is a problem with translation when the language family name has Hyphen

Description
I'm using https://github.com/amann/next-intl/tree/main/examples/example-app-router to start my build
When I add the language zh-Hant, {t('locale', { locale: cur })} in LocaleSwitcherSelect does not work, next-intl outputs it as LocaleSwitcher.locale, when I change zh-Hant to zh everything is normal.
Here are my modifications, where did I go wrong? Thanks.
config.ts
export const defaultLocale = 'en' as const;
export const locales = ['en', 'de', 'zh-Hant'] as const;

export const pathnames: Pathnames<typeof locales> = {
  '/': '/',
  '/pathnames': {
    en: '/pathnames',
    de: '/pfadnamen',
    'zh-Hant': '/中文',
  },
};

message/zh-Hant.json
  "LocaleSwitcher": {
    "label": "Change language",
    "locale": "{locale, select, de {🇩🇪 Deutsch} en {🇺🇸 English} zh-Hant {Chinese} other {Unknown}}"
  },

error code
INVALID_MESSAGE',
  originalMessage: 'EXPECT_SELECT_ARGUMENT_SELECTOR_FRAGMENT ({locale, select, de {🇩🇪 Deutsch} en {🇺🇸 English} zh-H    ant {Chinese} other {Unknown}})

Verifications

[X] I've verified that the problem I'm experiencing isn't covered in the docs.
[X] I've searched for similar, existing issues on GitHub and Stack Overflow.
[X] I've compared my app to a working example to look for differences.

Covered in the docs here: Which characters are supported for select values?


release-20.2: ui: fix metric rendering when node data is missing

PR: https://github.com/cockroachdb/cockroach/pull/66034
From release notes:

Drag to zoom on metrics graphs now supports time ranges under 10 minutes. [#66034][#66034] {% comment %}doc{% endcomment %}

Closing; doesn't need docs.


KAFKA-9868: Reduce transaction log partitions for embed broker

This PR tries to fix the flaky EOSUncleanShutdownIntegrationTest.shouldWorkWithUncleanShutdownWipeOutStateStore by making the bootstrapping of the test to be less painful with fewer number of partitions of txn log.
The integration test time on local also reduced one second.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@vvcephei @mjsax Call for a review

Retest this please.

Run 350 times on local and no failure.

@mjsax I suppose this should help for all integration tests, as they share EmbeddedBroker

Got 2/3 green, one failed test is due to
kafka.api.SaslSslConsumerTest.testCoordinatorFailover

java.lang.AssertionError: expected:<None> but was:<Some(org.apache.kafka.clients.consumer.CommitFailedException: Offset commit cannot be completed since the consumer is not part of an active group for auto partition assignment; it is likely that the consumer was kicked out of the group.)>


--desc flag to sort tweets in descending order

This initially included adding a check to the L29:tweets.go:Less() function
to return the opposite value of what is expected in an ascending
sort. I then remembered there's a built-in to the sort package to do exactly
that, and committed a change to use the more idiomatic approach.
RE: TODO.md, item 4:
* cli/output: ascending/descending time order

I wasn't sure exactly how you might want to word the -desc/--desc flag description, or if you wanted the flag itself to be called something else. It will be really easy for you to change the flag and/or the description to your own taste, as I'm sure you know. :)

:+1:  I'd prefer to have this timeline specific flag for the timeline command, no global..

Okedoke, that does make more sense. It's now a timeline-specific flag :+1:

Great! I did interactive rebase on the command-line and merged... I'm not so well versed in github ui :p


Remove horizontal scroll from the console editor

finally fixed this and tested with a mouse

Codecov Report

Merging #521 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #521   +/-   ##
=======================================
  Coverage   30.18%   30.18%
=======================================
  Files          21       21
  Lines        1978     1978
  Branches      240      240
=======================================
  Hits          597      597
  Misses       1336     1336
  Partials       45       45

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 4a20fd6...5c31e70. Read the comment docs.


Update 0033-search-in-rotated-sorted-array.py

i edited the "or" cases in if statements  i think this is better for understanding of concept

File(s) Modified: 0001-two-sum.py, 0002-add-two-numbers.py, etc...
Language(s) Used: python, javascript, etc...
Submission URL: _https://leetcode.com/problems/[problem-name]/submissions/xxxxxxxxx/_

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

Please, add the correct Submission URL.

Suggested Solution still matches the original solution


Add support for Java 8 primitive optionals

This change is

It's a lot of code to avoid boxing/unboxing. Can't judge if it's worth it :/

Yep, it's mostly boilerplate. We could probably use Jackson to coerce everything that's a PathParam or HeaderParam that's not already a string, but since that wasn't how we were already doing this I did the tedious repetition thing.
I suspect better to support these things than not, but likely worth a discussion about code and test quality of the whole project.
Also, latest Dropwizard already has all the Jersey stuff by default, we may want to split that stuff out so that Dropwizard 1+ users do not double install.
For good measure, I did also check to see if we could depend on dropwizard-jersey, but it unfortunately has a long dependency tree, including Jackson, which is a persistent issue for us when trying to run servers that talk to Spark.
On Mar 17, 2017, at 4:34 AM, Robert Fink <notifications<EMAIL_ADDRESS>wrote:
It's a lot of code to avoid boxing/unboxing. Can't judge if it's worth it :/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub[github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_palantir_http-2Dremoting_pull_376-23issuecomment-2D287332000&d=DwMCaQ&c=izlc9mHr637UR4lpLEZLFFS3Vn2UXBrZ4tFb6oOnmz8&r=Vp81aOxWZvuVgVK_wp-VF3pIYG92B19LcCw6XKeYC0U&m=ODfscx7IQVnQ4W5qyXF5pyALFIkmjIpRHb7fliAXI_s&s=5472qik_LybXFy0kP8bZEYrnaa5aRWoxWIFC5S0K_CU&e=, or mute the thread[github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AAp3Q14w8qglEu8n-5FSo8TjsfXhJ-2Da0nXks5rmm-2DxgaJpZM4MgM0P&d=DwMCaQ&c=izlc9mHr637UR4lpLEZLFFS3Vn2UXBrZ4tFb6oOnmz8&r=Vp81aOxWZvuVgVK_wp-VF3pIYG92B19LcCw6XKeYC0U&m=ODfscx7IQVnQ4W5qyXF5pyALFIkmjIpRHb7fliAXI_s&s=EafbmRX4xDBurf7ALvUwDmiHN9ULyrJjFYldmQYwbzQ&e=.

Half of this will go away when we remove jaxrs-clients, and yeah, definitely agree the code&test quality needs some love here. Anyway, +1 for this PR.


⚠ SolarOscp has degraded performance

In da8f77e, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 272 ms

Resolved: SolarOscp performance has improved in cf8898d after 10 minutes.


add online install cluster test spec

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

/area testing
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:

Codecov Report

Merging #465 (7028eb6) into release-1.3 (0cf52e5) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           release-1.3     #465   +/-   ##
============================================
  Coverage        12.30%   12.30%
============================================
  Files              109      109
  Lines            16767    16767
============================================
  Hits              2064     2064
  Misses           14456    14456
  Partials           247      247


Removed spring dependency

Spring is no longer a dependency. However the the usage has changed a
little. Now interface is more like ForkJoinPool or other Executers in
Java Standard library. You create instance of AsyncRequestSerializer
passing the config. And then call submit method on that instance with
with (key, request) and get the Future back.

It fixes both #2 and #3


JDK 11: JceSecurityCleanupTest

Hello @mjiderhamn,
since the first PR I've opened for Java 11 compatibility (#87) looks really cluttered, with multiple commits to the same file that sometimes revert each other, I'm splitting the changes into smaller PRs.
The JceSecurityCleanupTest fails with Java 11, because a NoSuchAlgorithmException gets thrown. However, this still triggers the leak, so we just catch the Exception and ignore it.

Thanks for the PR!


23 create contact us page

here i can not see even this letter for example

@dirk-jacobs
I pushed with your last feedback.
Ready to test...! :llama: :grin:
I'm gonna continues with 24 - Create user-profile page
@dirk-jacobs @Moamin-AbuEwaida @inggritenovaleta @rayanejsilva @perezrei @firity

@dirk-jacobs
I pushed with your last feedback.
Ready to test...! :llama:

@dirk-jacobs hey I made the changes

@dirk-jacobs @yoshimalaise @rayanejsilva @Moamin-AbuEwaida @perezrei
Hey Buddies, :llama: :heart:  check it please and @dirk-jacobs, I think, we did it and like we disputed then I'll improve contact-us but I'll make a new issue like @rayanejsilva suggested me.
Have a nice day...! :sunflower: :rose: :llama: :sun_with_face:


Correctly work with systemd/logind

Currently the session started by rdm is partly recognized by loginctl (no seat or tty attribute present) and all authentication attempts (for example shutdown/reboot) fail with Failed to power off system via logind: Interactive authentication required..
This indicates that the pam_systemd.so module is not properly executed for the session (although the runtime directory in /var/run/user/ is created..) which might be caused by a problem in the pam configuration file.
More research is needed to determine which practices we need to follow.
As proper systemd support is one of the core principles rdm is built on, this is a blocker for v0.1!

This might also related to the way pam is used. Most DMs seems to first open a session without authentication when showing the ui (with XDG_SESSION_CLASS set to greeter) and then launching the "real" session upon login.
Research should determine the reason for this behaviour and next steps for rdm.


Checking the config.OwnerPW before it is set value!

I don't think it is right here:
func prepareEncryptCommand(config *pdfcpu.Configuration) *api.Command {
       //......
	if config.OwnerPW == "" {
		fmt.Fprintf(os.Stderr, "missing non-empty owner password")
		fmt.Fprintf(os.Stderr, "%s\n\n", usageEncrypt)
		os.Exit(1)
	}
        //......
}

checking the config.OwnerPW before it is set where calling the config.OwnerPW setting in the main.go.
However, do you want to check the var "opw" here?

Correct! I am preparing a fix.
Thanks for discovering this :green_heart


Make data

Hey Aaron and Dylan! The new pdfs are loaded and working in the text extraction step. I also added two make_ .R files to help get started with the next steps, and the Docs folder and Rmarkdown file we discussed today. Have a nice long weekend!

Thanks, Pascale! I resolved three conflicts before the merge


Link pre-trained model has broken

I try download pre-trained model but i can not accsess the link download
--2020-11-04 13:33:07--  http://vllab1.ucmerced.edu/~wenbobao/DAIN/best.pth
Resolving vllab1.ucmerced.edu (vllab1.ucmerced.edu)... <IP_ADDRESS>
Connecting to vllab1.ucmerced.edu (vllab1.ucmerced.edu)|<IP_ADDRESS>|:80... failed: Connection timed out.
Retrying.

working url: https://raw.githubusercontent.com/BurguerJohn/Dain-App/master/model_weights/best.pth
There's also a standalone app for it in the same repo. It hasn't been updated for awhile but still seems to work fine.


Bump to 2.1.0rc2

Bump to 2.1.0rc2

@forsyth2, in the conda recipe, make sure to reset the build number to 0 when you update the version number.  It's not a big deal but it's a good practice.
https://github.com/E3SM-Project/zppy/blob/b1957fbff9d44c1b86b1aa5b76fcf3a581589985/conda/meta.yaml#L13

@xylar Thanks, I just merged #255 to address this.


[BUG] 090-companion_sudo - Syntax error causes sudo issues post-install (Ubuntu Server 20.04 LTS)

I've been setting up a headless instance of Companion on Ubuntu Server 20.04 LTS. Once the install completed, I noticed I could no longer sudo because of a syntax error in 090-companion_sudo. This is obviously very problematic because you then can't edit the sudoers files without sudo priveleges.
I re-imaged and started again, this time not exiting sudo -i once the installation had completed then running:
pkexec visudo
Which checks/validates all the sudo files when you exit visudo. This then flags an error with 090-companion_sudo.
The file as installed is:
companion ALL=NOPASSWD: /sbin/reboot,/sbin/shutdown,/sbin/poweroff
Editing it to the below (note the two additional spaces) fixes the sudo syntax error and means that sudo works moving forward:
companion ALL=NOPASSWD: /sbin/reboot, /sbin/shutdown, /sbin/poweroff
Not sure if this is specific to Ubuntu rather than Rasbpian Lite - but at the very least hope this helps someone in the future and savest them the time I lost!
OS: Ubuntu Server 20.04.6 LTS
Branch: Beta (V3)

I haven't been able to reproduce the validation issues you are seeing on Ubuntu 22.04, but I see no harm in adding the spaces so that is done and will be pulled in when you next run sudo companion-update.
Be aware that your local changes would be reverted when you ran an update


Use nameof for param names

Thanks.  This is an older code base and likely needs some more love like this.


Improve documentation around granularities

The possible types of granularities that can be used are only mentioned within the querying section here:
https://druid.apache.org/docs/latest/querying/granularities.html
It's possible to use the same set of granularities as segment granularity during ingestion. We need to clarify this in the docs. Maybe just update the title of this page and link to it from segment granularities too?

I'm ingesting data from kafka.
I'd like to know if durring the ingestion process, when I set "segment granularity" to daily, all data in kafka corresponding to this period of time will be compacted or Druid will take a aleatory data from kafka. I could not found it in documetation.
https://druid.apache.org/docs/latest/tutorials/tutorial-kafka.html
Obs.: If these data are compacted in a single line in Druid, I'd like to know if it is possible to add 5 minutes and 30 minutes to the  segment granularity options.

Picking this up after a chat with @techdocsmith about how we can link to the documentation about granularities under querying from the documentation on storage and ingestion to clarify ingestion granularity.


How forEach works ?

I cant figure out how forEach work, I followed dot.js instructions but it gives syntax error.

got it :)
[[ for (var comment in model.comments){ ]]
your code here
[[ } ]]

:)


Request: Please update the pot file.

I declared to send a Docker image with a pull request before.
However, given the burden of the developers, I judged that the decision was not good and changed the policy.
Since satisfactory results were obtained with some tests, I decided to release it as an Unofficial Docker image.
At the same time, I had some experimental features implemented.
Main subject
Demonstration
Looking at this, most are localized, but there are parts that are not partially localized.
There seems to be a missing part in the pot file.
Especially not priority, but please update the pot file.
As soon as the missing part is complemented, I update the localization file.
Thanks.

This is really nice. Thanks for sharing it!

Don't mention it. :sunglasses:
It already functions in part of the staging environment and has earned high reputation "It became easy to find high-load process".
I will continue to maintain Japanese localization. If I can be of assistance, please don't hesitate to contact me.
Best regards,

#1301 Bandwidth :arrow_right: Tx. Amount


Some issue in 13.0.1 manifest for .net 4.0 framework

It seems that some issue is inside the manifest of latest version for .net framework 4.0
Getting the dll by NuGet at first level of reference it seems working (viusal studio seems to view it as .net 4.0 runtime assembly), but when you reference the assembly at second level something is happening on some machine having also .net 4.5
We got multiple error of this type:
C:\WINDOWS\Microsoft.NET\Framework\v4.0.30319\Microsoft.Common.targets(1605,5): warning MSB3275: impossibile risolvere il riferimento primario "XXXLib" perché ha una dipendenza indiretta dall'assembly "Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed" che è stato compilato con il framework ".NETFramework,Version=v4.5".
(message is in italian, I hope it is clear)
In effect viewing the reference in "Assembly Explorer" reports that something is targeting 4.5

Instead with version 12 all works fine

Unfortunately i can't really help you with your problem; aside from suggesting you check every assembly/dependency your software is using  -- including each dependency's dependencies -- whether some have dependencies on Newtonsoft.Json of a different version than your project is using.
But i want to point out that "Runtime version" in your secreenshot refers to the version of the CLR (the Common Language Runtime) required to execute the IL code in the assembly. Do NOT mistake it as Framework version.
All the .NET Framework 4.x versions (including .NET Framework 4.5) use the CLR v4.0. Thus, naturally the Newtonsoft.Json library version for .NET framework 4.5 would list runtime version 4.0.x (again, runtime version does not equate framework version).
(In comparison, the older .NET framework 3.5 uses CLR v2.0. And fortunately, starting with .NET 5, there is no separate runtime (CLR) version anymore that could differ from the .NET version, so such confusions will be a thing of the past...)

Hi elgonzo, i think you are right on below part, but the version above is the .NET Framework 4.0 version of Newtonsoft library, it shouldn't display targeting 4.5 (as displayed in the upper part of the screenshot). In my opinion this is leasing msbuild giving error on build. The strange thing is that if the assembly is directly referenced by a .net 4.0 library it would compile. Instead if you reference the secondary library by another application, msbuild is getting confused about the mixed targetting.
Also you can note the same incoherence decompiling the dll with a .net decompiler (I used here dotPeek I attach another image below)
I strongly suspect that in the last 13.0.1 release for .net 4.0 on nuget something has gone wrong and some 4.5 targetting has been left

I repeat you can also see yourself creating a blank .net 4.0 application, loading .net 4 library and looking in the reference with assemply explorer you can see as in first screenshot that the system shows a .net 4.5 targetting
On my side, on our company framework we have reverted all reference to 12.0.0 and now all build servers are ok

I don't have any experience with Assembly Explorer (it's a feature of jetbrain's Resharper, right?).
But note that both the error and what you see in Assembly Explorer are consistent. Both strongly indicate that there is actually no Newtonsoft.Json assembly for .NET Framework 4.0 present during your build process, but in actuality Newtonsoft.Json assembly for .NET Framework 4.5 is present. (The error MSB3275 indicates that a dependency could not be resolved because the dependency was for a framework version higher than the depending library or the softare project build target allows (quote from the MSB3275 documentation i linked to: "Dependent assemblies can't be built against a higher version of the .NET Framework than you are currently building for. Consider targeting a newer version of the .NET Framework.").
I also took a look into the nuget package for Newtonsoft.Json 13.0.1 and the assemblies contained therein, and i did not find anything suspicious there.
Thus, based on the given information i tend to disagree with your statement that

[...] the version above is the .NET Framework 4.0 version of Newtonsoft library, it shouldn't display targeting 4.5 (as displayed in the upper part of the screenshot).

Given that both the error message and Assembly Explorer agree on the Framework version, i rather tend to believe the display of Assembly Explorer is truthful, and that the verion above is indeed the .NET Framework 4.5 version of Newtonsoft library, not the .NET Framework 4.0 version as you seem to insist.
I do not share your suspicion that the Newtonsoft.Json 13.0.1 assembly .NET Framework 4.0 is somehow borked. Based on the the (somewhat limited) information in your report, i don't think that this is a likely case. Personally, my suspicion is strongly aimed at your project and/or build process somehow pulling in the actual Newtonsoft.Json 13.0.1 assembly for .NET Framework 4.5 instead of the intended/desired variant for .NET Framework 4.0.
If the problem is not with your software project and build setup somehow pulling in the incorrect Newtonsoft.Json assembly variant (which i still do not rule out; i don't know enough of your project and build setup to discard this possibility outright), perhaps your local nuget cache got corrupted somehow?
If you are willing and able to try, perhaps go into the folder of your local nuget cache, remove the folder belonging to Newtonsoft.Json 13.0.1 from it (so that the nuget package will have to be fetched again), then clean and rebuild your project again, and see whether it rectifies the situation. While you are at it, double check that your project files use the package reference for the official Newtonsoft.Json nuget package and are not by chance referring to an unofficial Newtonsoft.Json package that has been modified by a 3rd party. Alternatively, to verify that the local nuget is not corrupted, you could also manually download the nuget package (https://www.nuget.org/packages/Newtonsoft.Json/13.0.1), extract it (for example with 7zip) and do a binary comparison of the files in the local nuget cache and the file from the manually downloaded nuget package.

Hi elgonzo, thanks for your assistance, actually I am going to think the problem as you say is not on dll itself but in our build servers configuration. I managed to get the same informations above on same servers by jetbrains tools reporting 4.0 on same dll and I checked the dll version also by binary decompiling (ILDasm) and it is reporting correctly 4.0.
The odd thing is that I have this problem only on latest version of dll and not on previous one (12) that works smoothly. It seems that in some condition the .net framework build compiler got confused on dll default targeting behaviour I must take some time to make a standalone clean virtual environment to make isolated tests to reproduce it and understanding what's happening

Hi I made some tests, and we can definitely archive this. The behaviour is specific for the machines where I had the problem and I can't reproduce on other clean machines.
I think the msbuild error is misleading perhaps for a bug of the specific toolset used on these machines. What I think is that the compiler get confused on how to retrieve the correct dll referenced by the current project and somehow it finds a wrong version on some machine path.
The error goes away simply changing dll version, toolset or also putting the correct newtonsoft linked dll in first library reference directory.
Thank you anyway for the help


[PLINT-519] Fix parsing voltage fields in Nvidia Jetson Integration

What does this PR do?
Updates the regex pattern that parses Nvidia Jetson voltage fields to parse the mW unit if its present.
Motivation
Fix the error could not parse voltage fields when tegrastats output contains units in the voltage field such as VDD_IN 5299mW/5299mW. See #17678
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes
The current regex will only accept the mW unit? Should we accept other units? The tegrastats docs only mentions the milliwatts (mW) as power unit, no other units are mentioned.

/merge


Clarify the difference between jest.runAllTimers and jest.runAllTicks

I read the docs for runAllTicks and runAllTimers ~3 times before I finally realized one was for the micro task queue and the other is for the macro task queue. I'm sure other people have done the same thing, so I think it would help to bold the "micro" and "macro" text.

thanks!

@facebook-github-bot shipit