create_and_save not found

In 0.4.2 I don't have the create_and_save method
2.2.1 :009 > Htmltoword::Document.methods.select { |m| m=~ /create/ }
=> [:create, :create_with_content]
Do I need to track master? The document implies this is available in 0.4+

As per #18 it looks like the version on RubyGems is not up to date so you will need to track master for this.

Hey! latest version 0.4.4 does have it included :blush:


Fix linkouts for new sources.

Linkouts for new sources isn't working due to incorrect base urls in the importers.

CKB correct link (uses Entrez ID instead of Gene name): https://ckb.jax.org/gene/show?geneId=
ex. https://ckb.jax.org/gene/show?geneId=2260
OncoKB: http://oncokb.org/#/gene/
CGI: same url for all: https://www.cancergenomeinterpreter.org/biomarkers
FDA: same url for all: https://www.fda.gov/drugs/scienceresearch/researchareas/pharmacogenetics/ucm083378.htm
NCI Cancer gene index: same url for all https://wiki.nci.nih.gov/display/cageneindex/The+Cancer+Gene+Index+Gene-Disease+and+Gene-Compound+XML+Documents


Create workspace

Completes -SPARK-559074
My suggestion on how to review this PR - close the packages folder, it only has file name changes. After that its a small PR
This PR is the second PR in the series of many PRs to create react-widgets a yarn workspace
First PR https://github.com/webex/react-widgets/pull/1438
This PR completes the final 4tasks in the list below
Tasks required to make react-widgets a yarn workspace

[x] Move components out of node_modules
[x] Adjust the tooling to use the new paths
[x] Enable yarn workspace in react-widget
[x] Add dependencies inside each package
[x] Add scripts for each package
[x] Fix tooling to use yarn workspaces packages

Checks

[x] Build is passing
[x] UTs are passing
[ ] Integration tests are passing

Testing

Ran yarn build:all
Ran yarn workspace @webex/ build:src
Ran yarn workspace @webex/ serve
Served all the demos
Ran all UTs

Issues found
All these errors are persistent in master and node-upgrade branch also

People list has ENOENT
 error while building -  any css file thats why this error - its doesnt affect anything
Widget-meet has ENOENT -  any css file thats why this error - its doesnt affect anything
All demo components fails the buildES function.
Issue with tsconfig- webpack build fails for all ts 'widget' components
Failling command
- npm run deploy is failing in master and node-upgrade as well
We are ignoring all the src files for eslint tests
journey tests are failling in the node-upgrade branch as well

The reason why github actions is failling is because the base branch is configured for npm. Once we merge this PR all the subsequent PRs shouldnt fail. Example - https://github.com/Shreyas281299/react-widgets/actions/runs/11475909150

Mostly good. However,
* We do not this file to be checked-in: `.yarn/install-state.gz` as per [this documentation](https://yarnpkg.com/getting-started/qa#which-files-should-be-gitignored). Let us remove it before merging the PR. Quoting the same here: `.yarn/install-state.gz is an optimization file that you shouldn't ever have to commit. It simply stores the exact state of your project so that the next commands can boot without having to resolve your workspaces all over again.`

* Please change the PR title

* The CI seems to be failing. Please check that one too

Removed the yarn/install-state file Changed the PR title
For the pipelines failure- 1. CircleCi, will fail because I havnt made any changes to circleCi scripts as we are moving to github-actions. 2. Why github actions is failling? When github actions runs install it considers the base branch(which is still using npm). After we merge this PR and make the repo use yarn after that all the PRs will work fine. I tested this in my fork, here is the github action for that - https://github.com/Shreyas281299/react-widgets/actions/runs/11476200784/job/31935688832

This takeaway was incorrect. The actually issue was that github was not checking out the correct commit.
https://github.com/webex/react-widgets/pull/1449 Fixed in the PR


unmanaged-cluster create should have --base-node-image flag

Feature Request
To enable users to quickly test and try different base node images, we should add a --base-node-image flag to creation for unmanaged-clusters.
This will bypass the base node image resolved from the TKR and use the user defined one instead.
Describe alternatives you've considered
The alternative today is to push a new TKR with a different base image defined and use the --tkr flag. Users may also be able to successfully re-tag an image in their local cache (via docker tag for example) but both of these have long feedback loops and required many steps.
Given service outages, it would also be advantageous to enable users to point to a different image to still be able to bootstrap quickly without having to go through the above steps.

Our config today has NodeImage. So let's make sure whatever flag we have has the same consistent syntax.
Meaning, either:
--base-node-image
BaseNodeImage:

or (my preferred):
--node-image
NodeImage:

Sounds good 👍🏼
Maybe this issue (and subsequent PR) can reconcile any missing flags that should map into the config file.


Revert cms branch

Changes branch from can-82 back to cms-content-entry

 Restore the CMS content entry branch and config


Show local eNom price cache

It good idea to show current enom prices fetched.
Currently when we fetch enom prices , it shows nothing . I want to check new enom prices in whmcs and then update the prices if we need .

Basically you would like to see a page containing the cached eNom prices somewhere?

Yes exactly .  . I  like to see a page containing the cached eNom prices

The module have a feature with fetching new prices from enom but we need to see changes and then update the prices .
There is a good idea to see a page containing the cached eNom prices and current enom prices. and we can choose which tld we need to update.

Thank you for expanding on the request. We'll see when we can implement this feature.


Remove run—testsuite script

Historically the script was for exporting vars, but now.we have the env vars in bashrc, so we this script is useless.
We just change dir and run rake

ensure production (30, 31, head) doesnt run it, check sumaform testrunner in gitlab
update doc
remove sumaform code for generation of script

This is a easy fix task, so some.newbies can take it, to leaern the workflow

Strongly disagree, for the following reasons:
1 - Syntax with rake is not natural, and is difficult to remember.
2 - We might change the implementation with rake (add options etc) while keeping the run-testsuite command. We could even drop rake completely and call cucumber directly. Have an abstract command like "run-testsuite" makes such implementation changes transparent.
What we can do is remove the variables from run-testsuite command. They are indeed not needed anymore.

ok,
convinced only only for 2nd point.
The concept "Natural" in computer Science could require  a philosophy dissertation

The concept "Natural" in computer Science could require a philosophy dissertation

True :smile:


Determine tool for automated testing

Description
Determine tool for automated testing
Dependencies
Are there any dependencies?
DOD

[ ] Look into a licence - cost vs time savings
[ ] Discuss with team
[ ] Determine this sprint what best go-forward option is
[ ] Implement
[ ]

Duplicate of #444


Testing worker with npm modules

Hi all, I have been using your project to unit test some small workers and it is working very nicely for me. But, I have run now into some problems that I am not sure if they happen due to the cloudworker not being prepared for this use case, or for something that I am doing wrong. So more than an issue it is a functionality question.
I have a worker that requires a couple of npm modules. To work with this in production, I follow the process suggested by Cloudflare, and I use webpack to pack together the modules and the worker.
For unit testing the worker with cloudworker I am running in the issue that when I load the worker into the cloudworker (I use mocha, and do the loading inside the before method) it will not recognize the require function.
To overcome this issue, I switched to mocha-webpack, and a process that essentially is packing the modules and the worker together, and then trying to load into cloudworker the javascript bundle with all the moduels and the worker. But in this case, the functions of the worker don't seem to be accessible from the tests, so I cannot invoke them to run the unit tests.
For now I am essentially mocking the functions that depend in external modules, but I still would like to load the modules at some point. Any suggestion or idea to solve this issue?
Thanks in advance!

Closing this as I figured out a way of achieving this.
For future reference, if you are loading your worker with code similar to this:
new Cloudworker(require("fs").readFileSync("src/worker.js", "utf8"));
there is a second parameter to the Cloudworker constructor that accepts an object that can be composed of several properties: debug, bindings and enableCache. Using bindings helped me to allow require to exist in the worker context by adding it to the binding like this:
new Cloudworker(require("fs").readFileSync("src/worker.js", "utf8"), {bindings: {require: require}});
Cheers,
Nicanor


How to use inline query correctly

Question

Hello i need to use inline querys on my bot, but im having a hard time trying to understand them, i already set inline mode on botfather my question is the next:
by example i have de next command
bot.onText(/\/start/, (msg) => {
bot.sendChatAction(msg.chat.id, "typing");
bot.sendMessage(msg.chat.id, "hello world",);
});

this is a simple command on my bot_A, now the way i understand inline querys is that in my bot_B i should write @bot_A /start and it should show me themessage "hello world".
I understand that i have to use an especific syntax on my original bot to set my message to be used as inline, do i have to make 2 type of messages? one normal and one inline or it still works in either way?
and if you could help me with an example of how i have to make an inline message.

First: inline messages are not catched by onText.
Second: i don't clearly understand the connection between inline_query and your code example.
So i can make you a simple example:
bot.on("inline_query", function (data) {
    var content = data.query;
    console.log(content);
});

When you have configured inline_mode in botFather, you can use by writing @yourbot content in every chat box.
Anyway, pay attention at this article (especially at cache and feedback parts)

Sorry that example is a normal onText message it didn't have any inline connection so with the example you used in    var content = data.query; i should put my "hello world" message ?

The object data is:
{ id: 'MESSAGEID',
  from:
   { id: USERID,
     is_bot: false,
     first_name: 'FIRSTNAME',
     last_name: 'LASTNEMA',
     username: 'USERNAME',
     language_code: 'it' },
  query: 'CONTENT',
  offset: '' }

So you can send a message with:
bot.sendMessage(data.from.id, "hello world");

so my full code should look lik this?

const TelegramBot = require('node-telegram-bot-api');
const mysql = require('mysql');
const utf8 = require('utf8');

// replace the value below with the Telegram token you receive from @BotFather
const token = 'token';

// Create a bot that uses 'polling' to fetch new updates
const bot = new TelegramBot(token, {polling: true});

bot.onText(/\/start/, (msg) => {
  bot.sendChatAction(msg.chat.id, "typing");
  bot.sendMessage(msg.chat.id, "hello world",);
  });

  bot.on("inline_query", function (data) {
    var content = data.query;
    bot.sendMessage(data.from.id, "hello world");
    console.log(content);
});

Put console.log under var content ;)
Anyway yes.

Oh wait i noticed now.
When you use inline mode for A you must configure bot.on("inline_query", ... for bot A, the send a message to bot B un a way like:
**bot2**.sendMessage(...);

Oh im sorry when i say bot_B was an example like saying a chat from x person, the point is that from any chat i invoke @bot_A and write any command it should send the answer to the person, like the youtube bot, you send a video title and in that case it shows you a list of videos, i only need a response from the bot

Anyway i think i have understand your problem.
You want that your bot send a message in the chat instead of the user.
BUT is not easely possibile because as you can see in my object data example comments ago, you not have chat_id but only from.id.

i know :/ believe me and thats only the start of what i have to do with the inline messages, well thank you for your time @sidelux  i will try to use what you showed me to make something work.


Test helper that asserts a given transition occurred

Automat should have a test helper that asserts a given transition occurred after running a block of code.
It might go a little something like this:
with assertTransitioned(someMachineInstance,
                        through=[SomeMachine.state1, SomeMachine.state2]):
    someMachineInstance.blah()
    someMachineInstance.otherblah()

This could use #36


Wormhole client wasm interface spike

This PR is intended to provide a place to review the results of the current state of this spike.
Changes:

Switches websocket libraries for WASM support.
(from"github.com/gorilla/websocket" to "nhooyr.io/websocket")
Removes hide progress bar flag along with proxyReadCloser and respective usages.
Adds javascript wrapper for wormhole client constructor and methods.
Adds go wrapper for javascript promises.
(cleanly ties go into the JS concurrency model while avoiding deadlocking)
Adds (crappy) transport switching based on protocol parsed from the relay URL.
(this may not be the direction we want to go -- we should discuss this)
Disables direct connection attempts.
(we need to choose a method to determine whether we're in a browser/wasm context (maybe the same thing used for protocol switching) and re-enable this in other contexts)
Removes \ns from some string constants.
(this should most likely be reverted but the goal (and future test) is for the message to look the same regardless of the transport used. We might have to make changes elsewhere to do this correctly)

Areas to improve:

Consolidate go->JS exports into a single JS object.
Error handling (can be driven out via testing)
Testing

go wrapper for Promise.
JS client API (may make sense to do these test in JS)
transport switching logic
mailbox/relay integration (including server setup/teardown, etc.)

Review complete.


Frontend can't scan table with staled schema

What type of bug is this?
Unexpected error, Other
What subsystems are affected?
Frontend
What happened?
Frontend might contain the staled schema, e.g., the heartbeat stream is broken, and some invalidate cache messages are lost.
However, Frontend will return an incorrect stream.
https://github.com/GreptimeTeam/greptimedb/blob/9282e59a3bb9de6984776933e5b4a5c394d5a407/src/query/src/dist_plan/merge_scan.rs#L177-L193
L190, It uses the staled schema.
L180, The RecordBatch might contain a newer schema.
We should be able to align one schema to another and tell the upper to invalidate the table cache.
What operating system did you use?
Doesn't matter
Relevant log output and stack trace
No response
How can we reproduce the bug?
Build a Frontend with a table contains a staled schema e.g.,  [col:int, col2:int, ts: Timestamp]
Build a Datanode with a table contains a new schema e.g.,  [col:int, col2:int, col3: int, ts: Timestamp]
Scan the table

Is this bug still present? If yes, let's resolve it as soon as possible.

This seems quite a common issue in our internal cache. We have the same cache invalidation issues for schemas and scripts.
For scripts, there is even no notification to tell the entry is stale. For table schema here, it can have a minute-level inconsistent time window.
Are we theoretically able to keep the cache up-to-date or at least invalidate it promptly? It seems we would then depend on the meta server ...

Maybe we could add a query node or a stream adapter to adjust the stream from different region servers.
Currently, the mito engine uses a similar approach to adapt SSTs with different schemas.

What's going on with this issue? @WenyXu

What's going on with this issue? @WenyXu

I'll fix it recently👀


Draft: testing d.ts

#185
@richytong hi!
Its only for demo. We can create d.ts for each exported function (https://rubico.land/docs) for the first time. In some types you don't have to bother with generic - the main idea is to give access to users with ts.
We can write d.ts and copy to dist
what you think about it?

Codecov Report
Patch and project coverage have no change.

Comparison is base (7150e31) 98.80% compared to head (9f4fba1) 98.80%.

:exclamation: Current head 9f4fba1 differs from pull request most recent head 53444d6. Consider uploading reports for the commit 53444d6 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #241   +/-   ##
=======================================
  Coverage   98.80%   98.80%
=======================================
  Files         233      233
  Lines        3420     3420
=======================================
  Hits         3379     3379
  Misses         41       41

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Hello @kalagin I'm not too familiar with TypeScript. Can we move the d.ts files to their own directory?

@richytong yes, we can create 1 d.ts file, for e.g index.d.ts and specify in package.json
....
  "types": "index.d.ts",
....

@kalagin I am good with all the types living in the index.d.ts file. Do you have a plan for adding the rest of the types?

@kalagin let's move the types in this pr to the index.d.ts file

@richytong im create tsc command for generating:
npm run types

it generates index.d.ts. it works
but i have problem (see demo.ts)
// in this case works fine
const areNamesEqual = eq("Ted", "George");
console.log(areNamesEqual); // false

// but in this case we need cast types
const personIsGeorge = eq((person: Person) => person.name, "George") as (person: any) => boolean;

we need cast types, because tsc generate type for eq like this:
function eq(left: any, right: any): boolean | ((value: any) => any);

without generics and overloads.
i think we can generate once and fix generated types and write generics like in eq.d.ts (prev commit)
while thinking what to do

next idea: we can overwrite union types like boolean | ((value: any) => any) to any, user will can use library with typescript. and then i can write types step-by-step with tests/type-tests

I think it's a good direction @kalagin . I would opt to keep the types as simple as possible, so use any when possible. I had a feeling it would be this challenging to do the types.

@richytong i cant make one d.ts, because with this solution tree-shaking not working import eq from 'rubico/eq
we can generate d.ts for each file into dist, but dist should become "root" folders for imports.
what do you think?
whats purpose of dist folder?

@kalagin I think it's probably the best to go your original proposal of having a d.ts file for each exported function. Do you know if we have to make a d.ts file for a module export like index.js?
Also the dist folder is for browser exports using unpkg as a cdn. The imports look like this
<script src="https://unpkg.com/rubico/dist/global.min.js"></script>

@richytong im add d.ts for each js file (npm run dts)
in the future we can get file by file, write advanced types and add ready d.ts files to exclude in tsconfig.json

I'm going to merge the changes now. Feel free to open another issue or PR if there is anything else.


Changed style of "Add products" button in comparision layout to "Border button" style

[x] Apply the AndroidStyle.xml style template to your code in Android Studio.
[x] Code is well documented
[x] Include unit tests for new functionality
[x] All user-visible strings are made translatable
[x] Code passes Travis builds in your branch
[x] If you have multiple commits please combine them into one commit by squashing them.
[x] Read and understood the contribution guidelines .

Raphael, guardian of the Buttons ;-)


Allow edit mode while timer is running

Hi, not sure why this feature was removed in 0.2.0 but I used to edit the time as it was running and now it's not possible.

Hi! This used to cause a lot of issues and confusion for a lot of people. Can you describe why you need this feature? What specifically would you like to be able to edit? Just the name, or the active times too?

My use case is when I forget to start the timer for a given task and I would just edit the start time immediatly after starting it.
Now I have to start, stop, edit the start time, continue again thus creating a subtimer which is not desirable.
I would like to at least edit the name and start time.

Toggl track mobile app has "Set to last time" option, which can be added to fix this issue.

Toggl track mobile app has "Set to last stop time today" option, which can be added to fix this issue.

This would not work for the first task of the day, which, in my case, is the one I mostly edit the start time.

Hi. This issue is still open and has the enhancement label applied to it, which was my way of communicating that this will be implemented again :)

Glad to hear it! Just came to say that I'm using it in the exact same way - forgetting to turn it on (or starting my work when I'm not with one of my devices) and then updating the start time immediately. Thank you for making a tool I use on a daily basis!

Also great to hear. I love your time tracking tool. Sorry for my last message being somewhat confrontational, there was a lot going on at work at the time.


add capmetro transit stops

Were you thinking of adding "7 transit stops are within 0.5mi of parks"?
Or actually showing thousands of stop markers?

I totally agree. Could you start to make assumptions about the importance of a stop based on how many different bus routes belongs to?
It seems like a feature of a park is that is would be near such a bus route intersection?

taking this idea in Issue #32


chore: Fix disruption reason casing

Fixes #N/A
Description
Disruption reason casing is still wrong for prometheus and for logging
How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

16 of 17 (94.12%)  changed or added relevant lines in 5 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.04%) to 77.927%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controllers/disruption/controller.go
5
6
83.33%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/node/termination/controller.go
2
66.47%

Totals

Change from base Build<PHONE_NUMBER>0:
0.04%

Covered Lines:
8900

Relevant Lines:
11421

💛  - Coveralls


cudatext: 1.148.0 → 1.150.0

Motivation for this change
Changelog
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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

[x] Fits CONTRIBUTING.md.

@ofborg eval

Result of nixpkgs-review pr 147615 run on x86_64-linux 1

2 packages built:

cudatext (cudatext-qt)
cudatext-gtk


Many-to-many light-opsin support

Currently, only one OptogeneticIntervention could be used on a neuron group at once. This is because only one Iopto gets added. We could support more by making Iopto a sum of currents identified by names, e.g., Iopto_name.

Yes, an epsilon dictionary parameter would define sensitivity to different wavelengths. For ChR2, for example, epsilon = {473: 1, 620: 0.5} or something like that, meaning it responds half as much to 620-nm light as to 473-nm light.

My best idea for implementing this refactor so far: make an opsin an InterfaceDevice and use an inject function syntax for both opsin and light, e.g.:
sim.inject(opsin, ng1, ng2, **kwargs)
sim.inject(light1, ng1, ng2)
sim.inject(light2, ng1, ng2)

An Opsin, when connected to an NG, would set up a light aggregator neuron for each target neuron and a Synapses implementing the opsin model as before, but from the light neurons to the target neurons, instead of target->target. The target neuron would keep track of transformed opsins with a dynamic attribute for the next step:
Injecting a LightDevice to a neuron would set up a Synapses object from each light source in the LightDevice to the light aggregator neurons of each opsin in the target neuron group. E.g., for an array of 10 LEDs, there would be 10 Synapses objects, one each onto the ChR2 light aggregator NG of a given target NG. If the target NG had a second opsin, e.g., Jaws, there would be another 10 Synapses objects, each onto the light aggregator NG of the Jaws opsin for the target NG

Could possibly have a current aggregator NG, so Light->light aggregator--(opsin model)—>current aggregator

Try attrs for parameter-heavy classes.
Structure:
Light - for single or multi-source device

takes LightModel, coordinates, orientation

LightModel

abstract, functions for T and visualization

FiberModel, LEDModel, etc.

concrete, contains default params

Don't subclass for specific parameters---instead provide a function like fiber473nm() that constructs a FiberModel with wavelength-dependent params.

More thoughts from notes on phone:

LightModel would have T function, as well as viz point generation function

implemented in #33


Small modal window error fix

Fixed error where message about wrong typed number is not disappearing after closing modal window

I believe that this bugfix didn't break anything.


Adding skip_common_chunks option to the get_files() template tag

Season's greetings. :)
I need get_files() to be able to use skip_common_chunks just like render_bundle() does, so I took a couple hours and implemented it.
The goal here is to be able to render <link rel="prefetch"> (not preload) tags manually, skipping the ones from the chunk that are already downloaded via render_bundle().
Have this as my free contribution. :)

bump

With all due respect, you are placing too much weight on the coverage. My code uses the same underlying functions that are already tested out.
Feel free to add tests though, I just can't be bothered.

Thanks for adding the tests. Could you please do a quick release so I can start using it?

Thanks. Can confirm it's working.


Allow declaring properties inside class constructors

It's a little clunky setting up properties in classes right now.
Here's what I have to do at the moment to set a property value in the constructor:
class Animal {
    public interruptedSound: string;
    constructor(public sound: string) {
        this.interruptedSound = sound.substring(0, 3);
    }
}
let dog = new Animal("woof");
console.log(dog.sound); // woof
console.log(dog.interruptedSound); // woo

Notice that interruptedSound's setup is split over two lines, and I have to declare it as a string even though it could be inferred from the fact that sound.substring returns a string.
What I'd like to be able to do:
class Animal {
    constructor(public sound: string) {
        this.interruptedSound = sound.substring(0, 3); // now a detectable property of type string
    }
}
let dog = new Animal("woof");
console.log(dog.sound); // woof
console.log(dog.interruptedSound); // woo

This would avoid having to declare all properties at the top before being able to use them in constructors, saving a lot of boilerplate on complex classes. F# already does something similar to make setting up types much quicker and simpler. There wouldn't be any JavaScript compatibility issues, because JavaScript already works this way.
In cases where the property is declared separately, give precedence to the declaration.
The private, public or readonly keywords could even be used before the assignment to modify the property, e.g.
class Animal {
    constructor(public sound: string) {
        private readonly this.interruptedSound = sound.substring(0, 3);
    }
    changeSounds() {
        this.sound = "meow";
        this.interruptedSound = "meo"; // error, interruptedSound is readonly
    }
}
let dog = new Animal("woof");
console.log(dog.sound); // woof
console.log(dog.interruptedSound); // error, interruptedSound is private

Thoughts?

@RyanCavanaugh any thoughts on this?

You can already do something like this (slightly unsafe if the constructor is aliased by a constructor with more arguments, but this is rare):
class Foo {
  constructor(x: string);
  constructor(public x: string,
    public y = x.toUpperCase(),
    private z = 3) {

  }
}

Ah, I didn't know I could access other parameters when assigning a value to constructor parameters like that. It'd still be nice to have the feature, but this is good enough for me, so I'll close this issue. Feel free to re-open if someone else thinks this would be worth introducing.

Actually, I want to re-open this one. With es6 classes, this feature would make it much, much less painful to upgrade from JavaScript to TypeScript. There are many perfectly valid, well written javascript classes that do something like
class Foo {
  constructor() {
    this.port = process.env.PORT || 12345;
  }
}

And these will all give errors that could be avoided if this was added to the language.
I just tried upgrading a medium-sized JavaScript project to TypeScript, and just over half of all errors are of this type.
That was on 207 javascript files, with around 100 lines of code each. There were 272 errors, 146 of which would go away with this feature.
@RyanCavanaugh while your workaround will work ok for existing TypeScript projects, it's not very idiomatic, so it doesn't help those moving from JavaScript (plus it's confusing). Do you know how realistic it is to hope to get this feature into a future typescript release?


Cannot find module './transformers/"ts"'

If you use lang="ts" in combination with the copy on a code fence, you get the following error:
Error while preprocessing /home/.../[...1]header.md - Cannot find module './transformers/\"ts\"'

This works:
```svelte
<script lang="ts">
</script>
```

This also works:
```svelte copy
<script>
</script>
```

This does not work:
```svelte copy
<script lang="ts">
</script>
```

Cheers, fixed in 0.18.4.


Can keep loras in memory?

The webui has an option Number of Lora networks to keep cached in memory, and I set it to 15
But occasionally switching Lora is slow, and then Loading TensorRT engine: this log appears

I got it ,The converted Lora file is empty


fix: :memo: examples using no longer present default keys

Replaced key return with back
Replaced key source with tv
Updated example images to reflect the changes

Hey, I'm working on a video about your card and noticed some examples used the original tv-card keys, so I updated them to avoid confusion.
Thank you for your work.
Pd: I looked into making it work with the Lovelace editor, but haven't found a way to include a row creation feature user friendly enough.

Thanks for your updates @fixtse! I hadn't touched most of the examples but I should go through and keep them all up to date. Make sure to also give credit to the creators of the cards this was forked from in your video! They did most of the work, getting it to work with Android TV wasn't too hard.
I'm not sure where to start when it comes to a proper lovelace editor. The card this was forked from had a .js file for one but it wasn't being used. This card's parent repo also had some weird design decisions that may not be UI friendly like not putting all the rows into an array and making them each a root level object key instead, which I don't want to do now as it would be a breaking change.

I saw you fixed them all, and I was also thinking about that google assistant key that you just added GREAT JOB!
This is the video btw https://youtu.be/Qv5mwCyLjh4 , hope I made it justice.
About the Lovelace editor, It should be in the main js file for home assistant to use the editor (as all the logic resides in one file), I looked at it, but it was designed for the original tv-card, and from that to the fork this one is based on, there are too many differences in structure for it to work as a base.
I was thinking something like the entities card, which has an option to add more and more entities, but instead of entities, it would be rows.
I will be looking in more depth into it in the future, as I need to learn more to add more functionality to my o365 card as well, If I find something that can help with the type of config this card has, I'll let you know

Just watched it. Thanks for the shoutout! Like and subscribed.
I'll keep kicking the config UI can down the road. Probably after I eventually refactor this to be written in TypeScript instead and compile into one JS file.


Convert simple API collection to OO controllers

Purpose or Intent
Collections that neither are a subcollection nor have subcollections can
be easily converted to OO-type controllers. The remaining collections
that do not fall into this category are more problematic, so will be
done in future revisions.
The converted collections are:

arbitration profiles
arbitration rules
arbitration profiles
arbitration rules
arbitration settings
availability zones
container deployments
currencies
flavors
instances
measures
pictures
provision dialogs
requests
security groups
servers
virtual templates
zones

@miq-bot add-label api, refactoring
@miq-bot assign @abellotti

This is fine @imtayadeway, one step at a time. At lease those collections are in their new file/namespace.  Thanks!!


tools: exclude non-project files from go version consistency lint check

I noticed that the lint check (Go versions consistency check) was examining my local vendor directory. This directory contains project dependency files, which we shouldn't try to lint. Therefore this PR introduces an exclusion list to avoid checking certain directories.

@dstadulis: review reminder

@dstadulis: review reminder


Added contribution guide

fixes #945
fixes #929
fixes #858

This PR finalizes set of actions to start with a new branch-based development model. I haven't found any suitable fully automated model like https://github.com/googleapis/release-please-action that matches our needs.
Anyway, the version updates are automated so for now we just need to maintain a releasing PR manually (which might be a good thing in the end).
Live preview - https://929-contribution-guide.datapackage-6gp.pages.dev/overview/contributing/

Hi @peterdesmet, can you please take a look? Once, it's merged we will have a free way to work on new versions I believe

@roll I'll try to have a look this week. Can you already a a .github/CONTRIBUTING.md (a default place for such a guide) and link to the web page?

@peterdesmet
Thanks!

Can you already a a .github/CONTRIBUTING.md (a default place for such a guide) and link to the web page?
I will try to do so

Have you had a chance to take a look?

I haven't, it's on my todo 😄

Or you would just like me to move CONTRIBUTING.md first?

We could move it or make it easier have that file just refer to the (to be published) contributing guide on the website, cf. what I do here for the code of conduct: https://github.com/frictionlessdata/frictionless-r/blob/main/.github/CODE_OF_CONDUCT.md

Agree, moved it to the repo root using a symlink


Manual segmentation of MS lesions in the spinal cord to be reviewed by neuroradiologist

Creating this issue to centralize all images which are tricky to annotate and which could be discussed with a neurologist.

This case is not that easy as well to know if it's one big lesion or multiple lesions : Canproco - sub-tor029_ses-M0_PSIR.nii.gz

[!Note]
Code to get image : fsleyes sub-tor029/ses-M0/anat/sub-tor029_ses-M0_PSIR.nii.gz derivatives/labels/sub-tor029/ses-M0/anat/sub-tor029_ses-M0_PSIR_lesion-manual.nii.gz &

In this case, lesions have been annotated with multiple values (1 and 2). One theory is that the annotation comes from the level of confidence of the lesion annotation. Subject : sct-testing-large - sub-rennesMS002_acq-sagcerv_T2w.nii.gz

The lesions in yellow could be focal lesions ? Dirty white matter ?

[!Note]
Code to get image : fsleyes /sct-testing-large/sub-rennesMS002/anat/sub-rennesMS002_acq-sagcerv_T2w.nii.gz /sct-testing-large/derivatives/labels/sub-rennesMS002/anat/sub-rennesMS002_acq-sagcerv_T2w_lesion-manual.nii.gz &


Remove usage of a classifier

based on RHOAR team, the use of the classifier should be nowadays avoided. They were initially using as workaround. It seems that they are now using another workaround which is "cleaner". It allows to stay closer to an out of the box SpringBoot project and then to have it working with odo
The other workaround is to specific a name on the maven-failsafe-plugin classes-directory, see https://github.com/snowdrop/spring-boot-http-booster/blob/a5b271f3bf1356d5e46b75ef7d463ea7d462c1a9/pom.xml#L194 and notice that there is no classifier https://github.com/snowdrop/spring-boot-http-booster/blob/a5b271f3bf1356d5e46b75ef7d463ea7d462c1a9/pom.xml#L168-L178
See the commit which did that https://github.com/snowdrop/spring-boot-http-booster/commit/1eac1c2780968a48a0aeceb9c7bb5b712cde72a2
@valdar do you know if it can cause trouble to follow the same way? Any opinions?

created https://issues.jboss.org/browse/ENTESB-9846 as it applies to all quickstarts/boosters/examples

To clarify this issue @apupier , are you suggesting we remove the classifier as defined here? - https://github.com/jboss-fuse/fuse-rest-http-booster/blob/824ceb03269309551509b1b3a0167446e596078d/pom.xml#L170

I suggest to follow what has been done in snowdrop project snowdrop/spring-boot-http-booster@1eac1c2
I.e. removing the classifier and configuring the maven-failsafe-plugin

Created https://github.com/jboss-fuse/fuse-rest-http-booster/pull/27


full server not required for jenkins images

Only client tools seem to be needed for Jenkins images. Do we have use cases
which require full server.
@droslean FYI

maybe the unit test failure is a gradle flake ... no idea why changing where we get oc from would matter there
the e2e failure was with our old e2e set, with flaky tests given image push issues
going to retest

/retest

@bparees found a possible cause for the gradle failure (if it is persistent):  https://github.com/keeganwitt/docker-gradle/issues/57
Here is the gradle log from the failed test:
STEP: printing container logs
OPENSHIFT_JENKINS_JVM_ARCH='', CONTAINER_MEMORY_IN_MB='8796093022207', using /usr/lib/jvm/java-1.8.0-openjdk-<IP_ADDRESS>.b12-1.el7_6.x86_64/jre/bin/java
OPENSHIFT_JENKINS_JVM_ARCH='', CONTAINER_MEMORY_IN_MB='8796093022207', using /usr/lib/jvm/java-1.8.0-openjdk-<IP_ADDRESS>.b12-1.el7_6.x86_64/bin/javac
Starting a Gradle Daemon (subsequent builds will be faster)

> Starting Daemon> Starting Daemon> Starting Daemon> Starting Daemon> Starting Daemon> Starting Daemon> Starting Daemon> Starting Daemon> Starting Daemon> Starting Daemon> Starting Daemon> Starting Daemon> Starting Daemon> IDLE
FAILURE: Build failed with an exception.

* What went wrong:
Could not create service of type ScriptPluginFactory using BuildScopeServices.createScriptPluginFactory().
> Could not create service of type CrossBuildFileHashCache using BuildSessionScopeServices.createCrossBuildFileHashCache().

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

* Get more help at https://help.gradle.org

BUILD FAILED in 2s

The implication would be there is some directory presence/permission difference between origin and origin-cli.  Not sure what it would be off hand.  Some experimentation is possibly required, adding the --stacktrace, etc. args.

cluster standup failure on e2e
/test e2e-gcp

yum failure this time

/retest

btw I didn't update the e2e image because I thought it may require additional packages comared to origin-cli. If something fails with it, it is totally not from this PR.
If e2e shouldn't require full server, then I can update it as well.

the e2e image does not come into play with the failure I previously described in the ci/openshift-jenkins job @akostadinov

/test e2e-aws
/test e2e-aws-jenkins
/test rhel-images

closing in deference to https://github.com/openshift/jenkins/pull/774 which uses the quay registry for the centos images explicitly.


Fix typo in Scaladoc of Service

LGTM. Will merge after token travis check.

That intermittent build failure on the client tests has gotten more intermittent, but we're not there yet. sigh
Merging this, as it's definitely not at fault.


Feature Request: Add a timecompressor as postprocessing option

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.05.18. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.05.18

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[ ] Question
[ ] Other

A common usage of youtube-dl is to download podcasts or instructional youtube videos for offline usage/archival. The people who listen to educative podcasts want to minimize their learning time, so they often increase the playback speed of the podcast mp3's or videos. On the other hand, they also want to save storage space.
Why can't we combine these two requirements in one feature? Youtube-dl could download the video and then re-encode it using ffmpeg at an increased framerate. With this "baked-in" playback speed increase, we also save storage space.
Example invocation:
youtube-dl https://www.youtube.com/watch?v=1n9KMqssn54 --playback-speed=2.0
As a proof-of-concept, I tried to reencoded the mentioned video to 2.0 playback speed using a manual ffmpeg call:
> youtube-dl https://www.youtube.com/watch?v=1n9KMqssn54
> dir *.mp4
[..]
02.03.2018  11:10       295.366.625 Using PyPy instead of Python for speed by Niklas Bivald-1n9KMqssn54.mp4
> ffmpeg -i ...1n9KMqssn54.mp4 -filter_complex "[0:v]setpts=0.5*PTS[v];[0:a]atempo=2.0[a]" -map "[v]" -map "[a]" timecompressed.mp4
> dir *.mp4
24.05.2018  22:08       133.155.279 timecompressed.mp4
02.03.2018  11:10       295.366.625 Using PyPy instead of Python for speed by Niklas Bivald-1n9KMqssn54.mp4

My PoC shows that the timecompressed version of the video saves about 56% storage space and about 12 minutes listening time (original duration ~24 minutes). I have to admit that the timecompressed version is not perfectly understandable and took a while to encode, but I haven't tweaked the settings yet and I am using a pretty old machine.
According to a recent lifehacker article, playback speeds from 1.4 to 1.7 are pretty common today: https://lifehacker.com/speed-up-your-podcasts-for-more-efficient-listening-1797029702
What do you think?

Why is this a duplicate of #15840? My proposal is about resampling the video for increased playback speed, #15840 is about slicing a video at a given start/ end time.


Removed code coverage badge since that is not currently being used.

Badge is stale, coveralls.io is no longer used in the crate from my inspection.

Thank you! :+1:


Can we calculate a band index contributing to intrinsic thermal conductivities at phono3py code?

Dear Togo,
I'm your follower and a graduate. But I have a question puzzling me so long. It is whether we calculate a band index contributing to intrinsic thermal conductivities in phono3py code. Just like at Lindsay, L.; Broido, D. A.; Reinecke, T. L., First-principles determination of ultrahigh thermal conductivity of boron arsenide: a competitor for diamond? Phys Rev Lett 2013, 111 (2), 025901. this article Fig. 3, in which the formula is ref[36] k(v)=1/V.... Excause me, I did not write it out in here. Please read the article Fig. 3 and ref[36](just a formula). How can do it in detail? If it do not, can you give me some adives to solve the problem? Thank you so much! Look forward to your reply! Best wishes!
Your Follower: Nileme

This should be discussed in the phonopy mailing list.
C_\lambda, \tau_\lambda, v_\lambda \otimes v_\lambda are in kappa-xxx.hdf5. So you can get the same results by selecting phonons with your band indices, but the integration with delta function has to be done by yourself.

I close this issue.

Thank you so much!


Update es-419.coffee

Hi, I just started to see your web but It will be a pleasure help you to improve.
Kinds Regards

Hi @corozb could you sign our CLA so we can accept your PR?

Hi Valentin, I did it. Now I'm ready?
El lun., 27 may. 2019 a las 0:32, Valentin Bryukhanov (<
<EMAIL_ADDRESS>escribió:

Hi @corozb https://github.com/corozb could you sign our CLA
https://contributors.codecombat.com/cla so we can accept your PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codecombat/codecombat/pull/5361?email_source=notifications&email_token=AJB3TLVYAVGN54ECQTJCIOLPXNW5BA5CNFSM4HPI5SP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWIZ5JY#issuecomment-496082599,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJB3TLVXD2GCPHIHUSUZNVDPXNW5BANCNFSM4HPI5SPQ
.

--
Cristian Orozco

Thanks!


Warmup takes forever

Hi @csukuangfj  I tried sherpa with mgb2 streaming transducer from here https://github.com/k2-fsa/icefall/tree/master/egs/mgb2/ASR/pruned_transducer_stateless5. I used https://github.com/k2-fsa/icefall/blob/master/egs/librispeech/ASR/pruned_transducer_stateless4/export.py to generate the jit model with the following command python pruned_trancducer_stateless5/export.py --streaming-model 1 --causal-convolution 1 --jit 1 --epoch 18 --avg 5 --bpe-model data/lang_bpe_2000/bpe.model.
It takes forever when I run (I waited for 2 hours and it is still stuck at warmup, see the screenshot)
/sherpa/bin/streaming_pruned_transducer_statelessX/streaming_server.py   --lang-dir data/lang_bpe_2000  --endpoint.rule3.min-utterance-length 1000.0   --port 6006   --max-batch-size 30   --max-wait-ms 5   --nn-pool-size 1   --nn-model-filename ./mgb2/exp/cpu_jit.pt   --bpe-model-filename ./mgb2/data/lang_bpe_2000/bpe.model

Could this be because I am using a larger bpe=2000? I tried your model from here https://k2-fsa.github.io/sherpa/python/streaming_asr/conformer/conformer_rnnt_for_English/server.html and it worked perfectly fine. Any ideas?

Could you please press
ctrl + c
and post the error message?

2022-12-30 10:26:28,100 INFO [streaming_server.py:296] Using device: cuda:0
2022-12-30 10:26:33,959 INFO [streaming_server.py:380] Warmup start
^C2022-12-30 10:27:04,352 ERROR [base_events.py:1707] Task exception was never retrieved
future: <Task finished name='Task-2' coro=<StreamingServer.stream_consumer_task() done, defined at ./sherpa/bin/streaming_pruned_transducer_statelessX/streaming_server.py:396> exception=Error('The following operation failed in the TorchScript interpreter.\nTraceback of TorchScript, serialized code (most recent call last):\n  File "code/__torch__/joiner.py", line 22, in forward\n      pass\n    else:\n      ops.prim.RaiseException("AssertionError: ")\n      ~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE\n    _3 = torch.slice(torch.size(encoder_out), None, -1)\n    _4 = torch.slice(torch.size(decoder_out), None, -1)\n\nTraceback of TorchScript, original code (most recent call last):\n  File "/alt-arabic/speech/amir/k2/tmp/icefall/egs/mgb2/ASR1/pruned_transducer_stateless5/joiner.py", line 55, in forward\n          Return a tensor of shape (N, T, s_range, C).\n        """\n        assert encoder_out.ndim == decoder_out.ndim == 4\n        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE\n        assert encoder_out.shape[:-1] == decoder_out.shape[:-1]\n    \nRuntimeError: AssertionError: \n')>
Traceback (most recent call last):
  File "./sherpa/bin/streaming_pruned_transducer_statelessX/streaming_server.py", line 421, in stream_consumer_task
    await loop.run_in_executor(
  File "/speech/toolkits/espnet/tools/anaconda/envs/k2/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/speech/toolkits/espnet/tools/anaconda/envs/k2/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "/speech/toolkits/sherpa/sherpa/bin/streaming_pruned_transducer_statelessX/beam_search.py", line 347, in process
    ) = streaming_greedy_search(
torch.jit.Error: The following operation failed in the TorchScript interpreter.
Traceback of TorchScript, serialized code (most recent call last):
  File "code/__torch__/joiner.py", line 22, in forward
      pass
    else:
      ops.prim.RaiseException("AssertionError: ")
      ~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
    _3 = torch.slice(torch.size(encoder_out), None, -1)
    _4 = torch.slice(torch.size(decoder_out), None, -1)

Traceback of TorchScript, original code (most recent call last):
  File "/alt-arabic/speech/amir/k2/tmp/icefall/egs/mgb2/ASR1/pruned_transducer_stateless5/joiner.py", line 55, in forward
          Return a tensor of shape (N, T, s_range, C).
        """
        assert encoder_out.ndim == decoder_out.ndim == 4
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
        assert encoder_out.shape[:-1] == decoder_out.shape[:-1]

RuntimeError: AssertionError:

Traceback (most recent call last):
  File "./sherpa/bin/streaming_pruned_transducer_statelessX/streaming_server.py", line 742, in <module>
    main()
  File "/speech/toolkits/espnet/tools/anaconda/envs/k2/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "./sherpa/bin/streaming_pruned_transducer_statelessX/streaming_server.py", line 716, in main
    asyncio.run(server.run(port))
  File "/speech/toolkits/espnet/tools/anaconda/envs/k2/lib/python3.8/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/speech/toolkits/espnet/tools/anaconda/envs/k2/lib/python3.8/asyncio/base_events.py", line 603, in run_until_complete
    self.run_forever()
  File "/speech/toolkits/espnet/tools/anaconda/envs/k2/lib/python3.8/asyncio/base_events.py", line 570, in run_forever
    self._run_once()
  File "/speech/toolkits/espnet/tools/anaconda/envs/k2/lib/python3.8/asyncio/base_events.py", line 1823, in _run_once
    event_list = self._selector.select(timeout)
  File "/speech/toolkits/espnet/tools/anaconda/envs/k2/lib/python3.8/selectors.py", line 468, in select
    fd_event_list = self._selector.poll(timeout, max_ev)
KeyboardInterrupt

Could you please delete the assert statement, re-export your model and try again?

@csukuangfj can you please elaborate which assert statement you mean?

Please see the above error message you just posted.
It is from joiner.py

Thank you it is working now.
I recorded small demo using my mic here check it https://youtu.be/2uh3zVAFyQ4

Thanks! I will update the documentation to add a link to the video.
Would you mind also recording a video showing how endpointing works?
(You only need to pause for  a while, e.g., 2 seconds, before you say the next sentence.)

Thanks @csukuangfj , here is the video with pauses https://youtu.be/t2SlrzgMd_k.
BTW I noticed that the streaming pruned stateless transducer model trained on MGB-2 is very robust to noise and background music but when I switch to English the model totally ignores my speech, It only transcribes when I speak in Modern Standard Arabic. Comparing this to Espnet blockwise streaming transformer/conformer trained on MGB-2, which is more sensitive to noise and music but it also transcribes everything that I say even when I switch completely to English it still uses Arabic scripts to transcribe it. Is there any reason why the  streaming pruned stateless transducer ignores my English speech? How can I make it more sensitive to any words being said?

here is the video with pauses https://youtu.be/t2SlrzgMd_k.

Thanks!

Is there any reason why the streaming pruned stateless transducer ignores my English speech? How can I make it more sensitive to any words being said?

Sorry, I don't have any suggestions for that.

By the way, the Arabic endpointing demo has been added to
https://k2-fsa.github.io/sherpa/python/streaming_asr/endpointing.html#endpointing-demo-arabic

Awesome thank you @csukuangfj


add an importable variable imported on the API side

Have been commited in branch define-parent-kwargs-in-view


[JS] Fix adaptivecards-fabric tests on Linux

Description
I noticed that the adaptivecards-fabric tests were failing in Linux. The failures are due to path case sensitivity -- fix is easy.
How Verified

Ran tests :)

Microsoft Reviewers: Open in CodeFlow

/azp run

thanks @dfisenko


Allows private vis for authenticated users [don't merge]

This PR:

Fixes the backend to enable requesting vizjsons of private & password protected visualizatons for authenticated users.
Removes the restrictions in the frontend to ask for previews of the aforementioned visualizations.
Specs: [pending].

Frontend tests were OK :+1: (details)

what would happen if I include in my page something like (user has a session enabled in cartodb)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

@javisantana Just tested locally. It works the same as if directly requested:
As the request to get the vizjson is done by the client, if you have an open session in the browser it will:

check if you're the owner of the vis (current_user) and you're authenticated
check if you have permissions to see the vis (current_viewer) and you're authenticated
check if the vis is public or with_link if unauthenticated

So, if I know a visualization protected with password I could create a page with that script, make the user enter in that page and I would be able to see the visualization (because I see the viz.json), right?

Yes, we should make sure by doing a POC but that should work.


StartRecordingAsync: add parameters 'frameRate' and 'bitRate', select format via extension (on Android)

This PR addresses issues #8 and #9 by making CameraView.StartRecordingAsync more flexible on Android ...

It adds two optional parameters 'frameRate' and 'bitRate'. Both of them were previously hard-coded, and the hard-coded values are now used as default values for the parameters. Both parameters are only used on Android right now.

The extension of the provided file name is used to determine the output format (only on Android as well). The following options are supported (for other extensions, an ArgumentException is thrown):

.3gp (with H264 & AAC codecs)
.mp4 (with H264 & AAC codecs)
.webm (with VP8 & Vorbis codecs)

@mancda, @tpgalan Can you test this and tell me if it works well for you? Do you have any feedback?

@mancda, @tpgalan Can you test this and tell me if it works well for you? Do you have any feedback?

Here is a CI-generated nupkg that can be used for testing:
https://github.com/janusw/Camera.Maui/actions/runs/7964843273#artifacts

Works great!  Thank you very much.

Works great! Thank you very much.

Thanks for the feedback!
@tpgalan If you have an comments, please let me know soon. Otherwise I'll just merge this as is.

Works great! Thank you very much.

Thanks for the feedback!
@tpgalan If you have any comments, please let me know soon. Otherwise I'll just merge this as is.

I didn't have time to test it, but when I do I'll let you know. I suggest to close the issue and I'll let you know once tested.
Thanks for your time.


With Vagrant 1.7.1 (and maybe 1.7.0), I can no longer launch unless the box is added or an AMI is specified

For a while now, I have had a Vagrantfile that uses boxes created by Packer.  It does this by setting options.vm.box and options.vm.box_url.
With Vagrant 1.6.5 and earlier, when I go to vagrant up, if the box isn't already added, vagrant will add it for me by fetching it from the box_url.
With Vagrant 1.7.1 (and maybe with 1.7.0, but I haven't tested that version), I get the following error:
There are errors in the configuration of this machine. Please fix
the following errors and try again:

AWS Provider:
* vagrant_aws.config.ami_required

I don't understand why all of a sudden it needs me to define an AMI.  The box has an AMI in it that should be enough.
If I manually download the box and manually run vagrant box add, then I can do vagrant up without issue.
Maybe this has to do with the re-ordering of provisioners that happened with 1.7? My best guess is that it is something to do with the box download logic being delayed.
For now, I am just going to vagrant box add myself since everything seems to work fine after that.

I'm experiencing the same problem.
My box is on vagrant cloud/atlas, but the initial download from there fails because no AMI is present in the local Vagrantfile.

Workaround: https://github.com/mitchellh/vagrant-aws/issues/275#issuecomment-111139926

Same problem with vagrant 1.7.4

FWIW I get this now with vagrant 1.7.4 and the most recent aws plugin.  The work around of setting "aws.ami = ''" doesn't seem to work anymore either, giving me a "There was an error talking to AWS. The error message is shown
below:
MissingParameter => The request must contain the parameter ImageId"
running VAGRANT_LOG=debug seems to suggest the AMI is getting set to '' with the workaround:
==> aws: Launching an instance with the following settings...
INFO interface: info:  -- Type: t2.micro
INFO interface: info: ==> aws:  -- Type: t2.micro
==> aws:  -- Type: t2.micro
INFO interface: info:  -- AMI:
INFO interface: info: ==> aws:  -- AMI:
==> aws:  -- AMI:
INFO interface: info:  -- Region: us-east-1
INFO interface: info: ==> aws:  -- Region: us-east-1
maybe should open a new issue...


Don't make OS specific calls in script/setup

I was reviewing the #927 PR and I realize there is a call to brew in the setup script. Since the bootstrap script is OSX specific, it could be nice to avoid os specific call in the setup script and start postgresql in the boostrap script instead of the setup one.

https://github.com/crimethinc/website/blob/master/script/setup#L18
script/setup first calls script/bootstrap, so moving it wouldn't make the script anymore Linux friendly. For bettor and for worse, the scripts are really meant for folx doing development on macOS.
I'm open to a curated set of scripts for Linux folx, if you'd want to do that. I know nothing about Linux development, so I don't know what that would look like.

Ok,
I personally don't use the bootstrap / setup scripts, so I'll check if there's folx doing dev on linux and who think that some linux scripts could be usefull.


Xiaomi Dehumidifier Control

I am using dmaker.derh.22l.
Everything is OK except dehumidification set value.
Even though set value of machine itself is 60 %,  set value appears as 75% in Home app.
If I adjust set value as 60% through Home app, set value of machine adjusted as 54%.
There is difference between Home app set value and machine set value.
Thank you.

Might be that some calculations are being done wrong. Can you maybe please post a homebridge debug log from the setup and when you try to control the value?

I am sorry for my late reply.
I am not familiar with coding.
What I can get from log is below.
[05/08/2022, 23:02:50] [miot] [샤오미 제습기] Device setup finished! Miot device ready!
[05/08/2022, 23:02:50] [miot] [샤오미 제습기] Connected to device: dmaker.derh.22l
[05/08/2022, 23:02:51] [homebridge-miot] This plugin generated a warning from the characteristic 'Relative Humidity Dehumidifier Threshold': characteristic was supplied illegal value: number 0 exceeded minimum of 30. See https://homebridge.io/w/JtMGR for more info.
[05/08/2022, 23:02:51] [homebridge-miot] Error:
at RelativeHumidityDehumidifierThreshold.Characteristic.characteristicWarning (/var/lib/homebridge/node_modules/.pnpm/hap-nodejs@0.10.2/node_modules/hap-nodejs/src/lib/Characteristic.ts:2142:105)
at RelativeHumidityDehumidifierThreshold.Characteristic.validateUserInput (/var/lib/homebridge/node_modules/.pnpm/hap-nodejs@0.10.2/node_modules/hap-nodejs/src/lib/Characteristic.ts:2045:14)
at RelativeHumidityDehumidifierThreshold.Characteristic.updateValue (/var/lib/homebridge/node_modules/.pnpm/hap-nodejs@0.10.2/node_modules/hap-nodejs/src/lib/Characteristic.ts:1457:20)
at DehumidifierAccessory.updateAccessoryStatus (/var/lib/homebridge/node_modules/.pnpm/homebridge-miot@1.2.4/node_modules/homebridge-miot/lib/modules/dehumidifier/DehumidifierAccessory.js:166:132)
at DehumidifierDevice.updateAccessoryStatus (/var/lib/homebridge/node_modules/.pnpm/homebridge-miot@1.2.4/node_modules/homebridge-miot/lib/base/BaseDevice.js:227:54)
at MiotDevice. (/var/lib/homebridge/node_modules/.pnpm/homebridge-miot@1.2.4/node_modules/homebridge-miot/lib/base/BaseDevice.js:66:12)
at MiotDevice.emit (node:events:539:35)
at MiotDevice._deviceFound (/var/lib/homebridge/node_modules/.pnpm/homebridge-miot@1.2.4/node_modules/homebridge-miot/lib/protocol/MiotDevice.js:283:14)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
I am not sure this information is helpful or not.
On the dehumidifier itself, adjustable humidity range is from 40 to 70%.
But, in HomeKit, adjustable humidity range is from 0 to 100%.
I always thank you for your kindness.
Best regards.

Based on the miot spec of your device the target humidity range is 30% to 70%.
Besides you would need to provide some homebridge debug log so i can see what the device is reporting. Check here https://github.com/merdok/homebridge-miot#troubleshooting to see on how to start homerbidge in debug mode.


[helm-chart]  re-add a customizable podSecurityContext

Description
What problem are you trying to solve?
K8S security scanners (like kubescape) check for the presence of container and pod security contexts.
The current helm chart in v0.30.0 specifies a container security context (which is good) but lacks a pod security context (which is bad).
Helm chart should allow adding a fully customized pod security context.
As there have been previous changes to the security context in #4278, #4660, #4681 a new solution should be implemented which allows full customization of both security contexts plus providing safe defaults.
That way the final responsibility goes to the user of the chart for applying a secure config with possibility to adapt it when needed for side-car containers etc.
How important is this feature to you?
Important, as security metrics are used KPI for out DevOps team.

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Version v0.30.0 of the chart is sufficient. I was confused by pre-release versions.


Update DG (Xiaoteng)

Modify the Logic component section
Add implementation details for del-player and move
Slightly modify the diagram of addPlayerSequenceDiagram
Add a use case for moving players

Codecov Report

Merging #240 (977979f) into master (850023d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #240   +/-   ##
=========================================
  Coverage     71.15%   71.15%
  Complexity      927      927
=========================================
  Files           145      145
  Lines          3065     3065
  Branches        366      366
=========================================
  Hits           2181     2181
  Misses          769      769
  Partials        115      115

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[Task] Polishing of JWT token as a feature.

Includes bugs we will fix
knit-picking for cleanup

@RakshaKShetty need a go ahead from QA on this.


spanner-jdbc: Support query hints for DML statements

Update StatementParser to include statement hints support for DML statements.
The Spanner JDBC driver would consider UPDATE statements that started with a statement hint as invalid statements. This change adds a check for statement hints at the beginning of a query, and accepts these as valid queries.
Fixes #1029 ☕

@ktchana LGTM. Please fix the commit comment as per conventionalcommits test.


How to get a player index?

We need something like this:
https://github.com/akiver/cs-demo-analyzer/blob/main/pkg/api/analyzer.go#L307C1-L307C76

Use the entity_id or user_id fields

@LaihoE Actually, I need to get the index of players in demo viewer

JS
let gameEndTick = Math.max(...parseEvent(filePath,"round_end").map(x => x.tick))
let players = parseTicks(filePath, ["user_id", "team_name"], [gameEndTick])
let ctPlayers = players.filter(x => x.team_name == "CT")
let tPlayers = players.filter(x => x.team_name == "TERRORIST")

ctPlayers.sort((a,b) => a.user_id - b.user_id)
tPlayers.sort((a,b) => a.user_id - b.user_id)

slots = []
slots.push(...ctPlayers)
slots.push(...tPlayers)

for (let i = 1; i < slots.length; i++){
    slots[i]["slot_num"] = i
}
console.log(slots)

Python
max_tick = parser.parse_event("round_end")["tick"].max()
df = parser.parse_ticks(["user_id", "team_name"], ticks=[max_tick])

ct_players = df[df["team_name"] == "CT"]
t_players = df[df["team_name"] == "TERRORIST"]

ct_players = ct_players.sort_values(by="user_id")
t_players = t_players.sort_values(by="user_id")

all_players = pd.concat((ct_players, t_players),ignore_index=True)
all_players.insert(0, "slot", range(1, len(all_players) + 1))

print(all_players)

@LaihoE Does it handle player disconnects and player substitution during the match?

just replace the wanted_tick with the tick you are interested in. Also I don't think the slots change during the game.

@LaihoE neither user_id nor entity_id didn't work as a sort property :(

@maxpain Hi, just checked this. Works fine for me with user_id, but not in every demo.
There's typo in JS snippet above
this one works as expected
for (let i = 0; i < slots.length; i++) {
	let current_ind = i + 1;
	slots[i]["slot_num"] = current_ind;
}

The problem with this is that they switch sides throughout the game and in overtimes, but their slots dont get changed.
And for some reason in some demos even when players start as T their slots are counted if they were CT.
So in one demo I tested it works as expected, in other it doesnt.
Not sure what to do about it for now.
I hope that Valve just bring back command to select player with SteamID as in CSGO 😄

@maxpain
I checked quite a few demos, same not consistent result.
I think for now the most reliable way is to get slots from game state integration while playing demo e.g. with allplayers_id component.
https://developer.valvesoftware.com/wiki/Counter-Strike:_Global_Offensive_Game_State_Integration
Result below.
0 here is 1, so have to shift them by 1 and replace 10 by 0 to get needed key.
You can see in this demo they start as T, but get keys as they were CT.
@LaihoE is there no way to get observer slot from demo itself?
{
  "allplayers": {
    "76561198012872053": {
      "name": "huNter",
      "observer_slot": 0,
      "team": "T"
    },
    "76561197999825422": {
      "name": "nexa",
      "observer_slot": 1,
      "team": "T"
    },
    "76561197998926770": {
      "name": "HooXi",
      "observer_slot": 2,
      "team": "T"
    },
    "76561198074762801": {
      "name": "m0NESY",
      "observer_slot": 3,
      "team": "T"
    },
    "76561198041683378": {
      "name": "NiKo",
      "observer_slot": 4,
      "team": "T"
    },
    "76561198246607476": {
      "name": "b1t",
      "observer_slot": 5,
      "team": "CT"
    },
    "76561198176878303": {
      "name": "jL",
      "observer_slot": 6,
      "team": "CT"
    },
    "76561199063068840": {
      "name": "w0nderful",
      "observer_slot": 7,
      "team": "CT"
    },
    "76561198013243326": {
      "name": "Aleksib",
      "observer_slot": 8,
      "team": "CT"
    },
    "76561198050250233": {
      "name": "iM",
      "observer_slot": 9,
      "team": "CT"
    }
  }
}

@ChetdeJong Is it maybe failing in pro games and working in mm? Maybe they do some odd stuff in the pro demo? Anyway I have not seen anything in the demo that would look like "observer_slot". I think this should be possible but I just don't know where to look :(

@maxpain in recent update they added spec_lock_to_accountid and some more spectator commands. Is that what you needed?

To me this seems like we don't need this anymore as you can now search by player name/id? @ChetdeJong @maxpain

To me this seems like we don't need this anymore as you can now search by player name/id? @ChetdeJong @maxpain

ye, that's exactly what I use now, just like in csgo

Closing for now.

@ChetdeJong do you have an example?

im testing with
 let players = parseTicks(demoPath, ["user_id", "team_name"], [tick])
    let ctPlayers = players.filter(x => x.team_name == "CT")
    let tPlayers = players.filter(x => x.team_name == "TERRORIST")

    ctPlayers.sort((a,b) => a.user_id - b.user_id)
    tPlayers.sort((a,b) => a.user_id - b.user_id)

    let slots = []
    slots.push(...ctPlayers)
    slots.push(...tPlayers)

    for (let i = 1; i < slots.length; i++){
        slots[i]["slot_num"] = i
    }

    const slot = slots.find((slot) => {
        return slot.name === player;
    });

@lukepolo
Hi, what's your end goal?
Do you need exactly spectator (observer) key or some id in general to switch to needed player?
Last time I checked the method that was discussed here didnt work consistently, so you can't get spectator key from it. If your goal is just to switch player you can use spec_lock_to_accountid with steamid or you can use spec_player with userid + 1 e.g. userid is 6, so you do spec_player 7 to switch to that player. Or you can use GSI if that works for you

spec_lock_to_accountid Seems perfect ,I’ll give it a go . I’ll share my script here soon , using this to generate  a mp4 of all their kills from a single round

yeah unfortunately this doesn't work , at least on linux

ill try that later today, going to leave this here too
https://gist.github.com/lukepolo/dac28b1d323a42d885092f8b51187cd7


Install php_pgsql

I need use php_pgsql.
Is possible install on asdf installation?

Doesn't this work?
PHP_CONFIGURE_OPTIONS="--with-pgsql" asdf install php <VERSION>

or
PHP_CONFIGURE_OPTIONS="--with-pgsql=<PATH TO PSQL>" asdf install php <VERSION>

The installation dont finished.

Please tell me the environmental information such as OS, its version, etc...

If you are using macOS, have you done all these steps?
brew install autoconf automake bison freetype gettext icu4c krb5 libedit libiconv libjpeg libpng libxml2 libzip pkg-config re2c zlib

export PKG_CONFIG_PATH="$(brew --prefix icu4c)/lib/pkgconfig:$(brew --prefix krb5)/lib/pkgconfig:$(brew --prefix libedit)/lib/pkgconfig:$(brew --prefix libxml2)/lib/pkgconfig:$(brew --prefix openssl)/lib/pkgconfig"

export PATH="$(brew --prefix bison)/bin:$PATH"

PHP_CONFIGURE_OPTIONS="--with-pgsql" asdf install php 7.4.12

Probably because openssl is not found, if the following log appears during configure:
checking for OpenSSL support... no

For example, I use Void Linux, so I had to specify it like this:
PHP_CONFIGURE_OPTIONS="--with-pgsql --with-openssl=/usr/include/openssl" asdf install php 7.4.12

Hi, i use arch linux

Probably because openssl is not found, if the following log appears during configure:
checking for OpenSSL support... no

For example, I use Void Linux, so I had to specify it like this:
PHP_CONFIGURE_OPTIONS="--with-pgsql --with-openssl=/usr/include/openssl" asdf install php 7.4.12

I will try --with-openssl

PHP_CONFIGURE_OPTIONS="--with-pgsql --with-openssl=/usr/include/openssl" asdf install php 7.4.12
It's work. Thank you very much.

I'm glad it works. Your question seems to have been resolved, so I will close this issue.


suspected issue with downloading inlets-pro on GCP

Expected Behaviour

Expect created server to run inlets(-pro) and output commands to result in a connected tunnel
Current Behaviour

Client shows:
$ ./inlets-pro client --connect "wss://$EXIT_IP:8123/connect" --token "$TOKEN" --license "$PRO_LICENSE" --tcp-ports $TCP_PORTS
2020/02/08 23:45:17 Welcome to inlets-pro!
2020/02/08 23:45:17 Starting client - version 0.5.3
2020/02/08 23:45:17 Licensed to<EMAIL_ADDRESS>expires: 102 day(s)
2020/02/08 23:45:17 TCP Ports: [3306]
Error: unable to download CA from remote inlets server for auto-tls: Get https://<IP_ADDRESS>:8123/.well-known/ca.crt: dial tcp <IP_ADDRESS>:8123: connect: connection refused

Logged in to the created server. This is what I found:
rheutan7@sharp-nightingale4:~$ cat /etc/default/inlets-pro
AUTHTOKEN=tjqMrBbreSA9141nI7KMOVOhJxvJTbXISdHGen6yWEZxBhXe7L4nMyDX4U4hNghw
REMOTETCP=<IP_ADDRESS>
IP=<IP_ADDRESS>
rheutan7@sharp-nightingale4:~$ cat /etc/systemd/system/inlets-pro.service
[Unit]
Description=inlets-pro Server Service
After=network.target

[Service]
Type=simple
Restart=always
RestartSec=2
StartLimitInterval=0
EnvironmentFile=/etc/default/inlets-pro
ExecStart=/usr/local/bin/inlets-pro server --auto-tls --common-name="${IP}"  --remote-tcp="${REMOTETCP}" --token="${AUTHTOKEN}"

[Install]
WantedBy=multi-user.target
rheutan7@sharp-nightingale4:~$ sudo systemctl status inlets-pro
● inlets-pro.service - inlets-pro Server Service
   Loaded: loaded (/etc/systemd/system/inlets-pro.service; enabled; vendor preset: enabled)
   Active: activating (auto-restart) (Result: exit-code) since Sun 2020-02-09 05:48:55 UTC; 1s ago
  Process: 1264 ExecStart=/usr/local/bin/inlets-pro server --auto-tls --common-name=${IP} --remote-tcp=${REMOTETCP} --token=${AUTHTOKEN} (code=exited, status=203/EXEC)
 Main PID: 1264 (code=exited, status=203/EXEC)

Feb 09 05:48:55 sharp-nightingale4 systemd[1]: inlets-pro.service: Main process exited, code=exited, status=203/EXEC
Feb 09 05:48:55 sharp-nightingale4 systemd[1]: inlets-pro.service: Unit entered failed state.
Feb 09 05:48:55 sharp-nightingale4 systemd[1]: inlets-pro.service: Failed with result 'exit-code'.
rheutan7@sharp-nightingale4:~$ which inlets-pro
rheutan7@sharp-nightingale4:~$ inlets-pro
-bash: inlets-pro: command not found
rheutan7@sharp-nightingale4:~$ ls /usr/local/bin/
rheutan7@sharp-nightingale4:~$

Possible Solution

Perhaps a verification script that ensures the binary is installed, and the inlets-pro service is running before exiting to know that there was an error.
The process should clean up after itself by deleting everything that was created if there is a failure
Steps to Reproduce (for bugs)

Run inletsctl create --provider gce and all the flags necessary
Run the output commands to connect with a client
See the failed connection
Log in to the created server and view the inlets-pro service logs: sudo systemctl status inlets-pro

Context

Unable to get inlets-pro to work with Google Cloud where most of my applications run
Your Environment

inlets version inlets --version

Version: 0.4.6
Git Commit: a03f5e2b9f7a1968795739ec39eaab99c0680447

Docker/Kubernetes version docker version / kubectl version:
N/A
Operating System and version (e.g. Linux, Windows, MacOS):
Linux
Link to your project or a code example to reproduce issue:
N/A

Hi @burtonr can you look at the cloud init log in /var/log/ (the log with a longer name suffix)
@utsavanand2 or @adamjohnson01 might be best looking at this.
It would appear the download script perhaps failed or was interrupted since the other files are in place.
Can you try again later and share the log please?
Alex

/set title: suspected issue with downloading inlets-pro on GCP

@burtonr @alexellis I'm getting on to this! One thing to note though I don't think this is related to the issue is opening up of ports. Which I'll put up as soon as I can today! I'll see if it fixes it. If not I'll see what I can do about it 🤞

Confirmed this is now working as expected as of version 0.4.8 from PR https://github.com/inlets/inletsctl/pull/58

Started seeing this from local k3s
unable to download CA from remote inlets server for auto-tls: Get "https://<IP_ADDRESS>:8123/.well-known/ca.crt"
What is this and why can't I configure the ca myself?

Hi @jjaybrown
Happy to help, please can you raise your issue?
Alex

/lock: resolved


Change Secret and ProviderSecret to send .to(process).

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1491  👈 (View in Graphite)
#1480
#1475
main

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

Merge activity

Dec 2, 8:41 PM EST: A user started a stack merge that includes this pull request via Graphite.


Move to using API v1

Morph 0.11.0 has a much nicer API, let's use it.

OpenSSL config dances should be removed here, btw, they're no longer needed.


on Windows, strip the '\?' prefix from the canonical path

This should help with https://github.com/rust-lang/rust/pull/74146#issuecomment-655723028

@bors r+

:pushpin: Commit 7d9d74e06505b48d2894a24ccbe716ffa931bd2b has been approved by RalfJung

:hourglass: Testing commit 7d9d74e06505b48d2894a24ccbe716ffa931bd2b with merge b245786f9d3466bc9ae66f894cb5288e32eaac07...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing b245786f9d3466bc9ae66f894cb5288e32eaac07 to master...


Fix nav alignment, about section contrast and layout issue. Fixes #59

Description:
This PR addresses the following issues as outlined in Fixes #59:

Navigation Elements:

Fixed alignment issues by adjusting the spacing between nav elements.
Removed the fixed width, allowing for better responsiveness across different screen sizes.
Updated the design of nav elements to have rounded corners by default, rather than only on hover, improving the visual appeal.

About Section:

Enhanced text readability by improving the contrast between the background color and the text color.
Adjusted the alignment of text and images for a more balanced layout.

Changes Made:

Improved the layout and spacing of nav elements to ensure proper alignment on different screen sizes.

Updated CSS to make nav elements rounded by default.

Fixed contrast issues in the About section to enhance readability.

Realigned text and images in the About section to ensure consistent visual flow.

Before:

After:

Related Issue:
This PR fixes #59.

@Mohitranag18 please resolve the conflicts.

ok sir

@Mohitranag18 check the deployment link, the courses section has changed it's not looking good.

here is that it is looking in the website.

I think that is course page I worked on home page

@Mohitranag18 All the pages have the same css code, you made the changes to css code, which caused the problem.
Pls check and make the changes.

@Mohitranag18 you made the changes in style.css, i think some component was linked to the courses page.

Sir there is nothing similar element in home page and course page except header, how can my changes reflect to course change.

@Mohitranag18 okay, Please resolve the conflicts and check the deployment.

conflicts are resolved sir, please review now


WIP: adding queryset_transform kwarg to ModelSelect2Mixin

fixes #510
I did not think of the issue that assigning a callable to a class attribute will make it a bound method, which is annoying as it requires a 'self' argument as first argument.
When writing this: It might also be desirable. It allows access to the widget, so the callable must take 3 arguments: widget, queryset, term.

Apparently my memory identified this callable-assigment as a problem, while in reality it was not. Cleaned up the implementation a bit and added a test.

Codecov Report

Merging #512 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #512   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           7      7
  Lines         238    243    +5
=====================================
+ Hits          238    243    +5

Impacted Files
Coverage Δ

django_select2/forms.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@jieter thanks for the example, I think now I understand your use case better.
I am wondering tho, why don't you just pass the modified queryset to begin with? Like so:
Select2ModelWidget(queryset=MyMode.objects.all().annotate(sort_order=F('title')).order_by('-sort_order'))

This should have the same effect, should it not?

Wouldn't that be equivalent to .order_by('-title') ? I think I need the actual search term to prefer records starting with it, rather than just alphabetical ordering on the title.

I think I can't follow you here. There is no difference when you specify the order for a Queryset, is there?
Why does it matter if you apply the search parameters before or after? If you lock at the actual query, there shoudn't be a difference.

class MyForm(forms.ModelForm):
    location = forms.ModelChoiceField(
        queryset=Location.objects.all(),
        required=False,
        widget=ModelSelect2Widget(
            queryset=Location.objects.all().annotate(
                sort_order=Case(
                    When(name__istartswith=term, then=Value(1)), default=Value(0), output_field=IntegerField()
                    ),
                ).order_by("-sort_order", "name")
            ),
            search_fields=['name__icontains'],
    )

like so?

term is not defined at field definition time.

Aaaah ok, now I see what you mean.
Please don't take this the wrong way, but this is a highly specific case. I believe writing your own subclass and overwriting filter_queryset is the best way to go.
Adding support for that would further drive complexity of the code base, which I want to keep as low as possible to allow smooth maintenance thought all Django versions.
Of course I am open to reconsider, but maybe you agree with my point.

Yes, I see, might be more trouble than it's worth. No problem!

If this is a common use case for you. Just build your own mixin. You already provided the right code for it in this PR. This should allow you to simply "configure" the callable, if this is what you need.


Implement correct array/scalar equality

Right now I'm adding a derived PartialEq, which just checks that the constituent parts are the same, but we need to check that the equality on sliced arrays works as expected. If it just checks equality on the underlying buffers, then array equality will not be true whenever comparing a sliced array to a non-sliced array.

In any case, comparisons will fail whenever checking an interleaved coord array to a separated coord array.


Run TCP integration tests with grunt-jasmine-firefoxaddon

Passes locally, runs on Shippable but times out on serves multiple clients and wrapper connectionsCount. Will investigate/poke to see if it resolves, but wanted to open pull request for consideration. Thanks!

Our Shippable setup has been busted for some time, unfortunately, so I'm not too worried about that. Many many thanks for sending this, it looks great to me. :+1:

Related issue:
https://github.com/uProxy/uproxy/issues/840


[0.5xpt] Add Game class

It should contain the following:

multiplayer as a boolean?
last_payed_at as a date

should override the method from the parent class
should return true if parent's method returns true OR if cover_state equals to "bad"
otherwise, it should return false


Supply - AndroidPublisherV3 - Rollout getting halted

Currently working on migrating supply to androidpublisherv3 - https://github.com/scorpion35/fastlane/blob/androidpublisherv3/supply/androidpublisherv3_tracking.md
Hit a road block trying to update the rollout of a version 😬
Code:
https://github.com/scorpion35/fastlane/commit/9a7f12cda9e02104c99c446c8e0b764a53d8dc1a
Use case & test results:
https://github.com/scorpion35/fastlane/commit/9a7f12cda9e02104c99c446c8e0b764a53d8dc1a#commitcomment-34808835
Here's the object before uploading it -

Does anyone know how to increment the rollout, please? Many thanks in advance!

You need to send "status": "inProgress" in the release object of the request body. I also think that the send("track=", 'rollout') isn't a thing anymore because both the beta and the production track can have rollouts, so that should be the actual track where you perform the rollout.
For the status thing see releases[].status her:
https://developers.google.com/android-publisher/api-ref/edits/tracks

Ah right! That's what I was thinking too and added it to my notes - https://github.com/scorpion35/fastlane/blob/9a7f12cda9e02104c99c446c8e0b764a53d8dc1a/supply/androidpublisherv3_tracking.md (but removed it because I wasn't 100%)
I have tried track = 'production' prior to setting it to rollout, and that doesn't seem to do anything at all (i.e. it didn't change any thing for the release on google play console). Let me try a couple of more tests.
Thank you very much for fast reply :)

I give up 😞 Tried many different combinations -

user_fraction = 0.51, status='inProgress', track = 'production', preserved existing release notes
user_fraction = 0.51, status='inProgress', track = 'production', removed existing release notes and only tried to update with user_fraction
user_fraction = 0.51, status='inProgress', track = 'rollout', preserved existing release notes
user_fraction = 0.51, status='inProgress', track = 'rollout', removed existing release notes and only tried to update with user_fraction

One thing I noticed is something seems to be invalidating the track and it makes sense for Google to not apply an invalid object to the track.

(^ this track is supposed to have much more information, but when I try to fetch the track information after updating the track, I am getting an empty)
Does anyone know how to properly update a track (rollout or update the release notes for a version that is already in progress, etc.)?
Thanks in advance for your help!!

Do you set the track in the request body?

In Google's sample they don't actually set there when updating the rollout percentage (you can see it in the part that @Xexiz quoted). Maybe that is the issue.


Move from namespace Tensile to TensileLite

Use TensileLite to disambiguate from Tensile project when building statically.

Should we namespace this file:
hipBLASLt/library/src/amd_detail/hipblaslt-ext-op-internal.hpp
e.g. amd_detail?

Should we namespace this file:
hipBLASLt/library/src/amd_detail/hipblaslt-ext-op-internal.hpp
e.g. amd_detail?

Suggest: hipblaslt , TensileLite, or anonymous
Should indicate ownership, look at other math libraries which do use namespaces. rocSPARSE I think does

@jichangjichang @kkHuang-amd need approval for this. Please let me know if you are aware of any downstream projects that will be impacted by this change.

@ellosel assume you built rocblas against this PR without issue.   Frameworks you could ping but if these weren't in the public API no one can really complain too much.

@ellosel assume you built rocblas against this PR without issue. Frameworks you could ping but if these weren't in the public API no one can really complain too much.

Correct - that is how I found the duplicate symbols.


Receive an unexpected event instant time[SUPPORT]

Tips before filing an issue

Have you gone through our FAQs?

Join the mailing list to engage in conversations and get faster support at<EMAIL_ADDRESS>

If you have triaged this as a bug, then file an issue directly.

Describe the problem you faced
A clear and concise description of the problem.
To Reproduce
Steps to reproduce the behavior:

use flink to write hudi table

The exception continue to occur

checkpoint
the checkpoint is not always failed, theck checkpoint can success

Expected behavior
Flink job can run normally
Environment Description

Hudi version : 0.10

Flink version : 1.13.1

Hive version : 3.1.2-amzn-4

Hadoop version : 3.2.1-amzn-3

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) : no

Do you have multiple writers writing into the same table at the same time ?

Do you have multiple writers writing into the same table at the same time ?

No, Only one writer

I find the data can be writed,when the checkpoint is success,but after running some time , the data will depulicate

your checkpoint size is huge, did you use the rocks-db statebackend instead ? And the 0.11.1 supports bucket index, you can try that.

your checkpoint size is huge, did you use the rocks-db statebackend instead ? And the 0.11.1 supports bucket index, you can try that.
I use the rocks-db statebackend, bucket index it only can reduce the checkpoint size ?

your checkpoint size is huge, did you use the rocks-db statebackend instead ? And the 0.11.1 supports bucket index, you can try that.
I use the rocks-db statebackend, bucket index it only can reduce the checkpoint size ?

Did you open incremental checkpoint, the bucket index does not use any state-backend at all.

your checkpoint size is huge, did you use the rocks-db statebackend instead ? And the 0.11.1 supports bucket index, you can try that.
I use the rocks-db statebackend, bucket index it only can reduce the checkpoint size ?

Did you open incremental checkpoint, the bucket index does not use any state-backend at all.

No, I write a lot table to hudi, there are ok , but  I don't know the reason why it's broken

The exception usually happens when it receives an event that does not belong to the current checkpoint, for example, multiple writers write into the table at the same time.

The exception usually happens when it receives an event that does not belong to the current checkpoint, for example, multiple writers write into the table at the same time.
Okay, I will check it , you can close the issue


HTML

HTML
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow
WHEN I open the console in my browser
THEN I can see the four topics I learned listed along with a suggestion on what I should study first
WHEN I visit the website using the URL
THEN I can access my website from any browser

Can you elaborate what you want to write in html file?
Assign this issue to me @axeljamal

Can you elaborate what you want to write in html file? Assign this issue to me @axeljamal

Thanks for checking. Have I done it right on my update?

Can I ask you what you actually want on your html page?


Feature Request: Prevent score regression on publishing

Thanks for building jsr, I loved the score feature.
I wonder if the CLI could prevent score regressions, or prevent publishing if certain requirements are not met.
For instance, I have a published package where all symbols have documentation and all entry points have documentation. I would like it to continue to have the docs for all symbols, but I can see myself forgetting to add them.
So, I'm proposing the CLI add a mechanism to either prevent score decrease or a --require-docs-percentage 100 parameter. This would work similarly to test coverage tools.

Related: https://github.com/jsr-io/jsr/issues/199


Feature Request: Support for Restart sampling

Please consider adding support for the Restart sampler. It's a new sampler that promises to be both faster and produce better quality images than the existing ones. It would be great to have it available in SHARK too!
https://github.com/newbeeer/diffusion_restart_sampling

it could be added once you find it here: https://huggingface.co/docs/diffusers/main/en/api/schedulers/overview


[Expreimental] Repeat OOPIF tests 300 times each

I am curious about the purpose of this CL. You could also use https://source.chromium.org/chromium/chromium/src/+/main:tools/flakiness/is_flaky.py to deflake.


Flatten ItemKind into an Item enum

Side note: this pattern is sort of nice:
let item: &Item = /* ... */;

match *item {
    Item::Mod(ref item) => {
        do_something_with(&item.ident, &item.content);
    }
    Item::Fn(ref item) => {
        do_something_with(&item.abi, &item.block);
    }
    /* ... */
}

Basically pattern matching on an item and rebinding the same name item just takes your variable and imbues it with those additional fields that exist on the type of item that it happens to be.

@mystor syn_codegen is a godsend, thanks again.

This is pretty nice :-). Can we also do this for Expr, Lit, etc? It would be nice to get rid of more of the *Kind types.

I will have a go at ForeignItemKind, TraitItemKind, ImplItemKind. We need a bit more design work on Lit, the current structure is strange. For Expr I think we can flatten Expr into an enum but we may not be able to expose individual parsers like syn!(ExprBinary) because of how precedence is complicated. I think that is fine, I bet fewer people want to parse a particular kind of Expr than a particular kind of Item.


pdngen missing repair strap

Describe the bug
With the update to hier-rtlmp & cts a problem has developed in ng45/bp_be_top where psm errors on a disconnected power grid.
[WARNING PSM-0038] Unconnected PDN node on net VDD at location (10.070um, 505.400um), layer: 1.
[WARNING PSM-0039] Unconnected instance PHY_2260 at location (10.070um, 505.400um) layer: 1.
[WARNING PSM-0039] Unconnected instance PHY_2250 at location (10.070um, 505.400um) layer: 1.
...

It appears that pdn is not putting in a repair strap in a channel on the left:

You can see a repair strap in the lower portion of the channel but not in the upper portion.
I have temporarily disabled psm on this design.  When this is fixed it should be re-enabled.
Expected Behavior
A fully connected power grid.
Environment
head of master

To Reproduce
DESIGN_CONFIG=./designs/nangate45/bp_be_top/config.mk make
Relevant log output
No response
Screenshots
No response
Additional Context
No response

@maliberty I ran the command, but my output looks mirrored from the picture you uploaded. And I haven't been able to see the issue yet on it.

Maybe something else changed - I'll rerun it and attach a test case if I can reproduce it.

Something has changed and the problem disappeared due to change in macro locations.  That's teach me to not record the commit id.  I'll close this and reopen if the problem returns.


devnet returns HTTP-500 when there is no param in the RPC call

When running Caigo with devnet with calls that do not have parameters, like starknet_protocolVersion, the JSON RPC implementation of go-ethereum, send the following payload:
{
    "id": 1,
    "jsonrpc": "2.0",
    "method": "starknet_protocolVersion"
}

That makes sense since there is no parameters, however, devnet and opposite to pathfinder fails with HTTP-500. We need to get the following payload:
{
    "id": 1,
    "jsonrpc": "2.0",
    "method": "starknet_protocolVersion",
    "params": []
}

It looks like some methods like starknet_syncing are not implemented either. I have workaround the issue which I think is an issue with devnet rather than us by adding []interface{}{}... in the RPC call which has as an effect to add the "params": [] in the RPC call. It would have to ve added to every single call.

The fix is included in #65 and will be covered by the addition of tests on devnet as described in #54. I will close that issue.


System.IO.Compression.Tests fail when built in Visual Studio

Tests skipped in https://github.com/dotnet/corefxlab/pull/1610
This likely has to do with the mismatch between the required and actual native library being used. We need to explicitly pick x86 or x64 and VS defaults to "ANY CPU"
Example failure:
Test Name:	System.IO.Compression.Tests.BrotliStreamTests.RoundtripCompressDecompress(chunkSize: 1, totalSize: 5)
Test FullName:	System.IO.Compression.Tests.BrotliStreamTests.RoundtripCompressDecompress (926e3b9e09be8890c746d3823888e4e9c039f226)
Test Source:	D:\GitHub\Fork\corefxlab\tests\System.IO.Compression.Tests\BrotliStreamTests.cs : line 200
Test Outcome:	Failed
Test Duration:	0:00:00.002
Result StackTrace:
at System.IO.Compression.Interop.Brotli.BrotliEncoderCreateInstance(IntPtr allocFunc, IntPtr freeFunc, IntPtr opaque)
at System.IO.Compression.BrotliNative.BrotliEncoderCreateInstance()
at System.IO.Compression.Encoder.InitializeEncoder()
at System.IO.Compression.Encoder..ctor()
at System.IO.Compression.BrotliStream..ctor(Stream baseStream, CompressionMode mode, Boolean leaveOpen, Int32 BuffSize)
at System.IO.Compression.Tests.BrotliStreamTests.RoundtripCompressDecompress(Int32 chunkSize, Int32 totalSize)
Result Message:	System.BadImageFormatException : An attempt was made to load a program with an incorrect format. (Exception from HRESULT: 0x8007000B)

Resolved in this pr https://github.com/dotnet/corefxlab/pull/1656.
Also VS team said that you may need to have VS Version 15.3.0 Preview 3.0 [26630.3002.d15prerel] to don't get illegal character error.