Could not resolve dependency: npm ERR! peer next@"^12 || ^13" from<EMAIL_ADDRESS>npm ERR! node_modules/next-contentlayer npm ERR<EMAIL_ADDRESS>from the root project

Hi Contenlayer Team,
Thank you so much for this amazing solution. We really enjoy working with it.
Today, our build failed in the deployment environment and I found that the issue behind this is the version compatibility between NextJS 14 and the ContentLayer version.
In my local, I installed the dependency using legacy-peer-deps, and everything worked well, however, when I pushed the code and during the build time, I got this issue:

This my azure-pipeline.yml
# Node.js with React
# Build a Node.js project that uses React.
# Add steps that analyze code, save build artifacts, deploy, and more:
# https://docs.microsoft.com/azure/devops/pipelines/languages/javascript

trigger:
- main

pool:
  vmImage: ubuntu-latest

steps:
- task: NodeTool@0
  inputs:
    versionSource: 'spec'
    versionSpec: '10.x'
    checkLatest: true
  displayName: 'Install Node.js'

- script: |
    npm install
    npm run build
  displayName: 'npm install and build'

Try to install using npm install --force

Getting same error with NextJS 14. npm install --force every time is not a stable option. Any plans on fixing this?

I temporarily used npm install --lagacy-peer-deps  but I faced issues with the CI/CD because you have to apply this also during the deployment process.


Add an Ingress to act as a load balancer for the kubernetes cluster

Description:
Add an Ingress to the kubernetes cluster to act as the load balancer of it. When user adds an integration custom resource, ingress should be updated with the newly added ingress paths. Also, this ingress creation should be customizable via the ConfigMap. If a user wants to create an Ingress automatically by the operator, he could add autoIngressCreation: "true" field to the ConfigMap.
Suggested Labels:
Type/Improvement
Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Fixed in - https://github.com/wso2/k8s-ei-operator/pull/15


Add another OTel test for the ingestor to replace old Argo e2e test

What does this PR do?
It creates another test for OTel that should replace the old Argo test by testing the OTLP ingestor on the "normal" agent.
It also removes some files related to the old Argo e2e test

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


missing data for LED1624G9 which is listed in the const.py

    "TRADFRI bulb E27 CWS opal 600lm": "LED1624G9" seems to be missing in the data pack directory

It won't break anything. Could be removed, but I suggest to just keep the entry, so it won't have to be added again in the future.

well, it's because I had added my own 'fake' data files for some models, both in the const.py file and the actual data files.
When I noticed you had added the model in the const.py file, I figured the model to be available in the data folder too. Since that wasn't the case, my entities weren't created any longer....
Ive now changed back to my own const.py file which lists the models in a way I can clearly see my own additions:
MODEL_DIRECTORY_MAPPING = {
    "IKEA of Sweden": {
        "TRADFRI bulb E14 WS opal 400lm": "LED1536G5",
        "TRADFRI bulb GU10 WS 400lm": "LED1537R6",
        "TRADFRI bulb E27 WS opal 980lm": "LED1545G12",
        "TRADFRI bulb E27 WS clear 950lm": "LED1546G12",
        "TRADFRI bulb E27 opal 1000lm": "LED1623G12",
        "TRADFRI bulb E14 W op/ch 400lm": "LED1649C5",
        "TRADFRI bulb GU10 W 400lm": "LED1650R5",
        "TRADFRI bulb E27 WS opal 1000lm": "LED1732G11",
        "TRADFRI bulb GU10 WW 400lm": "LED1837R5",
        "TRADFRI bulb E27 WW 806lm": "LED1836G9",

        "TRADFRI bulb E27 CWS opal 600lm": "LED1624G9",
        "TRADFRI bulb E27 WW 806lm": "LED1838G9",
        "TRADFRI bulb E27 WW clear 250lm": "LED1842G3"
    }
}

and have included the corresponding data files. My entities are again created.
My suggestion would be to only list available models in the const.py files, and not 'future' files.

There is no need to alter these python core files to include your own light models. This is strongly discouraged as you cannot upgrade the component anymore easily.
You have two options out of the box to include your own models:

Make a manufacturer/model folder in config/powercalc-custom-models. See https://github.com/bramstroker/homeassistant-powercalc#light-model-library. Powercalc will look there first.
Use the configuration option custom_model_directory, see https://github.com/bramstroker/homeassistant-powercalc#sensor-configuration. This allows you to define a custom directory in which the light model data is loaded from.

thanks, very nice indeed. And I will add these to those directories/config.
(Though is was very easy, simply c&p those 3 lines to the const.py file.)
Still, my original issue with this was, that when the const.py lists models that aren't included by means of the integration, it is confusing to the end user that might have included these as custom files/models.
So, its not as much an issue of altering the core files to include own models, it is more an issue of the integration Not including models it maps in the const.py file as supported....


Update .travis.yml

Small build update/cleanup

Hmm. For some reason this PR didn't trigger a build.
Please review and feel free to adjust.
I believe current build may fail because of caddy now using go modules.


Fix Ant Menu Deprecated Warnings

Is your feature request related to a problem? Please describe.
Ant Design Menu component suggest to use items instead of children for better performace.
Describe the solution you'd like
Check sidebar and Dropdowns page and fix console warnings.

working on it

Fixed in develop


[Igandhi2][Feature]

Create an Issue and link it here
Close the issue
Follow the development cycle that was followed in Project 2


fix proxy url join error

What type of PR is this?
kind/bug
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

cc @xiaodongluo


CASMINST-5220: Update rbd instructions

Description
This removes the shebang line from the rbd mapping script, and updates the instructions for running it so that python is called instead of calling the script directly.
This is a draft PR because the steps to initialize the venv on ncn-m001 is still required.
Checklist Before Merging

[X] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[X] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[X] My commits or Pull-Request Title contain my JIRA information, or I don't have a JIRA.

/backport backport/2233-to-main

Not ready to merge yet, for two reasons:

I identified some further things which need to be adjusted, based on my help on the frigg upgrade
The RPM build is now timing out. I am thinking it may be because of the large tar file now being added to the repo, I think I may be able to work around it, though. Looking at that.

Moving to draft PR in the meantime, to prevent merge.

Once again ready for review and merge


[Bug] Menu opened from taskbar can be immediately closed

Description
When you open context menu at taskbar and trying to get access to menu items, your menu can completly close. Next time it will work fine, as it has to be.
How to reproduce?
https://github.com/moudey/Shell/assets/47476146/30a2b89f-643f-4bbf-8e21-ccef5e82fd64
Explanation: mouse left-clicking highlighted in yellow and right-clicking in blue. Desktop refreshing returns back the bug.
Details:

Windows 11 Pro 22H2 Build 22621.2283
Nilesoft 1.8.1

I've also attached my menu-item files 'cause it might be my fault, or they could be useful for bug research (changed .nss file extension to .txt). As for me, there is nothing complex in them.
browser.txt
develop.txt
games.txt
life.txt

the problem has also appeared with other users - with two programs that try to change the Windows Shell at the same time
find which program does it and write back

I've completly forgotten about. Yes, I am using Start 11 (latest 1.47 Steam-version).

I don't remember if this problem was left to be solved after the next release version (the new version is built from the ground up and this problem may not exist then) or if it was fixed in one of the debug versions (https://discord.com/channels/<PHONE_NUMBER>707168318/1106387015425069151/1154640833396621413)

Ok, I'll check it out and let you know, if this bug is gone.

I tried the latest 47 debug-build and got "Register not complete error"
Log:
Nilesoft Shell
version 1.8.47 64-bit
<EMAIL_ADDRESS>https://nilesoft.org
Windows 11 Pro 64-bit, version 10.0.22621.2283
2023-09-23 15:33:07 [error] Register not completed.

https://discord.com/channels/1106387012707168318/1106387015425069151/1154640833396621413

Oops, sorry 😅
After short adaptation of my files to new .nss syntax changes, that you had made, I tested it and bug still appeared in the same scenarios.


Fix docker/travis

Fixes the Dockerfile and unit tests. Since we are currently using the docker build for Travis, this should also fix the Travis build.

This looks good for me.
@joshmoore - can you do a final check and merge? Thanks.


On provision, only show progress if not 0

I'm seeing this:

Which makes me think something is wrong.
I'd say we only show the message if the numbers aren't 0

@weikanglim Can this be closed now?

Yes, closing.


Update torchdyn requirement and actions versions

What does this PR do?
Upgrade torchdyn in the requirements after release 1.0.5
Fixes #3 permanently
Fixes requirement to torch<2.0.0 as torch>=2.0.0 creates incompatibilities between torch 2 compatible lightning bolt versions and pytorch-lightning=1.8.3 other utilities should work just fine.

To get tests to pass I had to add torch < 2.0 this is not ideal as torchcfm has no issues with 2.0, but the running package does.

Codecov Report
Patch coverage: 100.00% and project coverage change: +3.05% :tada:

Comparison is base (2abd38f) 31.18% compared to head (e8e848d) 34.23%.
Report is 14 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
+ Coverage   31.18%   34.23%   +3.05%
==========================================
  Files          46       55       +9
  Lines        5946     6268     +322
==========================================
+ Hits         1854     2146     +292
- Misses       4092     4122      +30

Files Changed
Coverage Δ

runner/src/utils/__init__.py
100.00% <ø> (ø)

runner/src/utils/rich_utils.py
82.60% <ø> (ø)

runner/src/utils/utils.py
65.74% <ø> (ø)

runner/src/models/components/augmentation.py
63.68% <100.00%> (ø)

runner/src/models/components/simple_mlp.py
70.83% <100.00%> (ø)

... and 10 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Split into runner-requirements.txt (for runner) and requirements.txt (for torchcfm package)

does the split requirement split fixes the torch2.0 requirements?

yes, runner now requires torch<2 but torchcfm does not


operator& can be overloaded as a free function

When checking if operator& has been overloaded you should check if it is a free function as well.
Let me know if I've missed anything

Could you point to the code that is doing the check. I don't believe the code is able to check if operator& has been overloaded as a member or free function, as you just check for valid expressions.


[SPARK-17642] support DESC FORMATTED TABLE COLUMN command to show column-level statistics

What changes were proposed in this pull request?
Support DESC FORMATTED TABLE COLUMN command to show column-level statistics.
sql: desc formatted stats_table cbool
output:
+--------+---------+-----+----+---------+--------------+-----------+-----------+-------+
|col_name|data_type|  min| max|num_nulls|distinct_count|avg_col_len|max_col_len|comment|
+--------+---------+-----+----+---------+--------------+-----------+-----------+-------+
|   cbool|  boolean|false|true|        1|             2|          1|          1|   null|
+--------+---------+-----+----+---------+--------------+-----------+-----------+-------+

sql: desc stats_table cbool  or desc extended stats_table cbool
output:
+--------+---------+-------+
|col_name|data_type|comment|
+--------+---------+-------+
|   cbool|  boolean|   null|
+--------+---------+-------+

How was this patch tested?
Added tests

@wzhfy

i am closing this


Restructure FlightTaskAutoLine

add FlightTaskAutoMapper that handles the different types of waypoint and generates setpoints for all types except of position and loiter
FlightTaskAutoLine generates the setpoint types position and loiter if the flight between waypoint is a straight line

Logs:
https://logs.px4.io/plot_app?log=f0a1f1d0-208f-4985-af62-a4fa07b31f62
https://logs.px4.io/plot_app?log=429c68a5-fbe1-4e42-800c-dc9ea8a8109a
https://logs.px4.io/plot_app?log=d4bd0426-3ba1-4075-a3df-44fb784f494d

@PX4/testflights can you pleas test all Auto related flight maneuvers?

takeoff
land
rtl
mission
loiter
Goto horizontally and altitude

couple flights on a Pixracer (v4):
https://logs.px4.io/plot_app?log=b6ed0f37-ed3c-4716-ae80-ce9fb1a2e397
https://logs.px4.io/plot_app?log=71511cdb-6380-4435-ac7d-306966d049b7
auto take-off; good tracking; interrupted mission for hold; land and rtl working properly


divided fields into fieldsets; added initial data; added tooltip about server input dependencies

also increased the spacing between section header and text fields, and changed color of fieldset headers to secondary as suggested in #130.
closes #130

thanks - some frontend tests seem to be failing though

Ah, right. I will fix it today.

I've fixed the tests. Also, the screen, because I forgot to add it before:

Awesome, thanks! :)


Import OpenSSF Criticality Score into dashboard

[x] Create the loader for the criticality data.
[x] Build Grafana widgets to show the data.

Fixed via #38


Fix build failure in verbs

While looking into #22372, the verbs build fails with the following error:
tensorflow/contrib/verbs/verbs_server_lib.cc:80:6: error: 'once_call' in namespace 'std' does not name a type
 std::once_call reg_mem_visitors_call;
      ^
tensorflow/contrib/verbs/verbs_server_lib.cc: In member function 'tensorflow::Status tensorflow::VerbsServer::Init(tensorflow::ServiceInitFunction, tensorflow::RendezvousMgrCreationFunction)':
tensorflow/contrib/verbs/verbs_server_lib.cc:85:18: error: 'reg_mem_visitors_call' was not declared in this scope
   std::call_once(reg_mem_visitors_call, []() { RdmaMgr::RegMemVisitors(); });

This fix fixes the build failures with once_call -> once_flag.
This fix fixes #22372.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Paul, can u take a look?

@yongtang Mind to apply the same fix to GDR? I see the same issue introduced in commit 33170cc661f3838aa7d0d7fc19bb0c6ba4812a3c


Fix vulnerability caused by filename escaping

Detail:
https://github.com/kittinunf/fuel/issues/876
~Local tests have not yet been run.~
~(I need to set up the Android-SDK perimeter, which is taking a bit of time.)~

Hey thanks so much for the fix! I will take a look on this tonight.

@kittinunf Thank you for your review 🙇
Could you review it again?

@motoyasu-saburi I will release the patch for this soon please hang on tight.

@kittinunf thank you for the library. When are you planing to push the v2 release?


Unable to read balances

Hello,
I'm running carbonvotes alongside a fully-synced local geth node (non-archive mode). However, it fails to record any votes because it always returns zero balance:

the root of the issue is missing trie node when trying to fetch balance of user account at particular block (https://github.com/lrettig/carbonvote/blob/master/lib/carbonvote/contract.rb#L26).

Has something changed? Why is balance coming back as zero?
Thanks

eth_getBalance requires blockNumber param to be as hex input instead of number.
Fix --> Replace : default_block_option_pending with :number_to_hex in the below line
https://github.com/EthFans/carbonvote/blob/d667f8ba4471d9f2b699f84eda040a3d9a7372e8/lib/geth/node.rb#L25


Only update sources to be mutated in the sandbox

Is your feature request related to a problem? Please describe.
One part of issue #2403 is that when we add file headers and strip comments, it leads to issues between the React library (https://github.com/facebook/react) and Jest. This can be solved by excluding the JavaScript sources, like I proposed in #2406, but it got me thinking about a more general solution.
If we would only update sources to be mutated in the sandbox, and keep the other sources as-is, it may avoid this kind of an issue with some other projects in the future.
Describe the solution you'd like
rework the sandbox setup as follows:

get all source files, like Stryker already does
only match files to mutate with the sandbox patterns
if a file both is in the files-to-mutate list and matches sandbox.fileHeaders pattern, then add the header
if a file both is in the files-to-mutate list and matches sandbox.stripComments pattern, then strip the comments

Describe alternatives you've considered
updated:

my proposal in PR #2412 should solve the same problem with smaller projects
exclude **/*.spec.* and **/*.test.*, like I proposed in PR #2406
exclude sources in **/__tests__/**, **/tests/**, **/test/**, **/spec/**, like I tried to do in PR #2405
configure sandbox to not update any sources, like I did for React as described in issue #2403

Additional context
I tried working on this idea myself. I think this should be pretty straightforward with Injector from typed-inject but do not yet really understand how to work with the typed-inject API.

P.S. After some testing, I think this would not solve the problem with the React library. So this idea may not be worth considering for the near future.

Hi @brodybits once again. I overlooked this issue, sorry for the late(r) reply.

If we would only update sources to be mutated in the sandbox, and keep the other sources as-is, it may avoid this kind of an issue with some other projects in the future.

Unfortunately, this won't work for the typescript use case, as stated in #2415
For example:
// concat.ts
function concat(a: string, b: string) {
   return a + b; // return type of this function will change because of mutation switching
}
// __tests__/concat.spec.ts
it('should concat', () => {
  const actual: string = concat('a', 'b');
  expect(actual).eq('ab');
});

In this case, __tests__/concat.spec.ts will give a compile error. You would notice it if you're using buildCommand: "tsc -b" or you use jest-ts for example, the test runner that supports type checking.
The solution I came up with is to prefix all javascript-and-friend files in the sandbox with // @ts-nocheck. That way, you'll disable type checking all together (works from TS3.7~ish). The nice thing is that this works for all typescript use cases.
Unfortunately, other // @ts-xxx directives can still interfere with // @ts-nocheck, that's why I've decided to also add the "stripComments" (or "removeComments") preprocessor. It removes all other comments from a file.
Both these preprocessors are on by default, because I thought that removing comments is usually safe to do. But then again... this is JavaScript we're talking about 😉.

think this would not solve the problem with the React library. So this idea may not be worth considering for the near future.

I'm really curious to know what the root cause of these issues is. I know that you can override jest's test environment with the /** @jest-environment */ directive, but you are alluding to something todo with the react library itself.

Closing this issue in favor of #2438


[fix][query] Correct query server legacy condition

Which problem is this PR solving?

Towards #6026

Description of the changes

Fixes an issue from https://github.com/jaegertracing/jaeger/pull/6055 where the condition for routing to the legacy implementation was accidentally flipped.

How was this change tested?

CI

Checklist

[x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[x] I have signed all commits
[x] I have added unit tests for the new functionality
[x] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: yarn lint and yarn test

kinda sad that no tests caught this


retrieving/parsing db and conditionals for db.where

I would like to keep a local db for my bot to pull from and add to for things such as blacklisted spam, brief chat history (so it can be included in ban logs), etc. Here's an example of what I'm attempting to do with blacklisting:
function isBlacklistSpam(str, callback) {
	var spam = false;

	// get the list of blacklisted words, and compare with str
	db.list('blacklist').then(function(result) {
		console.log(result);
		for (i = 0; i < result.length; i++) {
			if (str.indexOf(result[i]['word']) >= 0) {
				spam = true;
			}
		}

		return callback(spam);
	});
}

The blacklisted words are added via mod command:
var msg = message.toLowerCase().split(" ");

// blacklisting command for banlist
if (msg[0] === '!blacklist' && isMod(channel, user) && msg.length > 1) {
	db.insert('blacklist', {word: msg[1]}).then(function() {
		console.log("Added " + msg[1] + " to blacklisted words/links.");
		client.say(channel, "Added " + msg[1] + " to blacklisted words/links.");
	});
}

And here's the check against if there's blacklisted content in chat:
// spam checks
isBlacklistSpam(message.toLowerCase(), function(spam) {
	if (spam) {
		isMod(channel, botname, function(mod) {
			if (mod) {
				client.ban(channel, user.username);
				client.say(channel, "All spammers must die, " + user.username + ".");
			} else {
				console.log(channel + ' | ' + user.username + ' has posted a malicious link.');
			}
		});
	}
});

I'm unable to grab the 'word' element of the blacklist database via the means in isBlacklistSpam(), do I need to parse database output in a special way? Or at least, even if I enter a blacklisted word into chat, the bot doesn't resolve it. I'm uncertain of what I'm doing wrong here.
Here's another example of where I'm having issues with database usage, where I'm tracking users and their chat history:
// adding users to a db
db.where('userdata', {name: user.username} && {chan: channel}).then(function(result) {
	var results = result['items'];
	console.log(result);
	if (results.length <= 0) {
		//  adding to db with user obj, chat history, etc
		db.insert('userdata', {name: user.username, chan: channel, user_data: user, messages: [message]}).then(function() {
			console.log(colors.yellow("Added userdata for " + user.username + "."));
		});
	} else {
		results = results[0];
		// user exists already in db
		var cid = results['cid'];
		console.log("CID: " + cid);
		var msgs = results['messages'];
		msgs.push(message);
		console.log(msgs);
		// pop excess messages from front for storage
		if (msgs.length > 10) {
			var rem = Math.abs(10-msgs.length);
			msgs.splice(0,rem);
		}
		db.update('userdata', cid, {user_data: user, messages: msgs}).then(function() {
			console.log(console.yellow("Updated data for " + user.username + "."));
		});
	}
});

There's a few issues above, although I've changed things up to try and get it to work so many times I'm having trouble keeping track of what happened where anymore.
The first is that if it matches, it will match the second user in the db every time and update their user_data and messages instead of properly pulling the correct user or identifying that they are not in the db. The database never grew past the first two people to chat.
Second, and I know this is more of me being a Js newbie, but I can't push the new message onto the messages result, instead, it sets the msgs var as the integer 2. EDIT: well, this was a problem.. it seems to be working fine now.
Finally, I'm unsure of how to use db.where properly. In the docs it shows a few different ways, but they all use statically typed examples. How do I properly do this with dynamic variables (such as username/channel). I have a feeling that is the root or at least part of the issue I'm having here, as in my console output I can see that it's matching when 'name' is not the same as user.username in the element it retrieves, it seems to be matching purely off the second condition of channel.
Apologies for my mistakes here, it's still very much all new territory for me and I hate clogging up the issues board with my own personal errors, but I really appreciate the help.

I probably won't answer all of you questions, but here's a few things that I'm doing, maybe it can help:
I tried to use the db library from @Schmoopiie, but I couldn't deal with every function being a promise/callback, so I actually use lowdb (https://github.com/typicode/lowdb). It uses underscore/lowdash to interact with the database and is extremely straightforward in usage.
With that said, here's my highlight message function, which you could probably use to determine if a message contains a blacklisted keyword:
var _s = require("underscore.string");
...
server.locals.database = path.join(__dirname, "assets", "database.json");
...
var database = lowdb(server.locals.database);
var keywords = database("keywords");
...
function highlightMessage(comment) {
    var casedComment = comment.toLowerCase();

    var highlight = keywords.find(function(keyword) {
        return _s.contains(casedComment, keyword.value.toLowerCase());
    });

    return !_.isUndefined(highlight);
}

One more note, the where function is usually only used with a key/value pair, like db.where({id: 3}). Not sure about that lib, but in lowdb you would use the filter command.
Let me know if some of that helps, or you have more questions!

Thanks for the feedback/alternative suggestion, however I would prefer to utilize as many modules as possible from one author as it's more likely that they'll already be integrated and work well together. It's one of those things where I know my inexperience in matters would likely make adapting to a new library much harder than it needs to be. If it winds up that I simply cannot do what I want to do, then I will definitely consider expanding my horizons!
I noticed that by removing the conditional {name: user.username} && {chan: channel} in the db.where call to just {name: user.username} that all is well and fine, and it correctly pulls the appropriate table/element (or whatever it's called in proper database-speak). The only issue there is that I'd like to keep separate records for users if they're around on more than one channel (like myself).
I tried parsing it out with the alternative syntax to see if I could "trick" it into taking dynamic variables, i.e. db.where('userdata', ('@name == ' + n + ' && @chan == ' + channel))[...] but that didn't work either.
I suppose the other way to do it is to store an array of channel data in each name's entry, which I will attempt now!

Not sure if you found it already, but here's the relevant documentation for the twitch irc db: http://www.twitch-irc.com/docs/twitch-irc/Modules/Database/Where
So it looks like you should be able to do what you want.

Indeed, I've been using the docs extensively, it's the only way I've really been able to do much of anything! In my last post I mentioned just storing everything under one entry per-user, and that's been working fine so far. :+1:


feat: custom resource config for teams

This PR offers the ability (if required) to adjust the resource configuration of Tekton Tasks, Grafana, Tekton Dashboard and Alertmanager. Summary:

no resource defaults in teamConfig
no changes in apl-api
platform-admin can add resource customizations directly in values repo
only for edge cases where the defaults are not sufficient

Adjustments can be made by adding:
teamConfig:
    demo:
        resources:
            alertmanager:
                enabled: true
                resources:
                    limits:
                        cpu: "5"
                        memory: 5Gi
                    requests:
                        cpu: 5m
                        memory: 5Mi

Checklist

[ ] Architecture Design Records have been added as adr/*.md and appended to list in adr/_index.md, if applicable.
[ ] The values-schema.yaml file and test/** fixtures have been updated to reflect code changes, if applicable.
[ ] The OpenApi Schema from redkubes/apl-api project is compatible with definitions from values-schema.yaml file, if applicable.
[ ] Helm releases are meeting otomi's baseline security policies, if applicable.
[ ] Helm chart and helmfile changes are tested against upgrade scenario, if applicable.

Superseeded by: https://github.com/linode/apl-core/pull/1819. Will close this one.


[incubator/jaeger] Use additional cli parameters for collector and query components

What this PR does / why we need it: Uses the already existent collector.cmdlineParams and query.cmdlineParams in the deploy.yaml in order to enable dynamic configuration. I need to set es.sniffer to false to use this chart with AWS ElasticSearch service

/assign @mgoodness

Why do you need command line flags? All jaeger components accept environment variables with the same name as the flags, uppercase, where the symbols - and . are replaced by _. So if you need to add more params, why don't you just add them to the documentation and to the config map?

@pavelnikolov makes sense. Either way, the Charts docs suggests using this command line params in the situations where the needed variables are not configured but it is not really implemented ATM.
I will keep this Pull Request open in order to fix this and will open a new one adding this variables to the char!

@B0go @pavelnikolov  When the chart initially introduced it had this exact functionality, but it was removed in favor of using the environment variables.


P2P NodeDatabase vs NetworkManagerContract

Hi team,
Just curious why NetworkManagerContract was introduced instead of using what's exposed by quorum/node/api.go ?
Currently case is that on any peer connected to cluster it's possible to interact with NetworkManagerContract, execute its registerNode and add completely invalid peer (for example with public key = wqfqwf and enode id = wqfqwf).
Another outcome of using NetworkManagerContract right from start leads to fact p2p server is absolutely not aware of peers connected via smart contract.
Thanks in advance for understanding in case I missed pretty obvious reasons for this approach to be implemented.

This contract is used for creating a Network Map Service. This is more of an application service rather than Quorum specific. In a larger application, multiple nodes may provide competitive services and clients can lookup to get details of the node including public key to send transaction.
Since Quorum is intended to be used in a persmissioned environment, any node trying to fake the details could be identified and punished. In a future release we would like to introduce an on-boarding service, that would further secure this updates to this contract.
Thanks for your question. Please feel free to discuss further.

@dhyansraj ok, your aim is more less clear, current step could be considered as initial and small.
Just to clarify a bit more - this step is damaged currently. By damaged I mean complete absence of such NodeDB due to approach you enforce with current QM version - node API is not called in any way, means p2p server becomes aware of dynamically joined peer only when peer starts communications over whisper and constellation ports. Such live way of peer appearing doesn't match any of implemented ways node offers to add peer, nothing is reflected in NodeDB. Hence cluster consistence relies only on fact peer node did self-configuration via genesis and other params fetched from master when join happened.

we still use raft.addPeer for adding new nodes. Contract is just for display purpose.


How to run from source?

Hi,
I'm wondering how can I run this from source?
I've done:

npm install
npm run build
pipenv shell
pipenv install

All worked. Now how to start the server?

ok, I found that I can run the server with:

python -m uvicorn main:app --app-dir flatnotes --host <IP_ADDRESS> --port 8080


Fix #863 - Added a check for good pull from OWM and return if the object is None

If the OWM API returns an empty or None response, we will log the error and return.  The sensor will refresh at another time.
Possible future enhancement is to overrule the @Throttle decorator to retry, but probably not necessary at this time.  (I couldn't get it to fail unless I deliberately did something dumb)

Related #863

:dolphin:

Thanks @zmrow


Attachment upload breaks on a new version of 'request' module

The attachment didn't get uploaded when using<EMAIL_ADDRESS>(latest version).
However when using<EMAIL_ADDRESS>the upload still works.

Here's a list of the changes made from request 2.58.0 to 2.59.0: https://github.com/request/request/blob/master/CHANGELOG.md#v2590-20150720
Not sure what change could cause the upload to fail though.

<EMAIL_ADDRESS>fixes this. Closing now :)


No definition found for xy (F12)

Issue Type: Bug
Hi,
we have a very large SQL database (over 40,000 objects in total)
If I now select a table in a select and press F12 for definition, the message "no definition found for xy" appears.

I can repeat this several times ...
Then I wait 3 minutes and press F12 again, it takes a few seconds but then it shows the table definition.
All in all, this process takes a long time and the message that no definition was found is of course not nice. I always think I have a typing error or something. Then I wait a few minutes and it works. Could ADS not create an internal cache for this search? Or is there a way to speed up this process somehow?
Thank you and greetings Oliver
Azure Data Studio version: azuredatastudio 1.36.2 (2299a76973585fa962cfd06e8acab4abb41567de, 2022-05-19T04:18:20.649Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No

please reopen if it doesn't help.

I have now tested this with the Insider build. But no change to version 1.36.2. Exactly the same behaviour.
Does this automatically reopen the ticket, because I can't find a ReOpen button here.

@OSch24 what is your network latency to the target sql server?

@OSch24 Wie hoch ist die Netzwerklatenz für den SQL Server-Zielserver?

ca. 11 ms

Today the new official version 1.38 was installed.
The behaviour has not changed.
This is just for your information, in case it is important.

I don't know if it's important, but I currently have version 1.48 installed and it still has the problem.

Same in 1.49

Our database is really very, very large. Over 100 GB... but still, if I go directly to a procedure, it could "grab" just that one and show the preview or call it as an alter... with small databases it works every now and then.


the hook about time have an error

CreatedAt and Updated timestamps and are managed by pop. Currently they’re not user setable.


Error trying to import t5x

https://github.com/google-research/t5x/commit/bd1fa1c994ef57cbaf8bc1d182cd08779bf5338b#r141544411
After cloning this repo, I'm getting the following error when trying to import the t5x library that seems related to this change? Let me know if I can provide more info to reproduce this
import t5x
TypeError Traceback (most recent call last)
in <cell line: 1>()
----> 1 import t5x
3 frames
/usr/local/lib/python3.10/dist-packages/t5x/init.py in
15 """Import API modules."""
16
---> 17 import t5x.adafactor
18 import t5x.checkpoints
19 import t5x.decoding
/usr/local/lib/python3.10/dist-packages/t5x/adafactor.py in
63 import jax.numpy as jnp
64 import numpy as np
---> 65 from t5x import utils
66 from t5x.optimizers import OptimizerDef
67 from t5x.optimizers import OptimizerState
/usr/local/lib/python3.10/dist-packages/t5x/utils.py in
46 import orbax.checkpoint
47 import seqio
---> 48 from t5x import checkpoints
49 from t5x import models
50 from t5x import optimizers
/usr/local/lib/python3.10/dist-packages/t5x/checkpoints.py in
2070
2071
-> 2072 @ocp.args.register_with_handler(
2073 DatasetCheckpointHandler, for_save=True, for_restore=True
2074 )
TypeError: register_with_handler() got an unexpected keyword argument 'for_restore'

After checking out the commit before this, I can confirm there are no issues importing the t5x library @cpgaffney1

Related:

https://github.com/google-research/t5x/pull/1547

After checking out the commit before this, I can confirm there are no issues importing the t5x library @cpgaffney1
I have been attempting to use several commits preceding the one recommended, such as commit e4e262b, but I am still encountering the same error. I am unsure if there is an error in my approach and would greatly appreciate your guidance.
Here are the steps I have taken:

Executed the "Setup Environment" cell.
Added and ran the following code: !pip install git+https://github.com/google-research/t5x.git@e4e262b5bbd81e7ca42d83203ade3db0600f5779
Added and ran this code: import os; os.kill(os.getpid(), 9)
Executed the "Imports and Definitions" cell.
Subsequently, I received an error message.
I would be immensely grateful for any assistance you could provide.

!pip install --upgrade orbax-checkpoint==0.5.8 works as a workaround for me.

!pip install --upgrade orbax-checkpoint==0.5.8 works as a workaround for me.

Thank you very much for your help! Your method was very effective and solved the problem.

Not sure if this is relevant/related, but when I was applying this fix/workaround:

!pip install --upgrade orbax-checkpoint==0.5.8 works as a workaround for me.
Originally posted by @garzon in https://github.com/magenta/mt3/issues/151#issuecomment-2123348933

I got the following error:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
tensorflow 2.15.0 requires ml-dtypes~=0.2.0, but you have ml-dtypes 0.4.0 which is incompatible.
tensorflow-cpu 2.15.0.post1 requires ml-dtypes~=0.2.0, but you have ml-dtypes 0.4.0 which is incompatible.
Successfully installed ml-dtyp

Documenting here in case it is relevant.


BUG: scipy.signal.periodogram window parameter

Describe your issue.
When trying to pass an array as the window for scipy's periodogram, the behavior is inconsistent with the documentation. In the doc it says:
window : str or tuple or array_like, optional
Desired window to use. If window is a string or tuple, it is
passed to get_window to generate the window values, which are
DFT-even by default. See get_window for a list of windows and
required parameters. If window is array_like it will be used
directly as the window and its length must be nperseg. Defaults
to 'boxcar'.
However there is no nperseg parameter for scipy.signal.periodogram! Moreover, a confusing error message is thrown, as shown below.
I guess it's intended use is for windows precisely the size of the signal x. However in a signal analysis set up, it can be meaningful to first convolve the signal with a (shorter) window before performing the FFT (think moving average, although it's often a bad idea). When using the nfft parameter shorter windows work but I assume the signal gets truncated?
Reproducing Code Example
import numpy as np
from scipy.signal import windows, periodogram

nSamples = 1000
win_size = nSamples // 4
signal = np.random.randn(nSamples)

# manual short hann window as array
hann = windows.hann(win_size)

freqs, pxx = periodogram(signal, window=hann)

# this works, however doesn't fit the documentation
# not clear what exactly is happening here.. does it truncate the input signal?
# freqs, pxx = periodogram(signal, window=hann, nfft=win_size)

Error message
~/.local/lib/python3.8/site-packages/scipy/signal/spectral.py in _triage_segments(window, nperseg, input_length)
   1977         elif nperseg is not None:
   1978             if nperseg != win.shape[0]:
-> 1979                 raise ValueError("value specified for nperseg is different"
   1980                                  " from length of window")
   1981     return win, nperseg

ValueError: value specified for nperseg is different from length of window

SciPy/NumPy/Python version information
1.4.1 1.18.4 sys.version_info(major=3, minor=8, micro=10, releaselevel='final', serial=0)

Thanks for the bug report @tensionhead

I guess it's intended use is for windows precisely the size of the signal x.

Yes, I believe that is correct. This is also what ends up happening under the hood when you give a string name for the window (e.g., 'hann'). In periodogram, nperseg is set to the length of the axis and then in _triage_segments:
win = get_window(window, nperseg)

There win is an array with length nperseg.
The docstring for periodogram says " If window is array_like it will be used directly as the window and its length must be nperseg." Here nperseg should be replaced by "equal to the length of the axis over which the periodogram is computed".

However in a signal analysis set up, it can be meaningful to first convolve the signal with a (shorter) window before performing the FFT (think moving average, although it's often a bad idea).

The current way to achieve this would be to use ndimage.convolve1d I think.

When using the nfft parameter shorter windows work but I assume the signal gets truncated?

Only works if you set nfft to the window size you are using. I'd say that that does a different (worse) thing than ndimage.convolve1d, because it tries to use an FFT-based convolution while what I think you want is a regular convolution - then any short window size works fine.
To resolve this issue (as @andfoy is doing in gh-16776), I think we need:

An updated parameter description for window, as I suggested above,
A clearer error message that explains that if window is an array, what its length must be given the shape of input x,
A regression test checking the error raised is the correct one.

Thanks for getting back on this @rgommers! I think I actually misinterpreted the window parameter here, I assumed it's to convolve it with the input signal. However it's merely a multiplication (like 'tapering'), and hence this size restrictions make total sense of course!
But of course it's good to see that the slightly confusing error messages are getting worked on!

Great, thanks for confirming @tensionhead


hyprlandPlugins.hyprsplit: init at 0.43.0

Description of changes
Add hyprsplit plugin
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4579

@fufexan can you have a look please? I would like to get it merged

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1983


[WIP] Switch to master branch for keycloak repo

We got the upstream changes merged so we can use almighty/keycloak master branch instead of f8-custom.
FYI: @alexeykazakov

We will test keycloak master tomorrow and if it's good then we will be ready to merge this PR.


The default-gateway{,-v6} feature was lost during the libnetwork merge

This feature was added by @lebauce in https://github.com/docker/docker/pull/9381, shortly before the libnetwork mega-merge. After the merge, only the CLI flags were left in daemon/config_linux.go, but the values were not set in the new bridge network driver.

@eolamey we did port this feature into libnetwork. Why do you think the values are not set in the new driver

@eolamey I see what you mean. We did port the feature but it looks like we are not passing the cli flags to libnetwork when the feature is configured. We will fix this.

@eolamey I see you also made a PR to fix this. Let me review that instead.

Wow, @mrjana, we are way to fast for me :) I was preparing the PR, which I just properly created. I added integration test so that this won't get lost again :)


Do you plan to bring in layout support as you've done in https://marmelab.com/ng-admin-demo/index.html#/dashboard

Particularly, Looking for

Collapsible menus on the left hand side.
Drop down menus from the nav bar (right hand side)

I tried to use your MyLayout (demo code: https://marmelab.com/admin-on-rest//Theming.html#using-a-custom-layout)
I encountered error:
Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in.
This happens the moment I include <Admin appLayout={MyLayout}  restClient={restClient} authClient={authClient} dashboard={dashboard} catchAll={pageNotFound} >
Problem goes away when I take out appLayout={MyLayout}
I do have import MyLayout from './MyLayout' in App.js
Any thoughts/pointers are appreciated.

Duplicate of #1025. Sorry we forgot to update this part of the documentation. Please refer to the linked issue for more info

Documentation updated in reference to #1025, closing this one as duplicate.
As for trying your own implementation, I advise you to ask for help in StackOverflow.


Update scalafmt-core to 3.8.1 in scala-steward-dependencies

About this PR
📦 Updates org.scalameta:scalafmt-core from 3.7.17 to 3.8.1
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:n:3

merged with PR https://github.com/nationalarchives/tdr-draft-metadata-validator/pull/40


LibWeb: Styles inside @media are ignored if there is no query

I'm not sure if @media without actual query is valid but Chrome and Firefox seem to apply styles anyway. This reduction also comes from a real website iirc, so things like that exist in the wild.
Example:
<style>
    @media {
        body {
            background-color: crimson;
        }
    }
</style>

Chrome/Firefox
Ladybird

Thanks! Spec says that it is indeed valid, so this is a bug in our implementation:

An empty media query list evaluates to true.
For example, these are equivalent:
@media all { … }
@media { … }
https://www.w3.org/TR/mediaqueries-5/#mq-list


Show error in text boxes if fetch fails

This was a feature in #39 I forgot to port forward
Currently nothing is displayed if loading the sampling options fails

I guess this will conflict with #97. Should I modify #97 to use this? Or what should we do?

If you want to pull this into #97 that's good with me

If you want to pull this into #97 that's good with me

Done. Closing.


PageSpeed warning: CSS Value Appearance Non Standard

Great thanks to @davestewart for your efforts in bringing this back to life.
Good for Grav CMS default theme users.
According to PageSpeed insights on one of my Spectre using website, Google Chrome is planning to deprecate the "appearance" CSS property.
"Appearance" property is used inside Spectre css on buttons and forms as well as in Spectre Exp.
Not sure if this needs to be addressed.

Hello, and thanks for the appreciation 🙏
Totally happy to look at that.
I need to decide in what order to tackle things, but stand by and I'll take a look

I don't think there is anything to fix here. The appearance property is fully supported and there is no plan to deprecate it.
I unfortunately can't see what the original message was, but I would guess that it complained about the use of -webkit-appearance. That is technically a legacy property that should not be used, but it is still widely used to fix edge cases in many CSS frameworks, so I wouldn't be too worried about the use of it here.

Thank you for feedback.
I cannot see the same message anymore on PageSpeed.
-webkit-appearance was not mentioned in original message, otherwise I would have mentioned it.
I assume this issue can be closed.


[AST] Treat actors inheriting from NSObject as SwiftNativeNSObjects.

Previously, omitting @objc meant that ObjC-refcounting was used, which led to UAF
since the multi-step deinitialization was bypassed and the actor was eagerly deallocated.

I wasn't entirely sure if the best place to test this was at the SIL level, or LLVM IR, or if I should add a runtime test too to make sure the superclass is indeed SwiftNativeNSObject. Happy to add more tests.

@swift-ci smoke test

LGTM; this should have been kept in the condition when Doug introduced the @objc tweak.

LGTM; this should have been kept in the condition when Doug introduced the @objc tweak.

Seems like it should be an assertion that any default actor uses Swift reference counting


Get photo using POST method

Hi, I have an endpoint that is a POST route and I have to send the file name as a parameter to get the picture. I wasn't able to found any solutions in the documentation nor in the issues section.

you can override ImageProvider

you can override ImageProvider

@zmtzawqlp could you please give a sample code

@zmtzawqlp could you please give a sample code

copy all code
https://github.com/fluttercandies/extended_image_library/blob/e67006f75cdcdfabfe4890b9b9dd067cc17da48c/lib/src/_network_image_io.dart#L168
change this line into your code

copy all code
https://github.com/fluttercandies/extended_image_library/blob/e67006f75cdcdfabfe4890b9b9dd067cc17da48c/lib/src/_network_image_io.dart#L168
change this line into your code

Thank you

Thank you

I changed the part I needed to but I don't find a place to pass my custom provider as an argument.

I changed the part I needed to but I don't find a place to pass my custom provider as an argument.

Ok found It I have to pass the provider as an image to the ExtendedImage() constrictor.
child: ExtendedImage(image: CustomExtendedImage())

Ok found It I have to pass the provider as an image to the ExtendedImage() constrictor.
child: ExtendedImage(image: CustomExtendedImage())


multiply fractions with parenthesis

the multiplyFractions function doesn't handle cases like this where a fraction is in parenthesis:

(2 / x) * x -> 2x / x)
(2 / (4x)) * 4x -> (2 * 4x)/4x

this makes it harder for things to cancel, which means that we can't get #88 working
this would be very helpful to get done soon and I don't think it's too much work 😸

another possibility is just getting rid of those parenthesis in removeUnnecessaryParentheses because I don't think they're needed around a fraction

Fixed by https://github.com/socraticorg/mathsteps/commit/344d15b5d5e25251cef7d6976d6a73df16661d7d


[FEATURE] Support OpenSearch 1.x line

Is your feature request related to a problem?
The current implementation of the client only supports OpenSearch versions 2.x and higher. This is a blocker for many people who currently use the 1.x line of OpenSearch. We should refactor the code to allow for the client to work with older versions of OpenSearch.
What solution would you like?
Support for 1.x line of OpenSearch and integration tests to verify.
What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

Closing via #78


ForbiddenError cause by csurf

I installed the Vulnogram using Docker Compose, but a "Forbidden" page appears every time I log in. The code error is shown in the following figure. It should be caused by the CSRF verification. I debugged the code and found that the secret in csurf changes each request, resulting in the CSRF token not being able to match.

I tried to comment out the code of CSRF verification, but every time after logging in successfully, it would jump back to the login page, /user/login -> /cve -> /user/login.

Solved.
Default process.env.NODE_ENV is production, and cookie is secure on production, but httpsOptions is empty by default and it run as http, so cookie is broken.


No progress after USB device opened

Hi,
first of all thanks for sharing your project!!
As I am new to Arduino I was playing around with your app. The situation I'm ending up in is the following:
pi@pi:~/weather-station_0.1.0_linux_armv7 $ ll /dev/ttyUSB*
crw-rw---- 1 root dialout 188, 0 Aug 27 19:49 /dev/ttyUSB0
pi@pi:~/weather-station_0.1.0_linux_armv7 $ ./weather-station --device="/dev/ttyUSB0"
2018/08/27 20:11:53 Serving metrics at ':8080/metrics'
2018/08/27 20:11:53 Device '/dev/ttyUSB0' opened
So there is only one USB device and thats the Arduino, but I don't get the ready message and the app is stuck. I do not get any error or timeout (waited for some minutes). There aren't also any entries in /var/log/syslog or dmesg.
I tested the Arduino using Arduino IDE and the serial monitor, sending the homeduino commands by myself and I received 433MHz signals without any problems. So I'm quite sure the arduino part is fine.
Seems like my Arduino part is missing something but as there are no more dependencies in the Readme I'm quite out of ideas right now. Maybe you can give a hint?

You're using a Mac? I have the same problem, since you cannot set your tty device to 115200 baud via stty. Maybe using https://github.com/jacobsa/go-serial instead of just opening the device would help.

I do not understand what that has to do with a Mac problem. The program runs on the Raspberry Pi. In addition to Mac and Windows, I just tested whether the Arduino part works properly for itself, without weatherstation binary, only using the Arduno IDE. It worked on both platforms, so the Arduino works fine and the problem has to be on the Pi. I just do not know how to analyze further.

So on a mac, it is not possible to set the baudrate of the serial device. Maybe this is also an issue on the raspi. Arduino IDE (like screen) uses system calls in C to setup the serial connection baudrate. Normally, you would do this via stty but this is not working on OSX. Can you connect via screen and setting the baudrate?

After setting the tty to 115200 baud on a linux machine, it works somewhat:
~# stty -F /dev/ttyUSB0
speed 9600 baud; line = 0;
-brkint -imaxbel
~# stty -F /dev/ttyUSB0 115200
~# stty -F /dev/ttyUSB0
speed 115200 baud; line = 0;
-brkint -imaxbel

~# ./weather-station --device="/dev/ttyUSB0"
2018/09/01 17:21:45 Serving metrics at ':8080/metrics'
2018/09/01 17:21:45 Device '/dev/ttyUSB0' opened
2018/09/01 17:21:45 ready
2018/09/01 17:21:45 Wrote command 'RF receive 0' to '/dev/ttyUSB0' (13 bytes)
2018/09/01 17:21:45
2018/09/01 17:21:46 ready
2018/09/01 17:21:46
2018/09/01 17:21:46 ERR unknown_command

Any ideas?

Yeah, great!! That did the trick! Now I'm able to connect to Arduino from Pi. Unfortunately I get the same errors regarding unknown command:
pi@pi:~/weather-station_0.1.0_linux_armv7 $ ./weather-station --device="/dev/ttyUSB0"
2018/09/01 18:12:36 Serving metrics at ':8080/metrics'
2018/09/01 18:12:36 Device '/dev/ttyUSB0' opened
2018/09/01 18:12:36 ERR unknown_command
2018/09/01 18:12:36
2018/09/01 18:12:36 ready
2018/09/01 18:12:36 Wrote command 'RF receive 0' to '/dev/ttyUSB0' (13 bytes)
2018/09/01 18:12:36
2018/09/01 18:12:38 ready
2018/09/01 18:12:38
2018/09/01 18:12:38 ERR unknown_command
2018/09/01 18:12:38
2018/09/01 18:12:38 ERR unknown_command
2018/09/01 18:12:38
2018/09/01 18:12:38 ERR unknown_command
2018/09/01 18:12:38
2018/09/01 18:12:38 ERR unknown_command
2018/09/01 18:12:38

Hey guys, sorry for being late to this issue - I've been hiking for some days :)
There have been some changes in the main branch by @panzerdev, which didn't end up in the first release. These changes reset the Arduino into some expected state. I made a new v0.2 release.
Can you please check if that helps?

I get a slightly different error with v0.2.0
~# ./weather-station --device="/dev/ttyUSB0"
2018/09/02 00:07:12 Device '/dev/ttyUSB0' opened
2018/09/02 00:07:12 Wrote command 'RESET' to '/dev/ttyUSB0' (6 bytes)
2018/09/02 00:07:12 Line Scanned: ERR unknown_command
2018/09/02 00:07:12 Not ready msg: ERR unknown_command
2018/09/02 00:07:12 Line Scanned:
2018/09/02 00:07:12 Not ready msg:
2018/09/02 00:07:14 Line Scanned: ready
2018/09/02 00:07:14 Serving metrics at ':8080/metrics'
2018/09/02 00:07:14 Wrote command 'RF receive 0' to '/dev/ttyUSB0' (13 bytes)
2018/09/02 00:07:14 Write RF receive 0
2018/09/02 00:07:14 Line Scanned:
2018/09/02 00:07:14 Line Scanned: ERR unknown_command
2018/09/02 00:07:14 Line Scanned:
2018/09/02 00:07:14 Line Scanned: ACK
2018/09/02 00:07:14 Line Scanned:
2018/09/02 00:07:14 Line Scanned: ERR unknown_command
2018/09/02 00:07:14 Line Scanned:
2018/09/02 00:07:14 Line Scanned: ERR unknown_command

OK, resetting via screen solved the issue for me. Just the temperature is strange now:
2018/09/02 00:40:38 Line Scanned: RF receive 580 4028 1996 8924 0 0 0 0<PHONE_NUMBER>010102020201020202020201010202010101010102020202010202020201020203
2018/09/02 00:40:38 GT_WT_01: {ID:3554 Name:3554 Channel:1 Temperature:332 Humidity:66 LowBattery:false}
2018/09/02 00:40:39 Line Scanned: RF receive 576 4040 1996 8936 0 0 0 0<PHONE_NUMBER>010102020201020202020201010202010101010102020202010202020201020203
2018/09/02 00:40:39 GT_WT_01: {ID:3554 Name:3554 Channel:1 Temperature:332 Humidity:66 LowBattery:false}

Great. What do you mean by resetting via screen?
Which sensors are you using, the exact same GT_WT_01 models I have? I am
asking as there are two models which use different encodings.
On Sun, Sep 2, 2018, 00:43 Martin Hoefling<EMAIL_ADDRESS>wrote:

OK, resetting via screen solved the issue for me. Just the temperature is
strange now:
2018/09/02 00:40:38 Line Scanned: RF receive 580 4028 1996 8924 0 0 0 0<PHONE_NUMBER>010102020201020202020201010202010101010102020202010202020201020203
2018/09/02 00:40:38 GT_WT_01: {ID:3554 Name:3554 Channel:1 Temperature:332 Humidity:66 LowBattery:false}
2018/09/02 00:40:39 Line Scanned: RF receive 576 4040 1996 8936 0 0 0 0<PHONE_NUMBER>010102020201020202020201010202010101010102020202010202020201020203
2018/09/02 00:40:39 GT_WT_01: {ID:3554 Name:3554 Channel:1 Temperature:332 Humidity:66 LowBattery:false}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jckuester/weather-station/issues/12#issuecomment-417892403,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEHDsv9nSaj5bNA0MVLCY7axN_UP4RhMks5uWw2vgaJpZM4WOQSu
.

Using v0.2.0 I'm getting the same errors as @martinhoefling with this version (before the reset). It would be great if you could provide the command / commands used to reset via screen. Unfortunately I cant get a connection using screen, but it very likely I'm doing something wrong.

As far as I can remember, I just did connect via screen /dev/ttyUSB0 115200 and did a disconnect via CTRL+A K.
The sensor works with the non variant params as described in #13. The sensor is from Techvance.

So I again played around a bit and I'm in the same situation as @martinhoefling now. I could manage to connect to arduino via screen, but disconnecting using ctrl+a or any other command did not work. However in the end I just killed the screen process from another terminal. After that I started the weather station binary and got the same unknown command error slike before. But simply starting it again worked and it's receiving values.
Now it seems I can just start it, stop it and repeat this over and over and everything works. Not sure what excatly was the problem getting it to work.
I also get strange values for temperature and humidity (at least seems to be kind of in the same range, but its off by some values):
pi@pi:~/weather-station_0.2.0_linux_armv7 $ ./weather-station 1888=entrance
2018/09/03 21:34:57 Device '/dev/ttyUSB0' opened
2018/09/03 21:34:57 Wrote command 'RESET' to '/dev/ttyUSB0' (6 bytes)
2018/09/03 21:34:57 Line Scanned: ready
2018/09/03 21:34:57 Serving metrics at ':8080/metrics'
2018/09/03 21:34:57 Wrote command 'RF receive 0' to '/dev/ttyUSB0' (13 bytes)
2018/09/03 21:34:57 Write RF receive 0
2018/09/03 21:34:57 Line Scanned: ACK
2018/09/03 21:35:10 Line Scanned: RF receive 468 692<PHONE_NUMBER>4 0 0 0<PHONE_NUMBER>000000000000000012000000121212300030303030303012123012303031121231121231311224
2018/09/03 21:35:10 Device Unknown
2018/09/03 21:35:11 Line Scanned: RF receive 340<PHONE_NUMBER>2 0 0 0 0<PHONE_NUMBER>000000010000010112010101121212010101010101010112120112010121121221121221011223
2018/09/03 21:35:11 Device Unknown
2018/09/03 21:35:17 Line Scanned: RF receive 604 2032 4084 8988 0 0 0 0<PHONE_NUMBER>020201010101010101010102020201020202020102020101010202020101020103
2018/09/03 21:35:17 GT_WT_01: {ID:1888 Name:1888 Channel:1 Temperature:383 Humidity:57 LowBattery:false}
2018/09/03 21:35:18 Line Scanned: RF receive 600 2032 4080 8992 0 0 0 0<PHONE_NUMBER>020201010101010101010102020201020202020102020101010202020101020103
2018/09/03 21:35:18 GT_WT_01: {ID:1888 Name:1888 Channel:1 Temperature:383 Humidity:57 LowBattery:false}
2018/09/03 21:35:41 Line Scanned: RF receive 476 704<PHONE_NUMBER>6 0 0 0<PHONE_NUMBER>000000000000000012000000121212300030303030303012123012303030121230121230211224
2018/09/03 21:35:41 Device Unknown
2018/09/03 21:35:42 Line Scanned: RF receive 308<PHONE_NUMBER>4 0 0 0 0<PHONE_NUMBER>010000010101010112010101121212010101010101010112120112010101121201121201011203
2018/09/03 21:35:42 Device Unknown
2018/09/03 21:36:07 Line Scanned: RF receive 604 2028 4084 8996 0 0 0 0<PHONE_NUMBER>020201010101010101010102020201020202020102020101010202020101020103
2018/09/03 21:36:07 GT_WT_01: {ID:1888 Name:1888 Channel:1 Temperature:383 Humidity:57 LowBattery:false}
2018/09/03 21:36:08 Line Scanned: RF receive 604 2028 4076 8972 0 0 0 0<PHONE_NUMBER>020201010101010101010102020201020202020102020101010202020101020103
2018/09/03 21:36:08 GT_WT_01: {ID:1888 Name:1888 Channel:1 Temperature:383 Humidity:57 LowBattery:false}
2018/09/03 21:36:12 Line Scanned: RF receive 432 664<PHONE_NUMBER>4 0 0 0<PHONE_NUMBER>000000000000000012000000121212300000313131313112123112312121121221121221211224
2018/09/03 21:36:12 Device Unknown
2018/09/03 21:36:13 Line Scanned: RF receive 448 716<PHONE_NUMBER>4 0 0 0<PHONE_NUMBER>000000000000000012000000121212003000003000000012123012003000121230121230301234
2018/09/03 21:36:13 Device Unknown
2018/09/03 21:36:43 Line Scanned: RF receive 412 676<PHONE_NUMBER>6 0 0 0<PHONE_NUMBER>000000000000000012000000121212000000000000000112120112010131121231121231311234
2018/09/03 21:36:43 Device Unknown
2018/09/03 21:36:44 Line Scanned: RF receive 316<PHONE_NUMBER>4 0 0 0 0<PHONE_NUMBER>000000010101010112010101121212010101010101010112120112010101121201121201011203
2018/09/03 21:36:44 Device Unknown

I also get a lot of unkown device messages, most likley it's stuff from my neighbours I think??
@martinhoefling Seems we have bought the same sensor. Would be great to have this pull request!

Maybe another hint: When disconnecting the Arduino and reconnecting USB cable I have to

connect via screen
kill the screen process
start weather station binary (getting ERR unknown command)
start weather station binary again

This is can be reproduced.

@christiangroth , you also have a 'non variant' sensor, see #13. I am waiting for a response if my suggestion is an acceptable solution and can provide a pull request then. You can also just edit the protocol and adjust the values to weather12.coffee

Hi to all.... I had the same problems often and decided to reduce the homeduino software to make it only receive all the time, basically only use RFControl. At least that is the only thing I am doing with mine right now.

Hi @panzerdev , what exactly do you mean by "reduce the homeduino software"? Did you change the binary flashes to arduino? In case this project does onle use the RESET and RF receive commands as fas as I can see.

@christiangroth basically yes. I reduced the Arduino firmware until I was just left with always receiving and writing to serial.
https://github.com/panzerdev/433mhz-receiver/blob/master/arduino/homeduino.ino
It's not really homeduino anymore.
I also changed the go receiver to be multiple purpose because I have more 433 transmitter and the one is from the doorbell for example.

@panzerdev ah okay I see, thanks for sharing. Can be a solution, but I think I'll first with the fix and PR by @martinhoefling. So I'll close this issue now as there are no open problems left.


[Question]: in-memory testing database experience?

I'm testing an in-memory test environment. This is my config/database.yml:
test:
  adapter: sqlite3
  encoding: utf8
  pool: 5
  timeout: 10000
  #database: db/test.sqlite3
  database: ":memory:"

schema.rb is loaded when running rspec, but the test fails and complains: "ActiveRecord::StatementInvalid: Could not find table 'authorizations'"
If the test DB is not in memory but as a regular file, then it works fine. Is this an issue or just a particular environment wrong configuration?.

Considering that Omniauth does not explicitly talk to a database, I'm going to have to lean towards a configuration issue, or otherwise.
It looks like migrations haven't been run in your test environment.


feat(backend): request grant to query incoming payment receiver

Changes proposed in this pull request

The receiver service now uses a stored grant if one exists or requests a new grant when querying a remote incoming payment.

Context

fixes #583

Checklist

[x] Related issues linked using fixes #number
[x] Tests added/updated
[ ] Documentation added
[x] Make sure that all checks pass

Using a combination of this branch + https://github.com/interledger/rafiki/pull/777, I was able to run localenv with AS signature verification enabled:
https://github.com/interledger/rafiki/blob/632bdfb15a920062637dc8a273034e58b382205b/infrastructure/local/peer-docker-compose.yml#L17
and successfully create a quote (with backend auto-requesting a grant to read the receiver incoming payment)

Using a combination of this branch + #777, I was able to run localenv with AS signature verification enabled:
https://github.com/interledger/rafiki/blob/632bdfb15a920062637dc8a273034e58b382205b/infrastructure/local/peer-docker-compose.yml#L17
and successfully create a quote (with backend auto-requesting a grant to read the receiver incoming payment)

what sig headers did you use for making it work?

This query :point_up_2:
The signature headers are unchanged (TODO)
But the signature headers are there for the RS->AS grant request under the hood.

Is accessToken just dev-access-token for now? I'm working on updating those seed scripts.

Yeah, I changed it from the (Postman) default example-access-token.


Add example of using icons in a segment

We support using icons in a segment
https://github.com/driftyco/ionic/blob/2.0/src/components/segment/test/basic/main.html#L92 but we dont show an example of this.

Add this to API docs


Selecting an Element with Arrow-Keys shows only value not label

Hello,
if i selecting an Element in the Autocomplete List by typing the first Letter and use then the Arrow-Keys.
It will only use/show the value from the token.
I think if "label" is set, it would be better to show the label?!
Greetings

Not sure if this is related, but if I select the value from the autocomplete list using the arrow keys, it fills in both the value and the label, which probably isn't ideal behavior, to say the least.

I got the same problem too, which is not what I expected.

Same problem. It would make a lot of sense to show the label if it is available.

Duplicate of https://github.com/sliptree/bootstrap-tokenfield/issues/76
But +1, I am having the same problem, it seems that the normal function of the autocomplete is to select the value on arrow-keys, there must be a way to override it.
@chinobing for duplicate tokens try
$('#tokenfield').on('tokenfield:createtoken', function (event) {
    var existingTokens = $(this).tokenfield('getTokens');
    $.each(existingTokens, function(index, token) {
        if (token.value === event.attrs.value)
            event.preventDefault();
    });
});

From https://github.com/sliptree/bootstrap-tokenfield/issues/308#issuecomment-318564936


Added ticket system and ticket logging

Users can create a ticket with /ticket
This creates a Priv. Thread, and adds staff.
Logger uses Audit log to detect updates to the threads.
Upon resolution, staff should close/lock the thread.

Forgot to ping user on ticket creation


Add function to change bar colour

Implements #13

Tested in FF and Chrome and it's working fine.


GitHub Reverse Integration 17643059

GitHub reverse integration Pull Request for commit
Note: created due to build 'Omex GitHub Official' with id:23.4.10109.11451

@neilr81 did you create this RI?
If so why?

@neilr81 did you create this RI? If so why?

This is automatic.


Use stable sort when sorting tokens

For context - https://github.com/rubocop-hq/rubocop/pull/8729

Ahhhh!
Great investigation! 👏

Released 0.4.2


[BUG] High VRAM Usage For Inference, Torch Dtype Doesn't Matter

Describe the bug
My understanding of model parallelism is that the model is split over multiple GPUs to lower memory usage per GPU allowing larger models and speeding up inference.  Thus for GPT Neo 2.7B and two 3090s I would expect the VRAM usage to be roughly 5.5GB per GPU or 11GB in total if placed on one GPU.
The issue is that the VRAM usage is roughly 11GB on each GPU.  Additionally, when selecting torch.half for the dtype, the VRAM usage stays high and does not change. Due to this problem, I also OOM for GPTJ on one or both GPUs
To Reproduce
Steps to reproduce the behavior:

Install DeepSpeed from the source
Install Transformers from pip
Follow inference tutorial
Watch VRAM usage or experience OOM for larger models

Expected behavior
I would expect VRAM usage to decrease as you use multiple GPUs.  I would also expect the VRAM usage to decrease when using lower precision data types.
ds_report output
ds_report
DeepSpeed C++/CUDA extension op report
NOTE: Ops not installed will be just-in-time (JIT) compiled at
runtime if needed. Op compatibility means that your system
meet the required dependencies to JIT install the op.
JIT compiled ops requires ninja
ninja .................. [OKAY]
op name ................ installed .. compatible
cpu_adam ............... [YES] ...... [OKAY]
cpu_adagrad ............ [YES] ...... [OKAY]
fused_adam ............. [YES] ...... [OKAY]
fused_lamb ............. [YES] ...... [OKAY]
sparse_attn ............ [YES] ...... [OKAY]
transformer ............ [YES] ...... [OKAY]
stochastic_transformer . [YES] ...... [OKAY]
async_io ............... [YES] ...... [OKAY]
utils .................. [YES] ...... [OKAY]
quantizer .............. [YES] ...... [OKAY]
transformer_inference .. [YES] ...... [OKAY]
DeepSpeed general environment info:
torch install path ............... ['/root/anaconda3/envs/gpt/lib/python3.9/site-packages/torch']
torch version .................... 1.12.0
torch cuda version ............... 11.3
torch hip version ................ None
nvcc version ..................... 11.3
deepspeed install path ........... ['/root/anaconda3/envs/gpt/lib/python3.9/site-packages/deepspeed']
deepspeed info ................... 0.7.1+7d8ad45, 7d8ad45, master
deepspeed wheel compiled w. ...... torch 1.12, cuda 11.3
System info (please complete the following information):

OS: Ubuntu 20.04
GPU count and types: 2 3090s
Interconnects: 1 system, 2 3090s
Python version: 3.9.13

I am using a docker container with Nvidia Cuda already set up as the base image.
Launcher context
deepspeed --num_gpus 2 infer.py
deepspeed --num_gpus 1 infer.py
both lead to the same VRAM usage per GPU
Docker context
Are you using a specific docker image that you can share?
nvidia/cuda:11.3.1-devel-ubuntu20.04
then I am building python packages into the container
Additional context
When setting up the pipeline like the tutorial, if I load the model first with the data type I want, and then pass the model and tokenizer to the pipeline, it seems to lower the VRAM usage on one of the GPUs.  Doing this and running with only one GPU rather than two had lead me to be able to use GPTJ, but I want to use two GPUs for inference
#this works for one GPU
model_name = 'EleutherAI/gpt-j-6B'
model = AutoModelForCausalLM.from_pretrained(model_name,torch_dtype=torch.float16)
tokenizer = AutoTokenizer.from_pretrained(model_name)
generator = pipeline('text-generation', model=model,tokenizer=tokenizer, device=local_rank)

generator.model = deepspeed.init_inference(generator.model,
                                           mp_size=world_size,
                                           dtype=torch.half,
                                           replace_method='auto',
					   replace_with_kernel_inject=True)

#this leads to OOM for 2 3090s, with one being normal and one not
model_name = 'EleutherAI/gpt-j-6B'
model = AutoModelForCausalLM.from_pretrained(model_name)
tokenizer = AutoTokenizer.from_pretrained(model_name)
generator = pipeline('text-generation', model=model,tokenizer=tokenizer, device=local_rank)

generator.model = deepspeed.init_inference(generator.model,
                                           mp_size=world_size,
                                           dtype=torch.half,
                                           replace_method='auto',
					   replace_with_kernel_inject=True)

#this also leads to OOM for 2 3090s
model_name = 'EleutherAI/gpt-j-6B'
generator = pipeline('text-generation', model=model_name, device=local_rank)

generator.model = deepspeed.init_inference(generator.model,
                                           mp_size=world_size,
                                           dtype=torch.half,
                                           replace_method='auto',
					   replace_with_kernel_inject=True)

By changing the pipeline to the following I now get VRAM usage of roughly 12GB per GPU.  However, shouldn't the model be split over both GPUs and thus roughly 6GB each?  Perhaps I am misunderstanding model parallelism.  In either case, an issue similar to #2113 still occurs for GPTJ but not GPT Neo.  I get junk output using fp16 on two 3090s
generator = pipeline('text-generation', model=model_name, device=local_rank,torch_dtype=torch.float16)

GPTJ output:
[{'generated_text': 'DeepSpeed is,: to,,/ &.. by and.. a\n.. and- and.. the,,\n of\n [.,.\n:, &-. and a- the,\n\n). the'}]

GPT Neo 2.7B output:
[{'generated_text': 'DeepSpeed is a speedup technique for Java programs.\nSee  http://www.cs.indiana.edu/~mh/classes/java/cs405/labs/gong/\nHere are some examples.\n\n'}]

Hi @mallorbc ,
The problem is that the model selected from HF is Fp32 and it will load the checkpoint before coming to the model-partitioning on the DeepSeed-Inference side. For reducing the memory-usage on GPU, you can remode the device at the pipeline creation and pass the model to deepspeed.init_inference to do the model-partitioning, and set the device for the pipeline after the deepspeed.init_inference function call. Does this make sense?
Thanks,
Reza

By changing the pipeline to the following I now get VRAM usage of roughly 12GB per GPU. However, shouldn't the model be split over both GPUs and thus roughly 6GB each? Perhaps I am misunderstanding model parallelism. In either case, an issue similar to #2113 still occurs for GPTJ but not GPT Neo. I get junk output using fp16 on two 3090s
generator = pipeline('text-generation', model=model_name, device=local_rank,torch_dtype=torch.float16)

GPTJ output:
[{'generated_text': 'DeepSpeed is,: to,,/ &.. by and.. a\n.. and- and.. the,,\n of\n [.,.\n:, &-. and a- the,\n\n). the'}]

GPT Neo 2.7B output:
[{'generated_text': 'DeepSpeed is a speedup technique for Java programs.\nSee  http://www.cs.indiana.edu/~mh/classes/java/cs405/labs/gong/\nHere are some examples.\n\n'}]

This issue is related to another issue that you opened regarding the MP issue for the GPTJ. Please let me know if this PR solves the issue.
Thanks,
Reza

Hi @mallorbc ,
The problem is that the model selected from HF is Fp32 and it will load the checkpoint before coming to the model-partitioning on the DeepSeed-Inference side. For reducing the memory-usage on GPU, you can remode the device at the pipeline creation and pass the model to deepspeed.init_inference to do the model-partitioning, and set the device for the pipeline after the deepspeed.init_inference function call. Does this make sense? Thanks, Reza

@RezaYazdaniAminabadi ,
I did figure out how to reduce the memory usage from full fp32 to using the memory expected for fp16, but on EACH gpu.  So for GPTJ, roughly 12GB of memory is used on each GPU.  Shouldn't it be 12GB in total?  Roughly 6GB on each GPU?
Thanks your help.

Hi @mallorbc,
We have added a test-suite here that measure the memory consumption after init_inference and also the pipeline creation. Can you please try it to see if the memory is as expected?
You can follow the instructions here to call the inference_test.
Thanks,
Reza

Hi @mallorbc,
We have added a test-suite here that measure the memory consumption after init_inference and also the pipeline creation. Can you please try it to see if the memory is as expected? You can follow the instructions here to call the inference_test. Thanks, Reza

I will try this.  Thanks!


Android Compose Change the App Icon

Name of the Codelab or Codelab URL
https://developer.android.com/codelabs/basic-android-kotlin-compose-training-change-app-icon#4
Describe the problem
This is a minor typo problem. The page in the URL above notes we'll be creating an adaptive icon for the Tip Calculator app. I believe it should say Affirmations app instead of Tip Time app.
In which lesson and step of the codelab can this issue be found?
Unit 3, pathway 2, step 5
How to reproduce?
N/A
Versions

What version of Android Studio are you using? Chipmunk 2021.2.1 Patch 2
What API level are you targeting? N/A

Additional information
Add any other context about the problem here.
codelab: basic-android-compose-training-add-scrollable-list

YES,,, the downloaded images belong to the tip calcaulator... we can use these but they are not the ones displayed in the codelab for Affirmations
https://developer.android.com/codelabs/basic-android-kotlin-compose-training-change-app-icon?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-3-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-training-change-app-icon#5

Hello @williamFerrante  ,
Thank you for reaching out to us and reporting the issue. The app name and the assets were updated:


A question regarding Miri error running the “sanitize" example under crossbeam-epoch.

Hello, when I ran the sanitize example under crossbeam-epoch using Miri, I got an undefined behaviour error. As I am new to crossbeam, could you please offer some insight as to what could be the cause of the error? Thank you.
The command I used,
$ MIRIFLAGS="-Zmiri-symbolic-alignment-check -Zmiri-disable-isolation -Zmiri-tree-borrows" cargo miri run -p crossbeam-epoch --example sanitize

and the error,
error: Undefined Behavior: Data race detected between (1) atomic load on thread `unnamed-4` and (2) retag read of type `std::sync::atomic::AtomicUsize` on thread `unnamed-5` at alloc1563563. (2) just happened here
    --> /Users/lei.shi/.rustup/toolchains/nightly-x86_64-apple-darwin/lib/rustlib/src/rust/library/alloc/src/boxed.rs:1008:9
     |
1008 |         Box(unsafe { Unique::new_unchecked(raw) }, alloc)
     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Data race detected between (1) atomic load on thread `unnamed-4` and (2) retag read of type `std::sync::atomic::AtomicUsize` on thread `unnamed-5` at alloc1563563. (2) just happened here
     |
help: and (1) occurred earlier here
    --> crossbeam-epoch/examples/sanitize.rs:30:21
     |
30   |                     p.deref().load(Relaxed)
     |                     ^^^^^^^^^^^^^^^^^^^^^^^
     = help: overlapping atomic and non-atomic accesses must be synchronized, even if both are read-only
     = help: see https://doc.rust-lang.org/nightly/std/sync/atomic/index.html#memory-model-for-atomic-accesses for more information about the Rust memory model
     = help: retags occur on all (re)borrows and as well as when references are copied or moved
     = help: retags permit optimizations that insert speculative reads or writes
     = help: therefore from the perspective of data races, a retag has the same implications as a read or write
     = help: this indicates a bug in the program: it performed an invalid operation, and caused Undefined Behavior
     = help: see https://doc.rust-lang.org/nightly/reference/behavior-considered-undefined.html for further information

Environment,

x86_64 apple mac laptop.
rust version: 1.79.0; toolchain: nightly-x86_64-apple-darwin
crossbeam-epoch v0.9.18

I cannot reproduce this with the recent nightly. I believe this is the old Miri issue resolved in https://github.com/rust-lang/rust/pull/128778.


pytube is in python2 in Mac

why it so when I run the program why it is showing that it is in python2 library as it is outdated why is this happening

Maybe you installed pytube via pip instad of pip3?

Is it necessary to work on the environment while doing some work on python

I don't quite understand what you mean, but it should suffice if you install pytube via pip. Python2 is not supported anymore so it shouldn't be used. I can help you if you include the output of pip --version and pip freeze.


[wiki] Basic Authentication possible fix

Version Number and Operating System(s):
master; Windows
Expected behavior:
Get the correct data set from client.
Actual behavior:
Exception in response.GetByteArray()
Steps to reproduce:

create a client authenticator to send the data to server; set it before Connect()

public void AcceptChallenge(NetWorker networker, BMSByte challenge, Action<BMSByte> authServerAction, Action rejectServerAction)
{
	var data = ObjectMapper.BMSByte(0, Encoding.UTF8.GetBytes($"test:test"));

	authServerAction(data);
}

create a server authenticator to read the data

public void VerifyResponse(NetWorker networker, NetworkingPlayer player, BMSByte response, Action<NetworkingPlayer> authUserAction, Action<NetworkingPlayer> rejectUserAction)
{
	ulong steamid = response.GetBasicType<ulong>();
	byte[] ticketBinary = response.GetByteArray(response.StartIndex()); // here exception

	if (AuthUser(steamid, ticketBinary))
	{
		authUserAction(player);
	}
	else
	{
		rejectUserAction(player);
	}
}

Possible fix of VerifyResponse:
if (response == null || response.byteArr == null ||
	response.byteArr.Length == 0 || response.Size == 0)
{
	rejectUserAction(player);
	return;
}

ulong steamid = response.GetBasicType<ulong>();

byte[] ticketBinary = new byte[response.Size];
Buffer.BlockCopy(response.byteArr, response.StartIndex(), ticketBinary, 0, ticketBinary.Length);

First thing I noticed is you map a 0 then in verify response you try and get out a ulong. 0 != 0ul when you are mapping it out as the first would map to 4 bytes while a ulong would map to 8 bytes.
So your example code should be written like this:
public void AcceptChallenge(
    NetWorker networker,
    BMSByte challenge,
    Action<BMSByte> authServerAction,
    Action rejectServerAction
) {
    ulong steamId = yourMethodThatGetsThis();
    var data = ObjectMapper.BMSByte(steamId, Encoding.UTF8.GetBytes($"test:test"));

    authServerAction(data);
}

Then on the server:
public void VerifyResponse(
    NetWorker networker,
    NetworkingPlayer player,
    BMSByte response,
    Action<NetworkingPlayer> authUserAction,
    Action<NetworkingPlayer> rejectUserAction
) {
	ulong steamid = response.GetBasicType<ulong>();  // <--- Moves internal array pointer.
	byte[] ticketBinary = response.GetBasicType<byte[]>();

	if (AuthUser(steamid, ticketBinary))
	{
		authUserAction(player);
	}
	else
	{
		rejectUserAction(player);
	}
}

You are correct but:
If I change as you mention, I now get an OutOfMemoryException: response.GetBasicType<byte[]>() => at x = GetBasicType(StartIndex(), true); // x gets a huge value
If I remove the steamid completely (since I do not use it), the data sent is not correctly retrieved (in my case it is an encrypted text and decryption does not recognize it).
So, there is an issue of 4 bytes somewhere...

That is weird. Haven't seen that before and I have used byte[]s multiple times.
Just an fyi you should not touch byteArr as that is the internal array that might not be what you expect it to be. You can get out the correct byte[] by using CompressBytes()

So, if I do  this on the client side:
var data = ObjectMapper.BMSByte(byte[]);
authServerAction(data);

and
byte[] ticketBinary = response.CompressBytes();
on server side, the data is not correctly retrieved...

Yes you are seeing the meta data in front. The first 4 bytes is the number of elements as an int in the array. Then you have the 9 elements.
Please use byte[] ticketBinary = response.GetBasicType<byte[]>();

Mapping the string directly worked. Thanks! :)


Is it possible to just generate the static html file?

I see that a temp file is created, and everything is rendered completely with html and no javascript, is there a flag to just output that file instead of using a tmp and deleting it?
This would be pretty nice, if i were to just look for generating static websites.

Sorry, Chris Clark, no such flag. But feel free to fork and add one.  Cheers.
Sent from my iPhone

On Feb 13, 2021, at 11:21 PM, Chris Clark<EMAIL_ADDRESS>wrote:

I see that a temp file is created, and everything is rendered completely with html and no javascript, is there a flag to just output that file instead of using a tmp and deleting it?
This would be pretty nice, if i were to just look for generating static websites.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Texecom Premier Panel Component

Description:
This is a component that allows home assistant to communicate with Texecom Premier alarm panels. it receives data via the serial port and decodes it into the respective zones. Only zones are supported as binary sensors.
The user can configure the required number of zones, names and types.
Sorry if there are any mistakes or errors this is my first component. It is working well at home
Example Config
texecom:
    port: /dev/ttyV0
    uuid: 1234
    zones:
      1:
       name: 'Front Room'
       type: 'motion'
       panelzone: '1'
      2:
       name: 'Dining Room'
       type: 'motion'
       panelzone: '5'
      3:
       name: 'Kitchen'
       type: 'motion'
       panelzone: '20'

Hi @JDW2018,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


test KO just after creation

I just installed the API and added 1 entity,
After following the steps in the video, I cant run any thing
I tried to do npm test
Test Suites: 6 failed, 6 total
Tests:       83 failed, 83 total
Snapshots:   0 total
Time:        28.513s
Ran all test suites.

can someone help me please?

I just installed the API and added 1 entity,
After following the steps in the video, I cant run any thing
I tried to do npm test
Test Suites: 6 failed, 6 total
Tests:       83 failed, 83 total
Snapshots:   0 total
Time:        28.513s
Ran all test suites.

can someone help me please?


Dart analyzer stalls when editing inside html script tag

Summary:
There is an issue with the dart-analyzer becoming unresponsive when editing dart-code in an html-document using WebStorm.
After the analyzer has run a few times it will take an increasing amount of time to analyze even simple functions, untill eventually it will stop responding and the 'Dart analysis' progress-spinner continues (seemingly) forever.
What steps will reproduce the problem?

Open Webstorm and create an Uber Simple Web Application
Open index.html and add the following to the body

<script type="application/dart">
main() {
  var foo = 42;
}
</script>

Open the Dart-Analysis window-pane to observe any issues that the analyzer is reporting
Repeatedly make a few changes to the program (e.g. remove the line var foo = 42;) and type it again), while verifying that the 'Dart-analysis' continues to report on issues it finds within program

Expected:
The 'Dart-analysis' window continues to update whenever the program is changed, i.e. it displays error-messages and then removes them when the issue is resolved.
Actual:
The dart-analyzer takes an increasingly long time to analyze the program, until it becomes completely unresponsive.
What version of the product are you using?
Webstorm: 2016.1.1
Dart-SDK: 1.16.0-dev.5.0
Additional info
This issue also occurs with dart-1.15-stable

I can confirm the bug. In case of Dart in HTML editing Analysis Server sends
1460626511817:Noti:{"event"::"server.status","params"::{"analysis"::{"isAnalyzing"::true}}}

and never sends {"isAnalyzing"::false} afterwards, that's why IDE shows that analysis is in progress.
Also server stops sending any information to the IDE after some time.

I assume that this is obsolete because we no longer analyze Dart code in HTML files.


failed to load example assets/test_file.omf

Trying to run this example to load assets/test_file.omf but failed:
https://mybinder.org/v2/gh/OpenGeoVis/omfvista/master?filepath=Example.ipynb
Error messages:
Traceback (most recent call last):
  File "Example.py", line 36, in <module>
    proj = omfvista.load_project('assets/test_file.omf')
  File "E:\Python37\lib\site-packages\omfvista\wrapper.py", line 124, in load_project
    return project_to_vtk(project)
  File "E:\Python37\lib\site-packages\omfvista\wrapper.py", line 115, in project_to_vtk
    d = omfvista.wrap(e)
  File "E:\Python37\lib\site-packages\omfvista\wrapper.py", line 103, in wrap
    return wrappers[key](data)
  File "E:\Python37\lib\site-packages\omfvista\pointset.py", line 29, in point_s
et_to_vtk
    points = pse.geometry.vertices
AttributeError: 'PointSetElement' object has no attribute 'geometry'

Hi @rminhxm - this issue looks like it might be on the omfvista-side rather than the OMF backend (this repository). Could you share a few more details to help me understand what's going wrong in this example?
Where exactly did you download the 'test_file.omf' file?
What is your version of omfvista, omf, and properties?
import omfvista, omf, properties
print('omfvista: ', omfvista.__version__)
print('omf: ', omf.__version__)
print('properties: ', properties.__file__)

Hi @banesullivan , thanks for looking into this. Here with more info:

The test_file.omf is from this repository.
The output of the versions

omfvista:  0.2.0
omf:  1.0.1
properties:  0.6.0b0

I followed the README guide and install all dependencies with pip.

Thank you for sharing those details - I have reproduced this issue locally.
This appears to be the result of a change that is present on the master branch of OMF (this repo) which I haven't corrected in omfvista. I'll look in the commit history and see what I can do to update omfvista to reflect most recent changes to OMF

To generalize this issue (adopted from the docs), the following example is failing when using the master branch of omf:
>>> import numpy as np
>>> import omf
>>>
>>> pts = omf.PointSetElement(
...     name='Random Points',
...     description='Just random points',
...     geometry=omf.PointSetGeometry(
...         vertices=np.random.rand(100, 3)
...     ),
...     data=[
...         omf.ScalarData(
...             name='rand data',
...             array=np.random.rand(100),
...             location='vertices'
...         ),
...         omf.ScalarData(
...             name='More rand data',
...             array=np.random.rand(100),
...             location='vertices'
...         )
...     ],
... )
Traceback (most recent call last):
  File "<stdin>", line 4, in <module>
AttributeError: module 'omf' has no attribute 'PointSetGeometry'

Immediate solution... revert to the stable version deployed on PyPI: pip install omf==1.0.1


setting num threads to 1 for lightGBM (to test if perf improves)

setting num threads to 1 for lightGBM (to test if perf improves)

@imatiach-msft i just tested this version and:

when I do coalesce(1) it works as expected (so it means really slow) - we use singe thread for training - the CPU usage on every machine is very low
when I use the default number of partitions (which is the number of cores available in the cluster) I see again the problem with the out of sync iterations:

18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3894 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3745 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3920 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3977 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3783 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3868 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3891 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3939 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3921 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3916 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3777 with result: 0 and is finished: 0
18/06/25 06:56:03 INFO LightGBMRegressor: LightGBM running iteration: 3866 with result: 0 and is finished: 0

Looks like we either can use num_machines (for parallel training with multiple executors, where lightgbm uses all the CPU resources) or num_threads (for single machine and than you can tune how many threads you want to use).
Unless you have some other ideas - I think this is dead end I will move back to the previous num_machines only version and try to figure out the difference between the R and Spark.

thanks for testing it out @troszok, I will close both PRs #327 and #326 now since you are seeing worse performance and I am not seeing any difference.
I did go through LightGBM source and I see it being read and used:
https://github.com/Microsoft/LightGBM/blob/dc6995742a5284a1e942978e2542fc49adda9ea1/src/c_api.cpp#L126
but it is used separately from the num_machines variable:
https://github.com/Microsoft/LightGBM/blob/dc6995742a5284a1e942978e2542fc49adda9ea1/src/network/network.cpp#L27
Which is passed in network init
FYI @ywskycn @mhamilton723


Elementum doesn't found Grey's Anatomy serie

I've tried to search for grey's anatomy show, but it doesn't return any results. After looking closely to log and try to repeat the searches manually in trackers sites, I've found the error:
Most trackers use greys anatomy torrent name (without the '). Could you correct this parser to not use the " ' " char, please?

There is no perfect solution.
Some providers understand quotes and does not find result without it, some don't.

I've tried to add this to list of 'exceptions' in filtering.py:
title = title.replace("grey's anatomy", 'greys anatomy')
but didn't work. Could you help me to use it?

I've changed this in normalize.py, then it worked better for me:
control_chars = ''.join(map(unichr, range(0, 32) + range(33,47) + range(127, 160)))

I have added option to Burst 0.0.34 to remove quote from queries.


Updates to the "Presentation request declined" screen

Description of feature / user story
Updates to the "Presentation request declined" screen as the current one does not conform to style guide and is not streamlined.
Acceptance Criteria
Given the holder declines the proof request
When the verifier receives acknowledgement that the request was declined
Then they should be presented with a clear UI that says the request was declined
And they should be presented with a button to attempt a new proof
Wireframes or relevant image assets / links
https://xd.adobe.com/view/0a112c19-6f28-48cd-8ba6-3d80c7cd3771-5f42/screen/c33df5a4-95f5-4907-83f7-2a78f65c953a/specs/

backlog review - only minor word changes. Input to change the wording to "Proof request declined, information not shared" as a caption

Waiting on publish of latest bc-wallet with latest bifold

@bryce-mcmath Does this need to be tested with the verifier? I didn't see the change in declining a proof with the showcase with v 1.0.13 1395.

Yes, this is for what the verifier sees when another user declines their proof request

@jleach ^

Verified in 1.0.15-1452