Certain channels do not resolve

This may be a YouTube limitation but reporting here in case it's otherwise. If you try getChannel() with this URL it will fail with "Item not found" error:
https://www.youtube.com/c/ChilledCow
but the same URL with a trailing slash will work just fine:
https://www.youtube.com/c/ChilledCow/
Using just "ChilledCow" also works. However, other channels do not work even with trailing slash or with only the end path.
const { YouTube } = require('popyt');
const youtube = new YouTube(YOUTUBE_KEY);

/*
Working:
'https://www.youtube.com/c/ChilledCow/'
'ChilledCow'

"Item not found":
'https://www.youtube.com/c/ChilledCow'
'https://www.youtube.com/c/fosterkittensofficial/'
'https://www.youtube.com/c/fosterkittensofficial'
'fosterkittensofficial'
*/

const query = 'https://www.youtube.com/c/ChilledCow/';

youtube
  .getChannel(query)
  .then(res => console.log(res.data.snippet))
  .catch(err => console.error(err));

When I load those failing URLs in the browser, they load fine and I can't discern any particular difference between the working and non-working URLs. Thank you for the library! 🙇🏼♂

The problem with the trailing slash was with my code. I have since fixed that, but I cannot push it until I finish my current YouTube API compliance review.
The problem with certain channels not working at all is due to a limitation with the YouTube API. There is no way to get a channel from its custom URL, so popyt sort of mimics this by grabbing the end of the URL and searching for it. In the case of "fosterkittensofficial", their actual username is "Cindy Congdon", so "fosterkittensofficial" returns no results (and if it did, they would most likely be garbage). I have also edited the documentation of the getChannel() function to clarify this.
Once my compliance review is complete, I will push these new changes and close this issue. Thanks!

No worries and no rush! Glad a cause could be found.
Yeah for urls where there's no id or username, I just grabbed the html as is then regexed the id:
/(?<="channelId":")[A-Za-z0-9]{24}/g
/(?<="externalId":")[A-Za-z0-9]{24}/g
/(?<="browseId":")[A-Za-z0-9]{24}/g

Wasn't sure if channelId was always present so the other two are just fallbacks.

I would add this, but the wrapper is purely meant to use the YouTube Data
v3 API. The frontend can change unexpectedly, so scraping the website isn't
an option.
A good solution for anyone that really wants to be able to get a channel by
its custom URL, though.
On Wed, Feb 17, 2021 at 11:51 PM Ron Au<EMAIL_ADDRESS>wrote:

No worries and no rush! Glad a cause could be found.
Yeah for urls where there's no id or username, I just grabbed the html as
is then regexed the id:
/(?<="channelId":")[A-Za-z0-9]{24}/g
/(?<="externalId":")[A-Za-z0-9]{24}/g/(?<="browseId":")[A-Za-z0-9]{24}/g
Wasn't sure if channelId was always present so the other two are just
fallbacks.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/jasonhaxstuff/popyt/issues/297#issuecomment-781048396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRI62R7B7SDW5RR2J7LPJDS7SMEZANCNFSM4VTH2FSQ
.

Oh 100% agreed, just an account of how one person ended up resolving it.


Corrupted json generated under load (?)

We have an issue when generated payload json is rejected by seq due to two-characters missing from it. Basically it looks same as #28 issue.
It could happen like xx times a day or not happen for weeks. As it usually happens when server struggling under high load I think it could be some races in code, but I wasn't able to find anything suspicious in library code so far.
Library version is 6.0.0
Seq logs:
[2024-02-17T03:04:06.3843507+00:00 DBG] Rejecting CLEF payload from <client_ip> due to invalid JSON, item could not be parsed;

Application logs:
Logger crashed with 2024-02-17T07:03:52.7761603Z HTTP shipping failed with BadRequest; dumping payload to <path-to.json>

Thanks for the heads-up; are you also using the bufferBaseFilename option as in the other ticket? Could you please share some additional details of your Serilog.Sinks.Seq setup - options used, operating system, .NET version? Thanks!

var levelSwitch = new LoggingLevelSwitch()
{
	MinimumLevel = LogEventLevel.Verbose
};

var loggerConfig = new LoggerConfiguration().MinimumLevel.ControlledBy(levelSwitch);
// + bunch of property enrichers

var logger = loggerConfig
	.WriteTo.Seq(
	seqUrl,
	//apiKey: settings.SeqApiKey,
	controlLevelSwitch: levelSwitch,
	bufferBaseFilename: buffer)
	.CreateLogger();

// Microsoft.Extensions.Logging factory
var factory = LoggerFactory.Create(builder => builder.AddSerilog(logger, dispose: true));

Currently trying to reproduce issue in sample console app without success yet.
But was able to reproduce another issue we have sometimes: when I use large messages (but smaller than default limit of ~261k), provider often tries to send batches with size > 10Mb (up to ~100Mb) failing with RequestEntityTooLarge error from server. Errors goes away after messages sizes reduced to some point.
RequestEntityTooLarge error repro. Here with 2_000 array elements (arrays used tp have high density of JSON syntax in message to detect original issue) I observe packets of 14Mb with max message size in bytes ~23Kb.
With bigger arrays packet size increases.
var log1 = factory.CreateLogger("Test1");
var log2 = factory.CreateLogger("Test 2");
var log3 = factory.CreateLogger("Tes3");

var size = 2_000;
var data1 = new int[size];
var data2 = new bool[size];
var data3 = new TimeSpan[size];
for (var i = 0; i < data1.Length; i++)
	data1[i] = i;
for (var i = 0; i < data2.Length; i++)
	data2[i] = i % 2 == 0 ? true : false;
for (var i = 0; i < data3.Length; i++)
	data3[i] = TimeSpan.FromMinutes(i);

var size1 = Encoding.UTF8.GetBytes(JsonSerializer.Serialize(data1)).Length;
var size2 = Encoding.UTF8.GetBytes(JsonSerializer.Serialize(data2)).Length;
// ~22Kb
var size3 = Encoding.UTF8.GetBytes(JsonSerializer.Serialize(data3)).Length;

for (var i = 0; i < 50; i++)
{
	Task.Run(() =>
	{
		var cnt = 100_000;
		while (cnt-- > 0)
		{
			log1.LogError("Test:{data1}", data1);
			log2.LogWarning("Test: {data2}", data2);
			log3.LogInformation("T est: {data3}", data3);
		}
	});
}

await Task.Delay(TimeSpan.FromHours(1));

Thanks for the follow-up. Unless you're working with occasionally-connected machines, I'd strongly recommend using direct delivery instead of local buffering.
Because "durable" mode uses a simple local text file instead of (e.g.) a database engine, it doesn't have the degree of robustness required for low-maintenance/high-throughput logging.
Without the buffer file, the sink still internally uses a memory buffer to smooth out occasional periods of network or server unavailability (up to around ten minutes), so it's rare to lose logs just because of say, a server restart.
If you're concerned about having a fail-safe copy of the data anyway, writing the logs to a rolling set of newline-delimited JSON files locally should provide that.
Let me know if this helps, or if you would like to dig into this scenario further.

That's clear. We prefer to have buffering on even with such errors as they really rare.  E.g. yesterday we discovered one of processes collecting logs to files but for some reason not sending them to Seq for days (cannot say why as I didn't looked into it, could be our issue) so buffering helped us to preserve those logs and send them to Seq after application restart.
Let's close it for now. I will look into it later (your words gave me idea to test application/seq restart scenarios) and if it will rise some results - will reopen it. Still it would be nice to have fix for second issue with oversized requests as it is easy to reproduce.


Stop resolving advertise-client-urls

We're not validating acurls right now. And, resolving acurls may lead TLS verification failure, by having different hostname.

@sorah Hrm... I think the peer url will have the same tls problems, right? I would like to solve this problem completely.

apurls may be. Let me test...

I couldn't encounter problem on adding/removing members for apurls.

@kelseyhightower This is also a ssl related issue. Can you take care of it?

@sorah Sorry for the delay. I am trying to fix the root problem. Do you think https://github.com/coreos/etcd/pull/2603 help to fix your issues?

:eyes: > 2603

Solved by #2603


[Bug]  Flink CDC Orcale  Connection Pool Number settings cannot take effect

Search before asking

[X] I searched in the issues and found nothing similar.

Flink version
Flink 1.6 version
Flink CDC version
Flink 2.3 version
Database and its version
Orcale 19c
Minimal reproduce step
connection.pool.size=50, but it cannot take effect, orcale session show 500 connections
What did you expect to see?
Fix the problem
What did you see instead?
I dont know
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

OK

Closing this issue as it has been migrated to Apache Jira.


Pathfinder doesn't avoid lava

Seems to me like the pathfinder gladly walks into lava as if it were ignoring it.
To recreate this, grab the basic GPS example from mineflayer, create a small lava lake, stand on the other side, and say come.
Pathfinder walks straight through the lava lake rather than around.

https://github.com/PrismarineJS/mineflayer-pathfinder/issues/129#issuecomment-793027102

u mean movements.blocksToAvoid.add(BLOCK_ID)

That could be a bug with the physic simulation. The path plan to jump over lava but the simulation find that it can swim and reach the other side. Could be useful to visualize with pviewer what the bot is thinking. Thanks for reporting.

That seems consistent with what I'm seeing. There are some situations where it does try to avoid lava fine, such as flowing lava above-ground.
On a related note though, it will still accidentally touch the corner of flowing lava or fire since it's cutting the corner. To fix this the bot would need to take corners with lava/fire significantly wider than regular corners.

#149 Not a perfect fix, but it shows where the problem is. It's not with pathfinding but with completing that path. With that fix it(almost always) jumps over lava correctly

There are some movements generated that violate the rules because they don't check all the blocks the bot will occupy. For example when a bot moves diagonally and down one block it does not check the final block for safety.
Here is a video demo: https://youtu.be/0xLNvio0cNA
Here is the line for that specific issue. https://github.com/PrismarineJS/mineflayer-pathfinder/blob/c267bb2e1079fb515b886e3951f7e19230a0b64b/lib/movements.js#L285


translation bugs on "Documents" page

http://<IP_ADDRESS>:8000/workflow/documentation_list/0/0/
Column names and pagination at bottom of columns have not been translated

I translated the word "Actions" myself. Pagination elements are from the DataTables JS library & not trivially translatable.

So the other column titles: Date Created, Document Name, Project and Actions are not easily translatable? Is that correct?

Sorry, I wasn't clear on this. Column titles ARE easy to translate, and should be translated now. It's just the pagination controls ("showing X of Y entries / Previous / Next") that's hard to get to

Other than the actual TRANSLATION this is fixed


docs: Fix markup in command_line.rst

Also remove $ since it was it wasnt used in luigi_patterns.rst

I can't imagine anyone -1ing this


Events page has become stuck!

@Avinash905 @FuryACE007 @vinitvijal @ng169 , Please assign this issue to me as a part of @GSSOC'23. I would love to contribute.
Thanks!

@Sathvik902 It has been resolved now. Thank you!!


Add Feed The Beast.app (latest)

This should be fixed now, thanks!

Looks good, thank you. Could you please squash your commits?

Done! :wink:

Thank you. Merged.


”Giurgiș”, ”Grindeni”, ”Hădăreni”, ”Linț” are listed in the drop-down menu

Description:
When user writes ”Giurgiș”, ”Grindeni”, ”Hădăreni”, ”Linț” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Giurgiș"

Expected results
The location appears in the drop-down menu as "Sat Giurgiș, MUREȘ (Chețani)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Grindeni"

Expected results
The location appears in the drop-down menu as "Sat Grindeni, MUREȘ (Chețani)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Linț"

Expected results
The location appears in the drop-down menu as "Sat Linț, MUREȘ (Chețani)".


[GOBBLIN-1596] Ignore already exists exception if the table has already been created…

… by another thread or job entirely
Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1596

Description

[x] Here are some details about my PR, including screenshots (if applicable):
Hive Registration throws an error when creating a table to add partitions to, if the table already exists. Although we should have locks to prevent this, we can't guard against other jobs that could also cause a race condition too. We can ignore the exception in the error handling for ensureHiveTableExistenceBeforeAlternation

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #3451 (14767b1) into master (26f6930) will decrease coverage by 3.14%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3451      +/-   ##
============================================
- Coverage     46.61%   43.47%   -3.15%
+ Complexity    10342     2031    -8311
============================================
  Files          2074      404    -1670
  Lines         80956    17382   -63574
  Branches       9030     2124    -6906
============================================
- Hits          37740     7557   -30183
+ Misses        39741     8988   -30753
+ Partials       3475      837    -2638

Impacted Files
Coverage Δ

...a/org/apache/gobblin/util/limiter/NoopLimiter.java
40.00% <0.00%> (-20.00%)
:arrow_down:

...lin/util/filesystem/FileSystemInstrumentation.java
92.85% <0.00%> (-7.15%)
:arrow_down:

...main/java/org/apache/gobblin/util/HadoopUtils.java
29.93% <0.00%> (-0.66%)
:arrow_down:

...lin/hive/metastore/HiveMetaStoreBasedRegister.java

...blin/data/management/copy/RecursivePathFinder.java

...trumented/extractor/InstrumentedExtractorBase.java

.../troubleshooter/AutoTroubleshooterLogAppender.java

...obblin/runtime/NoopTaskEventMetadataGenerator.java

...adata/provider/DatasetAwareFsMetadataProvider.java

...ics/reporter/RecursiveScheduledMetricReporter.java

... and 1663 more

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 26f6930...14767b1. Read the comment docs.


ForEachElement Performance Degradation

Version
4.16.1
Describe the bug
After upgrading we have noticed our bots slowing down by a factor of 10.  we have pined it down to a bug in the constructor of the ForEachElement.  When you perform a linq query on a ForEachElements Actions the GetDependancies gets called several times rather than once.  For complicated dialogs this can make a 2 second turn call now be 20 seconds.
To Reproduce
Download and run the bot in the attached zip, you will see that GetDependancies is called 6 times to run only 4 dialog s.
Expected behavior
GetDependancies is only called once when performing a linq query on the actions.
Screenshots

Additional context
Here is our attempt at a fix
public ForEachElement(IEnumerable<Dialog> actions = null)
          : base(true)
        {
            _scope = new ActionScope(actions?.ToList());
        }

        public List<Dialog> Actions => this._scope.Actions;

Sample Bot Showing Bug
ForEachElementConstuctorBug.zip

Hi @karrimrabibiomni
Thank you for raising this issue. I've added your suggestion in this pull request: https://github.com/microsoft/botbuilder-dotnet/pull/6394


Added Darkmode

I tried a lot of different ways to do this but unfortunately had to adopt this method.Please Review it.
Thanks a lot for assigning me this issue. :

@chinmaym07 Looks good to me. A small change request. The social icons shown in the footer. Can you invert the color of those as well?

Just saw the deployed app. Love your work ❤ . Thank you.
Some small suggestions

If possible, let's make the switch size smaller.
The color of the title text Nice Blogs -> rgb(217, 194, 194) and switch should be the same when dark mode is enabled.
If possible, we can either try to use gatsby-plugin-dark-mode. It would be easier to maintain the state of the theme across all pages for dark mode and code would look more cleaner.
The blog page needs some tweak for dark mode. Refer follow screenshot:

Ohh Sorry.!! I will implement this and will update you.
Thanks a lot for your valuable feedback.!

@chinmaym07 No worries. Here is my little attempt to resolve this issue: https://github.com/abhaynikam/gatsby-nice-blog/commit/4a8600c27f51b72dc2ef65e8247f195d04c328b1
Feel free to cherry-pick the code. Also, I would love to add you as a contributor in the readme page. You can open another PR for same with link to your Profile. Happy Coding 😊
Also, instead of a switch, we can try adding sun/moon icon. Similar to: https://novela.narative.co/.

Closed in favor of #6


Installation problems

I'm trying to install this module through npm and I'm getting some errors
$  npm install -g browserslist
npm ERR! not a package /Users/JuanCaicedo/code/sd/neodarwin/browserslist
npm ERR! addLocal Could not install /Users/JuanCaicedo/code/sd/neodarwin/browserslist
npm ERR! Darwin 16.4.0
npm ERR! argv "/Users/JuanCaicedo/.nvm/versions/node/v4.4.5/bin/node" "/Users/JuanCaicedo/.nvm/versions/node/v4.4.5/bin/npm" "install" "-g" "browserslist"
npm ERR! node v4.4.5
npm ERR! npm  v2.15.5
npm ERR! path /var/folders/14/nlyqbhl13pd8gt58sxppfvdw0000gn/T/npm-26858-1cff5bb6/unpack-606835e2c786/package.json
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall open

npm ERR! enoent ENOENT: no such file or directory, open '/var/folders/14/nlyqbhl13pd8gt58sxppfvdw0000gn/T/npm-26858-1cff5bb6/unpack-606835e2c786/package.json'
npm ERR! enoent This is most likely not a problem with npm itself
npm ERR! enoent and is related to npm not being able to find a file.
npm ERR! enoent

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/JuanCaicedo/code/sd/neodarwin/npm-debug.log

Details

I don't get this error when installing other modules globally, ie $ npm install -g bower
I get this same error when installing browserslist locally
I'm using node v4.4.5 through nvm

Let me know if I can provide any more information

Do you have browserslist config? It could be related with https://github.com/ai/browserslist/issues/101
In this case just add browserslist manually to your package.json and run npm install.

@ai Yes I think that's the problem. I'll close this since it sounds like it's a known issue


date and CDN added

Changes

Created a new script.js in the js folder that has the getActualDate() function.

The file script.js it's commented line by line using pure JavaScript.

Only the index.html was changed.

Added the script.js to it through the script tag.

In the index.html in line 53 I changed the ID "fecha_diaria" to "timeToday".

And changed the local jQuery and Bootstrap files to CDN because if someone already get connected to a webpage that has jQuery and/or Bootstrap it won't have to download it again. Making your website slighty faster.

What I don't did

I don't replicate the changes in the index.html to all the others because maybe you don't like to use CDN or you don't like the ID name. So you can do it whoever you want or let me know and I will replicate those changes.

I wish you the best my friend!

Sorry I forget that I also changed the FontAwesome local path to CDN aswell!


Add --test-configuration flag for test configurations

This flag needs to be added so that mbedgt does not throw an error for the unknown flag.
See this PR - https://github.com/ARMmbed/mbed-os/pull/4795 - for more information about this flag's use in mbed OS.

Bump @bridadan

LGTM


When a server can't be connected to, try and show a more detailed error

we may need to try and ping the IP or domain to get a legit error

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @zach in #desktop-bugs.

I may have fixed this. Would need to test setup errors

Yeah I get a better error in console now:
[BlueBubblesApp] [Socket] -> Data: SocketException: Failed host lookup: 'fluffy-elephant-12.loca.lt' (OS Error: No address associated with hostname, errno = 7)


zopen audit  FSUM7329 <(command) and >(command) not implemented

11:37:33 RC=(4) [SYSA] bash-5.2$ zopen audit -v
VERBOSE: Obtained vulnerability json cache.
VERBOSE: Found all installed packages.
/hewitt/zopentools/zopen_repo/usr/local/bin/zopen-audit 154: FSUM7329 <(command) and >(command) not implemented
11:38:12 RC=(0) [SYSA] bash-5.2$ zopen audit
/hewitt/zopentools/zopen_repo/usr/local/bin/zopen-audit 154: FSUM7329 <(command) and >(command) not implemented

Looks like we've added some bash'isms into the script. FYI @KeplerBoyce

Should I remove the zopen_vulnerability.json file?
bash-5.2$ pwd;ls -l
/hewitt/zopentools/zopen_repo/var/cache/zopen
total 704
-rw-r--r--   1 @02858   @ISCICS1  320611 Jun  6 20:07 zopen_releases.json
-rw-r--r--   1 @02858   @ISCICS1     909 Jun  6 20:07 zopen_releases.timestamp
-rw-r--r--   1 @02858   @ISCICS1     906 Jun  7 07:59 zopen_releases.timestamp.current
-rw-r--r--   1 @02858   @ISCICS1    6561 Jun  6 11:39 zopen_vulnerability.json

You don't have to, we'll make a fix and issue a new release of meta

Installed  meta-main.20240607_174758.zos
bash-5.2$ zopen audit
/hewitt/zopentools/zopen_repo/usr/local/bin/zopen-audit 154: FSUM7329 <(command) and >(command) not implemented

Installed meta-main.20240611_213145
bash-5.2$ zopen audit
CVE Summary:
0 vulnerabilities (0 low, 0 moderate, 0 high, 0 critical)
Anything I need to reference in zopen_vulnerability.json or is the stdout what's important?
thanks

Installed meta-main.20240611_213145
bash-5.2$ zopen audit CVE Summary: 0 vulnerabilities (0 low, 0 moderate, 0 high, 0 critical)
Anything I need to reference in zopen_vulnerability.json or is the stdout what's important?
thanks

Great, yes stdout is what is important. zopen audit downloads and parses the  zopen_vulnerability.json.


Fixed locale switcher for RTL

Issue Reference
Fixed locale switcher for RTL

Checked and found that the language switcher issue in RTL view has been fixed. Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/e09d7fc9-7cfc-4915-9105-6a4289fe1c41


Fixes issue #5112

See #5112

Thank you @jun-sheaf , looks good. Just to make sure- the default output is the same as it was before, right? And it's changing only by specifying this new config flag?

Yes, the tests ensure this is the case.

That's great, thank you @jun-sheaf !


Build fails: file not found for module procedures

Building this program with Cargo fails with the error "file not found for module `procedures`"
$ cargo build
   Compiling scratchnative v0.1.0 (/home/nate/repos/scratchnative)
error[E0583]: file not found for module `procedures`
  --> src/parser/ast/mod.rs:11:1
   |
11 | mod procedures;
   | ^^^^^^^^^^^^^^^
   |
   = help: to create the module `procedures`, create file "src/parser/ast/procedures.rs" or "src/parser/ast/procedures/mod.rs"

error[E0425]: cannot find function `procedures_to_statement` in module `procedures`
   --> src/parser/ast/mod.rs:254:37
    |
254 |         "procedures" => procedures::procedures_to_statement(file, block.clone(), str_array[1]),
    |                                     ^^^^^^^^^^^^^^^^^^^^^^^ not found in `procedures`

Some errors have detailed explanations: E0425, E0583.
For more information about an error, try `rustc --explain E0425`.
error: could not compile `scratchnative` due to 2 previous errors

Output of cargo -V
$ cargo -V
cargo 1.66.1

I forgot to push a file. I fixed this locally and will eventually commit it

hey i'm still getting the same error. have you pushed the file yet?


separate work with time.Until for different go versions

problem with go 1.7: https://travis-ci.org/kamilsk/breaker/jobs/585952163

won't do cause support only go 1.11+


Translation for Eden savage + Small QoL update

Added defaulting to help screen when using translation script + a small notification about for unexpirienced users that they have to use the public key (not private one)
Also added timeline replacements (created via script) for eden savage

Thanks for doing this! The script changes are really good too.


How to End a workflow after a decisions

I have a decision that will return a true or false.  If the return is true I have a more task to run.  But for the false I just want to end the work flow completely.  It there a specific entry to put in for ending the work flow in the false case?
"false": [
]

@rabonja you can have a branching where false leads to no more tasks.  That will essentially end the workflow.  true branch of the decision could run a set of tasks.  equivalent of this in a programming lang will be something like
void workflow() {
   if(true) {
      //bunch of statements
   }
   //else do nothing which essentially returns out of the function
}


refactor: use useForm hook in Linear and Zoom extensions

Description
Refactor the Linear and Zoom form commands to use the new useForm hook.
Pending for a function to reset the values before opening the PR.
Screencast
No screencast as no functionality has changed.
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I get an error without setting a due date 😥


Refactor for loops

This refactors a for loop that had some computational inefficiencies.

You need to squash the commits as required by our pipeline (see https://docs.godotengine.org/en/latest/contributing/workflow/pr_workflow.html#modifying-a-pull-request)

How can we check the performance improvement?

I'm not necessarilly against manual optimizations, especially if they improve readability. This PR is a case of common subexpression elimination (I think), which is something I would be surprised the compiler can't work out by itself. All in all, why unconditionnaly compute compressed and uncompressed line width if we are trying to micro-optimize?
My suggestion would be, considering it a readability improvement rather than an optimization, to put each line width computation inside the relevant branches of the if.

Yeah agree optimizer will eat stuff like this for breakfast I think, there might not even be any difference to compiled code.
That said, I personally prefer the after version for a few reasons:

Makes it clearer to the reader and compiler what we are expecting
Works faster in debug
It is very easy to inadvertently break the optimized version in future by adding a function with side effects (for instance an overloaded array operator that checks for errors, COW etc etc)


Remove text-encoding dependency

Issues:
The repo use text-encoding lib in the utils https://github.com/uber/xviz/blob/master/modules/parser/src/utils/text-encoding.js#L18
text-encoding is deprecated and no longer maintained. Nodejs expose TextEncoder from util module in v8, and from global module in v11.
https://nodejs.org/api/util.html#util_class_util_textencoder
https://www.npmjs.com/package/text-encoding
Suggested fix:

Completely removing 'text-encoding` dependency;
Should we replace it with TextEncoder from nodejs? (is uber/xviz meant to function in node env?)

This might also be a question for loaders.gl, which currently includes this polyfill under node (and xviz uses loaders.gl).
Some vis.gl framework based apps support IE11, I assume that they probably still need this. We could update the instructions for building for IE11 to mention including this polyfill, and not include it by default. @tsherif

It looks like TextEncoder isn't supported in IE or Edge: https://caniuse.com/#feat=textencoder
(Edge surprises me, actually. I'll double check that on my Windows machine at home.)

We have
  "browser": {
    "text-encoding": false
  }

in package.json so it wouldn't work in these browsers regardless.

I can confirm that TextEncoder is not supported in Edge.


refactor(errors): move http errors to a generic error structure

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: C8mKfk066PnlSkAOei2n


Compatibility with recent taquito

Heya, and thank you!
It seems to work quite nicely :)
Small issue: it's not quite compatible with more recent versions of taquito. At best type errors (15.0), at worst runtime errors (15.1).

Ok, so the 15.1 issues seems to be a general issue with taquito - not sure how that only showed up now. Package managers...
Type error I get with 15.0:
Argument of type 'TzktExtension' is not assignable to parameter of type 'Extension | Extension[]'.
  Type 'TzktExtension' is not assignable to type 'Extension'.
    Types of property 'configureContext' are incompatible.
      Type '(context: import("[...]/node_modules/@tzkt/ext-taquito/node_modules/@taquito/taquito/dist/types/context").Context) => void' is not assignable to type '(context: import("[...]/node_modules/@taquito/taquito/dist/types/context").Context) => void'.
        Types of parameters 'context' and 'context' are incompatible.
          Type 'import("[...]/node_modules/@taquito/taquito/dist/types/context").Context' is not assignable to type 'import("[...]/node_modules/@tzkt/ext-taquito/node_modules/@taquito/taquito/dist/types/context").Context'.
            Types have separate declarations of a private property '_rpc'.


Deprecate allowing auto-replacement

Following these Zulip discussions, I realised that my deprecation script produced a deprecation syntax that did not allow for auto-replacement in Sébastien's #10185.
This PR fixes the deprecation statements, allowing self-correction: 119 times I replaced
@[deprecated xxx] --> @[deprecated].

bors r+
Thanks!
(Although I wonder why the code action kicks up in one case and not the other, as the message
`Matrix.linfty_op_nnnorm_def` has been deprecated, use `Matrix.linfty_opNNNorm_def` instead

is the same in both cases).

@sgouezel I think that it is a quirk of @[deprecated] **alias** that ignores the code action, but deprecated itself works as intended.  I can look into the deprecated alias implementation, but I have never worked before with code actions, so it may take a while.

Since we have a working solution, I'm not sure it's worth your time looking into the details of the implementation.


zos-cache's ssd in readonly filesystem

when trying to creat a vm :-

RuntimeError: [Errno 30] Read-only file system: '/opt/var/data/zrobot/zrobot_data/github.com/threefoldtech/0-templates/zerotier_client/20ec6397-5e7b-4cc4-9bd1-ab25fbef5e9d': 500 Server Error: INTERNAL SERVER ERROR for url: http://<IP_ADDRESS>:6600/services

the zos node in kds farm :-
node_ip=<IP_ADDRESS>
node_id=ac1f6b4571cc
after checking found out that sda (ssd) was remounted in new name sdal , which contain zos cache
/dev/sdal1 on /mnt/storagepools/sp_zos-cache type btrfs (ro,relatime,ssd,space_cache,subvolid=5,subvol=/)
/dev/sdal1 on /var/cache type btrfs (ro,relatime,ssd,space_cache,subvolid=257,subvol=/cache)
/dev/sdal1 on /var/log type btrfs (ro,relatime,ssd,space_cache,subvolid=259,subvol=/logs/log-20181015-0834)

demsg output

[776244.929004] blk_partition_remap: fail for partition 1
[776244.929008] btrfs_dev_stat_print_on_error: 3 callbacks suppressed
[776244.929011] BTRFS error (device sda1): bdev /dev/sdal1 errs: wr 164, rd 3119, flush 0, corrupt 0, gen 0
[776244.929037] BTRFS info (device sda1): no csum found for inode 160587 start 3497984
[776244.929039] blk_partition_remap: fail for partition 1
[776244.929042] BTRFS error (device sda1): bdev /dev/sdal1 errs: wr 164, rd 3120, flush 0, corrupt 0, gen 0

this happend after a test of disabling the disk then enable it again

echo 1 > /sys/block/sda/device/delete
then enabling the disk again by running this command :-
echo "- - -" | tee /sys/class/scsi_host/host*/scan

According to @zaibon the zos cache disk should not get picked up by zerodb.

verified :
with deploying s3 on a farm with single node with single ssd hard disk (which is used by zos cache ) to make sure that s3 will not pick it to deploy the 10 gb disk of minio vm and it's okay as this error showes that zos-cache it's not picked up
RuntimeError: Not enough free space for namespace creation with size 10 and type SSD,NVME


Linux Mint: Left menu bar unresponsive after closing folder.

Problem:
After closing a folder using File > Close folder the folder it's closed but the left menu bar it's unresponsive. Close (x) button it's unresponsive also. Weirdly that i got functionality of the top menu (File, Edit, View...etc). That functionality it's verrrrry limited and weird. when i go  "File > Open File" i can open the dialog window but chosen file it's not opened.  Same with open folder. Full screen menu can be executed by mouse click but key F11 it's not working to revert the action.
Steps to reproduce:
Open vscode with a folder from a previous session.
Click File > Close Folder
left menu bar must freaze at this point.
SO: Linux Mint 17.3
uname -a:
3.19.0-32-generic #37~14.04.1-Ubuntu SMP Thu Oct 22 09:41:40 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
vscode vers

full screen

linux mint version and frozen vscode window

@VictorIancu something in the developer console when this happens?

Hy coul you be a little more specific with developer console? you mean
F12 and inspect element?
On 03/11/2016 10:28 AM, Benjamin Pasero wrote:

@VictorIancu https://github.com/VictorIancu something in the
developer console when this happens?
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/vscode/issues/2512#issuecomment-195255026.

Help > Toggle Developer Tools > Console, I meant


Work well with libraries that add side-effects to the reducer signature

With libraries like redux-side-effect or redux-loop, there is a potential case where the memoized reducers won't fire when they should. Consider a case where the user can click a button and that button triggers an action like { type: 'BUTTON_CLICKED }. If that action has side effects in the reducer, then when the user clicks the button multiple times and the state hasn't changed, the memoized return value will be used and the side effects won't be triggered.

I've added memoized and non-memoized versions here, so users can choose which version they want or customize the memoization if necessary. There's still some room here to think about effect systems, but I'm considering this issue sidestepped for now.


feat(chart): prevent CRDs from being recycled

Description
With current implementation a chart uninstall also removes the CRDs, which triggers deletion of all created CRs.
Motivation and Context

Add a configuration to keep CRDs by default, instructing Helm to not delete such resources. Also add the possibility to specify extra annotations. This is especially interesting for Argo CD users, as Argo CD does not install charts the Helm way. In such a case, an extra annotation to prevent prunning resources is required.
Regression

How Has This Been Tested?

Case 1: keep all CRDs
helm chart template

Case 2: keep CSI VolumeSnaphots CRDs only
helm chart template --set crds.jaeger.keep=false

Case 3: keep Jaeger CRDs only
helm chart template --set crds.csi.volumeSnapshots.keep=false

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have added unit tests to cover my changes.

thanks @cmontemuino!
bors merge

@cmontemuino could you please re-run the helm doc generator, here's the failure from CI:
 ## Values

-| Key | Type | Default | Description |

-|-----|------|---------|-------------|

-| csi.volumeSnapshots.annotations | object | `{}` | Annotations to be added to all CRDs |

-| csi.volumeSnapshots.enabled | bool | `true` | Install Volume Snapshot CRDs |

-| csi.volumeSnapshots.keep | bool | `true` | Keep CRDs on chart uninstall |

-| jaeger.annotations | object | `{}` | Annotations to be added to all CRDs |

-| jaeger.enabled | bool | `true` | Install Jaeger CRDs |

-| jaeger.keep | bool | `true` | Keep CRDs on chart uninstall |

+| Key | Description | Default |

+|:----|:------------|:--------|

+| csi.volumeSnapshots.annotations | Annotations to be added to all CRDs |
{

}
 |

+| csi.volumeSnapshots.enabled | Install Volume Snapshot CRDs | `true` |

+| csi.volumeSnapshots.keep | Keep CRDs on chart uninstall | `true` |

+| jaeger.annotations | Annotations to be added to all CRDs |
{

}
 |

+| jaeger.enabled | Install Jaeger CRDs | `true` |

+| jaeger.keep | Keep CRDs on chart uninstall | `true` |

bors try

Thanks @cmontemuino. Are we good to merge now?

bors merge


spike: twitter queries for improved fairness and usefulness

Description
We currently define a configuration in config/twitter.json that defines a list of queries the validators as miners to mine.  The list of queries is somewhat "random", and we can improve said list.  Currently it focuses on crypto and web3 related topics, but this can be expanded on.
Some queries, like crypto analysis, often don't have many tweets associated w/ them, and it is unfair to miners who are asked to mine said query, as they won't return as many tweets as other, more broad queries.  Furthermore, we need to better define what the downstream use case for the twitter data is - currently it just sits on validator hardware.
Ideas
It was proposed that perhaps the list of queries is dynamic, pulled from Twitter itself, perhaps the trending topics section, etc.  This would both ensure volume AND usefulness, as we are harvesting data with the most volume / relevance.
Imagine we create an admin app through which we generate / update the list of queries, publish it to a public spot, same format as the current file. Validators pull this instead of the file in the repo to use for synthetic tweets.
Then also, using the protocol API we already created, validators post the raw synthetic tweet responses. We dedupe the tweets themselves and store them, and we keep stats on the miners and validators that returned them (identifying miners and validators by their hotkey / coldkeys)

Quick hotfix for queries has been merged here https://github.com/masa-finance/masa-bittensor/pull/286

What about scraping something like https://trends24.in/, get the trends with major volume about crypto and and them to the query list?

I found a little hard to find a crypto specific trends list, seems like we would need to fetch an entire trending topics list and filter by "is crypto related"

Imagine we create an admin app through which we generate / update the list of queries, publish it to a public spot, same format as the current file. Validators pull this instead of the file in the repo to use for synthetic tweets.
If this isn't an automated process, lets say we go and manually update the list, I would not build something new, a simple PR can do the same with the current list and we don't spend time on new tooling

I will think more about the "trending topics" today...
As it stands, our current list of queries typically have a lot of volume (2,000+ tweets a day, 100 within the first hour of the new day).
While pulling from a trending list is feasible, it's is also the "highest hanging" fruit, and there are other, easier ways to increase volume + miner demand.  From easiest to hardest (and highest priority to lowest):

Removing the current queries that tend to not return AS many tweets, i.e. crypto pump, crypto dump... to name a few
Increase the count of tweets asked for in volume checking (currently 100).  We've been using this spreadsheet to calculate how much we are "stressing" the protocol node.  I would aim for at least 5x capacity, meaning each node has to successfully run 5 credentials to keep up with validator demand.
Similar to above, look at increasing the amount of miners called for each volume check (currently at 10).  This number works in conjunction with the tweet count.
Can also look at cadence, though I would leave this untouched if possible, as it's currently set to 1/100th of a tempo which is an easy number to work with.
Dynamic queries.  If we find that given the increased volume from efforts above, our static queries don't return enough tweets, this then builds the case for implementing dynamic querying, where we specifically look for trending tweets where there is enough volume.

Being discussed in https://github.com/masa-finance/masa-bittensor/issues/295

Moving this to in review as MIP #2 (#295 ) captures this discussion!


gRPC service and client cannot pass a header list

Description:
In gRPC, users can specify headers as:
map<string|string[]>

However, the current implementation cannot pass a header list. Instead, it only sends the last item of the header list as the specific header.
Steps to reproduce:
Implement a gRPC service that passes a header list.
Affected Versions:
SL Beta 1

Part of this error happens because HttpCarbonMessage in the HTTP package does not support adding headers; it only supports setting headers. Therefore, HTTP native implementation should also need to improve.


Add k8s labels

This adds the app.kubernetes.io/{name,version,instance} labels to the kubernetes yaml and extends the existing version bump functionality in the makefile to update the version when a release is made.

@c-nixon can we rebase + squash these into one commit before this gets merged?


Support DropwizardClientRule with custom ObjectMapper config

Hi,
As far as I can tell, it's not possible to apply any custom configuration to the ObjectMapper used by DropwizardClientRule's FakeApplication (since the application and its environment are kept private). Is this by design?
I'd like to register a Module and set some SerializationFeatures on the ObjectMapper used - without that, the JSON the client under test sends cannot be deserialized.
It looks like at the moment I'm forced to run a full integration test with DropwizardAppRule (which is a shame, as it's slower and much more difficult to manipulate). Is there any alternative, or if not, is there any argument against exposing the application and/or environment from DropwizardClientRule?
Thanks,
Rowan

I submitted https://github.com/dropwizard/dropwizard/pull/1446 to address this


Sliding Boundaries Archive Heatmap Boundaries [BUG]

pyribs version: 0.4.0
Python version: 3.9
Operating System: Windows

Description
Heyho,
I am trying to plot a heatmap of a sliding boundaries archive that shows the boundaries. Therefore, I set the parameter boundary_lw to a value large than 0.
The following picture shows the result of the function call (Fig. 1):

But what I would like to have the following (Fig. 2):

Steps to Reproduce
The function call I have used is:
from ribs.visualize import sliding_boundaries_archive_heatmap
import matplotlib.pyplot as plt
fig = plt.figure(figsize=(8, 6))
sliding_boundaries_archive_heatmap(archive, boundary_lw=0.5, ax=ax1)
plt.show()

I can not share the archive I am using.
Potential Fix
I found the following solution while looking at the code.
The current implementation of plotting the boundaries is the following:
if boundary_lw > 0.0:
    ax.vlines(x_boundary,
              lower_bounds[0],
              upper_bounds[0],
              color='k',
              linewidth=boundary_lw)
    ax.hlines(y_boundary,
              lower_bounds[1],
              upper_bounds[1],
              color='k',
              linewidth=boundary_lw)

However, to my understanding of the code, we would need to switch the indices. That is, to plot the boundaries correctly, we need the lower and upper bounds of the other dimension. Hence, the code should be:
if boundary_lw > 0.0:
    ax.vlines(x_boundary,
              lower_bounds[1],
              upper_bounds[1],
              color='k',
              linewidth=boundary_lw)
    ax.hlines(y_boundary,
              lower_bounds[0],
              upper_bounds[0],
              color='k',
              linewidth=boundary_lw)

I have used this code to produce Figure 2.
I am not sure if I am just using the code incorrectly or if this is a bug. I would be happy to hear feedback.

Hi @LennartPurucker, thank you for reporting this bug! It indeed looks like the boundary lines are being drawn incorrectly. We are addressing this in #280.

Hi @btjanaka, the pull request looks great, thank you!
This solves the bug and my problem. Hence, I am closing the issue.


the accuracy of implement

I ran this code using the entire Food101 dataset and found that the model trained with the original code achieved only around 0.06 accuracy on the total validation set. However, when I used the entire training set of Food101 with learning rates of 0.005, 0.001, and 0.0005, the best accuracy I obtained was approximately 0.906. Has anyone else encountered a similar issue?

Hello @zqy-happy, I'm unable to understand the issue here. Could you please give a lot more context following the issue template for us to help you?

Hello @zqy-happy, I'm unable to understand the issue here. Could you please give a lot more context following the issue template for us to help you?

Thank you! I'm glad to receive your reply. So, first of all, I ran the entire code from the GitHub repository "https://github.com/huggingface/peft/tree/main/examples/image_classification/image_classification_peft_lora.ipynb". The dataset part was loaded as shown in the project using the code "dataset = load_dataset("food101", split="train[:5000]")". I obtained results that were essentially the same as those in the repository.
However,I decided to reload the entire Food101 dataset using the following code: "dataset = load_dataset("food101", cache_dir="./datasets")" and "train_ds = dataset['train'] ,
val_ds = dataset['validation']". After reloading the dataset, I performed testing again on the fine-tuned model using the code "trainer.evaluate(val_ds)". The results I obtained were around 0.06.
Furthermore, I used similar code but with the complete Food101 dataset and experimented with different learning rates such as 0.005, 0.001, and 0.0005. The following are the results I obtained:
Learning Rate: 0.005
Evaluation Loss: 0.4101414680480957
Evaluation Accuracy: 0.8912079207920792
Evaluation Runtime: 332.0338 seconds
Evaluation Samples per Second: 76.046
Evaluation Steps per Second: 0.596
Epoch: 29.0
Learning Rate: 0.001
Evaluation Loss: 0.3431929647922516
Evaluation Accuracy: 0.9047524752475248
Evaluation Runtime: 209.8496 seconds
Evaluation Samples per Second: 120.324
Evaluation Steps per Second: 0.944
Epoch: 34.0
Learning Rate: 0.0005
Evaluation Loss: 0.3281923234462738
Evaluation Accuracy: 0.9061782178217822
Evaluation Runtime: 224.6711 seconds
Evaluation Samples per Second: 112.387
Evaluation Steps per Second: 0.881
Epoch: 36.0
I would like to know if there were any issues during this process.

cc @sayakpaul who has a lot more context on this

I am unable to reproduce this issue.
Could you maybe provide a Colab Notebook for this? You can use this one: https://github.com/huggingface/peft/blob/main/examples/image_classification/image_classification_peft_lora.ipynb

I am unable to reproduce this issue.
Could you maybe provide a Colab Notebook for this? You can use this one: https://github.com/huggingface/peft/blob/main/examples/image_classification/image_classification_peft_lora.ipynb

I apologize for the delay in my response. I am not familiar with Colab, but I have a repository where I execute code on a Linux platform with GPU, which can provide more detailed information.
Moreover, I realize that I may not have articulated my intention clearly！ Simply put, I would like to inquire about the baseline results achieved by others, including yourself, when fine-tuning a Vision Transformer (ViT) using Lora on the entire 'Food101' dataset or any other datasets.
Thank you.

If you run the Colab notebook, it should give at least >80% accuracy (top-1). So, for now, I am going to close the issue.


Proxy banning

There needs to be a possibility to ban players from the network.

The RPC API will also be expanded to support this of course.


Refactor functional tests

Split the functional Components Integration tests out of Aspire.Hosting.Tests into a new EndToEnd project.
Start the test AppHost project using dotnet run instead of running the DistributedAppBuilder inline in the tests. This allows for the tests to run on a separate machine than where the tests are built. - Along with this, we need to pass information from the test AppHost to the test about its endpoints, and give it a signal to "stop". Instead of trying to find a way to CTRL+C cross-platform, read from stdin for a "Stop" command.

The main new code is in EndToEnd.Tests TestProgramFixture.cs.
Microsoft Reviewers: Open in CodeFlow

Over the weekend I was playing with the idea of disabling certain components/routes in the test project depending on what connection strings were populated and then using the AppHost to selectively enable the various dependent services. The idea being that for any given integration test you could narrow the scope of the containers that you were launching and improve performance.
I think it would be good to tackle this at the same time or as a fast follow on this PR.

Over the weekend I was playing with the idea of disabling certain components/routes in the test project depending on what connection strings were populated and then using the AppHost to selectively enable the various dependent services. The idea being that for any given integration test you could narrow the scope of the containers that you were launching and improve performance.
I think it would be good to tackle this at the same time or as a fast follow on this PR.

This is tracked by https://github.com/dotnet/aspire/issues/1704. (cc @ReubenBond @DamianEdwards).


fix: support linewrap on windows

Thanks!
@orisano or @unasuke Could you comment before and after screenshots what's fixed on windows by this changes.

I confirmed it on my windows 10. (powershell.exe and cmd.exe)

When I try this on iTerm2, a mouse cursor is disappear.
master branch

this branch

The original branch before rebasing works fine.

umm..., I can not reproduce in my environment.
Is the mouse cursor disappearing? Not the terminal cursor?

sorry. That's a terminal cursor.

OK. Could you rebase this branch to master and push it? Maybe it's a mistake of my rebasing.

rebased

Hmm... A terminal cursor is properly displayed.


Enhancement/export internal packages

Export internal packages so they can be reused outside of the boilerplate.

build github.com/vardius/go-api-boilerplate/cmd/user: cannot load github.com/vardius/go-api-boilerplate/internal/errors: no matching versions for query "latest"

Would you mind taking a look ? Going through the commits I can't see any existing leftover and this is github.com/vardius/go-api-boilerplate/pkg/errors everywhere. Re-run the tests Travis is still looking for a non-existing path, seems like cached or something.
P.S. Please try to proceed with the PRs yourself when you have time since the two masters are too different from each other for me to be able to complete it.


Where to group what function

I'm working on ripping Imgsrc.ru, it has the following steps:
Get URLs to albums
Each page contains 50 URLs that can easily be sanitized into the album URL. (What method do I use for this?)
Each album URL contains a number of sub-URLs. (What method do I use for this?)
Each sub-URL do I use the following methods to download all images from the site.
getURLsFromPage()
downloadURL()
This is the part that works just fine.

Each page contains 50 URLs that can easily be sanitized into the album URL. (What method do I use for this?)

getUrlsFromPage

Each album URL contains a number of sub-URLs. (What method do I use for this?)

AFAIk there is no built in method, you'll have to write your own. You can check the myhentaicomics ripper to see how I did it

Each sub-URL do I use the following methods to download all images from the site.
getURLsFromPage()
downloadURL()
This is the part that works just fine.

I didn't quite any stand why you meant there could you reword it?


Here Updated Audiocraft install tutorial - Enjoy

In my tutorials I use GitHub readme files for install instructions
Updated instructions GitHub readme file of my below tutorial
It installs latest xformers development version as well
You will get this message on windows Error caught was: No module named 'triton'
Since triton optimizer not supported on windows but it still works perfect so just ignore it
Enjoy
Moreover I have 1 click auto installer : https://www.patreon.com/posts/ai-music-auto-84334460

Tutorial
Audiocraft / MusicGen - AMAZING Text-To-Music AI Model By Facebook | Tutorial | Better Than MusicLM

Did you really put this behind a paywall and expect people to accept it?

Did you really put this behind a paywall and expect people to accept it?
Are you serious?

you didn't bother to check any of the links did you
this video shows how to install and use https://www.youtube.com/watch?v=v-YpvPkhdO4
this readme file as well https://github.com/FurkanGozukara/Stable-Diffusion/blob/main/Tutorials/AI-Music-Generation-Audiocraft-Tutorial.md
that patreon thing is just automation of what have been shown

That what I was talking about.
The one click installer is nice, and what I look for because I throw it in my bookmarks so I can replicate it on different machines.
Throwing that behind a paywall irritates me, because now I have to replicate what you've done and offer the one click installer for free.

That what I was talking about.
The one click installer is nice, and what I look for because I throw it in my bookmarks so I can replicate it on different machines. Throwing that behind a paywall irritates me, because now I have to replicate what you've done and offer the one click installer for free.

well if i were funded by a corporation i would gladly release it for free
but youtube revenue is nothing so i am making an incentive for people to donate

if anyone still needs it, i've set up my own repo with MusicGen and AudioGen
here is the colab: https://colab.research.google.com/github/camenduru/MusicGen-colab/blob/main/MusicGen_ClownOfMadness_plus_colab.ipynb
here is a repo: https://github.com/GrandaddyShmax/audiocraft_plus
Do let me know if you face any issues

Thank you very much Furka. I can't run it using the instruction from official repo but your instruction really made it work. Thanks again

if anyone still needs it, i've set up my own repo with MusicGen and AudioGen here is the colab: https://colab.research.google.com/github/camenduru/MusicGen-colab/blob/main/MusicGen_ClownOfMadness_plus_colab.ipynb here is a repo: https://github.com/GrandaddyShmax/audiocraft_plus
Do let me know if you face any issues

Amazing!
I want to make a very simple and intuitive interface only for sound effect generation (AudioGen). Should I just edit the app.py file or do I need to edit other files as well?

Yeah i think just editing the app.py would be enough

Thank you very much Furka. I can't run it using the instruction from official repo but your instruction really made it work. Thanks again

ty so much for the comment

Did you really put this behind a paywall and expect people to accept it?

That what I was talking about.
The one click installer is nice, and what I look for because I throw it in my bookmarks so I can replicate it on different machines. Throwing that behind a paywall irritates me, because now I have to replicate what you've done and offer the one click installer for free.

Actually there already are one-click-installers: https://github.com/rsxdalv/one-click-installers-tts it's more than just AudioCraft, but it can be narrowed back down to only audiocraft.

@X13X I just released my own version of the launcher for free with a friendly menu to install, update, run, and uninstall... no need to worry about python versions since it will use a conda env to isolate everything.
I am planning to create another launcher for audiocraft-plus aswell
https://github.com/deffcolony/ai-toolbox/tree/main/audiocraft


game startup slowdown

Hi!
First I really want to thank you for this amazing mod. I am using it to create new empty scenes and set up transition logics, and it is doing its job amazingly, saving me lots of time to invent the wheel myself. One small concern is that the game startup is slowing down quite noticeably, even though I am only adding entirely empty scenes (with doors only). Do you know the root cause of slowing down?

Not sure what would be causing the slowdowns, but i can add some timers into the modloading process and see what part is running the slowest.
It may also be dependent on the system specs. When i have some time, i can send you a custom version of WeaverCore, and you can then send me what the times are on your computer.

Yeah let's do this when you have time. This will be very helpful

Sorry for taking a while. I had to get some other important things done first before I could take a look at this. In the latest development version of WeaverCore, I added some performance logs into the WeaverCore mod loading process. Each step of the loading process will print to ModLog the amount of time that has passed since WeaverCore first started loading, followed by a message. Here's an example log from my computer. WeaverCore starts loading at Timestamp 0 seconds, and finishes loading at timestamp 18.499 seconds.
ModLog.txt
One key thing I found that influences the length of time is the compression method used to pack your mod. When building your mod, there are three main compression methods you can pick from:

NONE - This applies no compression algorithm to the asset bundles, resulting in a much larger mod DLL (Usually 10x more than LZ4)
LZMA - This compression algorithm results in the lowest mod dll file size, but results in the longest load times
LZ4 - The recommended algorithm to pick, as it offers a good balance between file size and load times.

On my machine, the NONE compression option resulted in the fastest load times. LZMA resulted in the slowest load times, and LZ4 was somewhere in between. Here's my results with Inferno King Grimm and WeaverCore:

No Compression = 18.100 seconds
LZ4 = 18.581 seconds
LZMA = 20.171 seconds

When you can, be sure to use the latest develop version of WeaverCore and test the load times of your mod.

Thanks for adding the perf log. Attached is my modlog. It took 60s to load on my computer and I don't see what's the bottleneck, looks like just generally slower at each step
ModLog.txt

I think I got the issue fixed. The problem was that when WeaverCore was doing it's patching and initialization processes, it was looking through every DLL it could find, and attempting to find WeaverCore attributes, assetbundles, and other WeaverCore specific things in them. Since only a few DLLs actually use WeaverCore related things, this results in a massive waste of time. For partically large DLLS, like HKMirror and MMHOOK_Assembly-CSharp, this can add tons of time to the startup.
[INFO]:12.07 - Patching assembly MMHOOK_Assembly-CSharp
[INFO]:13 - Finished patching assembly MMHOOK_Assembly-CSharp
[INFO]:9.26 - Patching assembly HKMirror
[INFO]:11.904 - Finished patching assembly HKMirror
I was able to get this fixed through the use of a function I never heard about till now called Assembly.GetReferencedAssemblies, I am able to use this to see if a DLL depends on WeaverCore before doing a bunch of checking over it. I just pushed my changes to the latest develop branch, so you should be able to test it yourself.
I tested it myself, and even with a few mods installed, this brought down my startup time from 18.581 seconds to 10.478 seconds. This should also make the log file smaller, because WeaverCore doesn't need to do as much wasted processing as before.

Thanks for the quick fix! You are super helpful! Looking forward to the an official release so that I can upgrade. :)

I updated to latest dev to test your fix but my mod is not loading anymore :)
ModLog.txt

Course it's crashing right where I try to use the Assembly.GetReferencedAssemblies function. I'll see if I can come up with a workaround.

I updated to latest dev to test your fix but my mod is not loading anymore :) ModLog.txt

This should be fixed now. I created my own version of the Assembly.GetReferencedAssemblies function that shouldn't crash this time.

I updated to latest dev to test your fix but my mod is not loading anymore :) ModLog.txt

This should be fixed now. I created my own version of the Assembly.GetReferencedAssemblies function that shouldn't crash this time.

Still doesn't load for me though. Error log looks the same. Attached is my source code and dll. Pls take a look when you have time. Thanks!
dll.zip
PantheonOfZotesScenes-master.zip

The log shouldn't be the same though, since I'm using a different method for checking assembly dependencies. Are you sure the log is identical to the previous one? I can come up with a different method if that's still the case

I changed the load method to something that doesn't use Assembly.GetReferencedAssemblies. Could you try testing it again with the latest WeaverCore commit?

Now it's working and the loading time is just 10s! Whatever you did it seems all working fine! Thanks so much for working on this.
I do have a last question about backward compatibility. So if I build my mod with this dev branch (or a future public release with the fix), and someone else want to load my mod, do they have to upgrade to a newer version of weavercore? If they don't, what happens ? (Slow loading time? Or just failure of loading?) This is kind of a reasonable question because Chinese community tends to not use mod installer and sometimes have outdated dependencies.

So if I build my mod with this dev branch (or a future public release with the fix), and someone else want to load my mod, do they have to upgrade to a newer version of weavercore? If they don't, what happens ? (Slow loading time? Or just failure of loading?) This is kind of a reasonable question because Chinese community tends to not use mod installer and sometimes have outdated dependencies.

It depends if your mod is using features/changes that are exclusive to the dev branch. I haven't really done a whole lot of changes to the dev branch, so it hopefully shouldn't break with the current release version, but there's no guarantee. If you want to test it out yourself, you can download the release version of WeaverCore, and attempt to load your mod (which was built with the dev version) with the release version. The only major changes I have made so far in the dev branch was adding Soul Totems, adding PersistentBoolItem/PersistentIntItem classes, and some other bug fixes. https://github.com/nickc01/WeaverCore/commits/develop/?since=2024-01-20&until=2024-02-19

Yeah I just tried a release version of WeaverCore and it still loads (just slow loading time though). Thanks again :)


fix renderJsonErrorMessage -  error variable was overriding it self

In the function renderJsonErrorMessage error was defined as parameter and array. So it was overriding itself.

Good catch. Thank you.


To prevent nodes that did not succeed at one time, the calculated estimateResponse is the lowest

What is the purpose of the change
To prevent nodes that did not succeed at one time, the calculated estimateResponse is the lowest
Brief changelog
dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/loadbalance/ShortestResponseLoadBalance.java

Codecov Report

Merging #6201 into master will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #6201      +/-   ##
============================================
- Coverage     60.89%   60.82%   -0.07%
+ Complexity      493      492       -1
============================================
  Files           999      999
  Lines         39898    39898
  Branches       5744     5745       +1
============================================
- Hits          24295    24269      -26
- Misses        12910    12934      +24
- Partials       2693     2695       +2

Impacted Files
Coverage Δ
Complexity Δ

...uster/loadbalance/ShortestResponseLoadBalance.java
84.61% <100.00%> (ø)
0.00 <0.00> (ø)

...rg/apache/dubbo/remoting/utils/PayloadDropper.java
15.38% <0.00%> (-30.77%)
0.00% <0.00%> (ø%)

...bo/config/event/listener/LoggingEventListener.java
37.50% <0.00%> (-25.00%)
0.00% <0.00%> (ø%)

...ava/org/apache/dubbo/config/DubboShutdownHook.java
63.82% <0.00%> (-10.64%)
0.00% <0.00%> (ø%)

...e/dubbo/remoting/transport/netty/NettyChannel.java
55.68% <0.00%> (-4.55%)
20.00% <0.00%> (-1.00%)

.../src/main/java/org/apache/dubbo/rpc/RpcStatus.java
70.23% <0.00%> (-3.58%)
0.00% <0.00%> (ø%)

.../remoting/transport/netty4/NettyClientHandler.java
57.62% <0.00%> (-1.70%)
0.00% <0.00%> (ø%)

...pache/dubbo/registry/support/AbstractRegistry.java
78.06% <0.00%> (-1.49%)
0.00% <0.00%> (ø%)

...g/apache/dubbo/registry/consul/ConsulRegistry.java
62.11% <0.00%> (-0.63%)
30.00% <0.00%> (ø%)

.../apache/dubbo/config/bootstrap/DubboBootstrap.java
52.55% <0.00%> (-0.38%)
0.00% <0.00%> (ø%)

... and 2 more

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 1df50c1...52ffded. Read the comment docs.

If like that, the nodes that have not been selected will never be selected, because all nodes initialled with zero succeeded average elapsed. Maybe we need to find a more effective way.


Add basic KUTTL tests for Cinder Operator

This PR adds 2 very basic kuttl tests for cinder - to deploy and cleanup.

With some tweaks to the install_yaml targets, the tests are passing. I've noticed that we are just testing the Cinder CR, but there are others defined (CinderAPI, CinderBackup..., see https://github.com/openstack-k8s-operators/cinder-operator/tree/master/api/bases). We can either just provide some basic tests and let the dev team expand as needed or make them more complete here, both are valid solutions. At this point we should probably ask for review from the core team and get their feedback

/retest-required


Reactions print literal message

I'm running a weechat instance on a FreeBSD server inside tmux.
When I type a reaction, e.g. 3+:smile:, the output in Slack is +3:smile: as a new message, instead of reacting to the 3rd most recent message.

Weird, I just tried that exact incanation and it shows up as a reaction for me. Are you sure you're running an up-to-date version?

I also tested this and it worked for me. Make sure you are typing only this on the line (no spaces on either side)

Maybe we should eat any trailing spaces at least?

Actually trailing space works with the regex (just tested in REPL)


Update web server to http4s

We have been using akka-http for a while.  As a tech project, would like to explore changing to http4s as part of standardizing the stack.
Opening this as a discussion to explore ideas.  Here are the major parts that would need to be moved:

JSON - there is custom json validation that is now outdated.  Should move to circe, care must be made here to ensure request and response payloads remain backward compatible

Authentication - the signing process is separate, which I hope would make using auth in http4s straight forward (but have not delved deeply as of yet)

Services - the existing Akka HTTP stuff is rather thin, all the business logic resides behind services.  Like with JSON, remaining backward compatible is key.

Tests - we have a lot of tests using the akka-http test kit.  This is definitely a lot of work

Considerations:

We have hundreds of automated E2E tests, so staying backward compatible should be covered by running those tests

As a tech project, would like to explore changing to http4s as part of standardizing the stack.

@pauljamescleary What's the benefit of moving to http4s? How does this standardize the stack?
TBC, I'm not opposed, I just want to understand the rationale.

Good questions. The json stuff originally implemented is dated and there are much better ways to do it now that would simplify the code. That’s the gateway drug to be honest.
We have moved to using other typelevel libraries to where typelevel libraries are the norm. Those include fs2, cats, cats effect and pureconfig. Moving to http4s and circe rounds out our typelevel migration. The effect is consistency, akka-http is a bit of an odd ball at the moment.  We have to do an unsafeToFuture in it which is ok but would be preferable to be cats effect all the way down.
Some random thoughts there as I’m mobile right now, but happy to have the discussion.

Yeah, it would be great to not have to worry about processing the effectual actions. The fewer unsafe* calls that the devs have to make, the fewer will be made unnecessarily.
I do like to remain pragmatic, though, and this would be a tough to do incrementally.  Maybe it's time to consider a major release that makes some major foundational shifts - http4s, consolidated portal, Scala 3, improved CI pipeline, etc.
Since our current release isn't seeing many functional changes (though more are in the pipeline) it may be a good time to split off, and start work on VinylDNS vNext. We can port any functionality added to the 0.x version to the vNext version as they trickle in.
I would propose strictly foundational changes for vNext.  We can build new and exciting features on that clean foundation after initial release. I think we have a good start for the foundation changes from several of your issues, we can wrap those into a roadmap/execution plan.
Something to consider. Maybe @vinyldns/maintainers can chat and come up with a strategy.

@raghu999 That's a good idea.  I'll start a thread over there


Update deselected wiki icon, add hover state

Fixed viewport default and hovered icons:

Resizable viewport default and hovered icons:

Lotto: that looks worse
Hydrox: no outline makes it harder to read
deathbeam: no outline looks better. its not fixing the core problem, but at least its less contrasting with everything else around it

@devLotto @Hydrox6 Would updating this PR to use an icon with a more subtle outline be an acceptable solution? (that is, white-tinted, but not 100% white like the image currently uses)

I've updated the images to have a soft white outline, which roughly matches the design of nearby UI elements such as the minimap and minimap orbs.

@devLotto Does the updated image address your concern adequately?

The updated icon looks nice, maybe the hovered one should be a bit less lit up so the beveled details on the Wiki text are still there.

I still think the hover image has too much detail lost, is this better?

I don't feel very strongly on this, but I would think that having a more vibrant hover state would be helpful so people know that there are available actions when it is hovered.


Templates are not being picked up

My .mustache templates are not being copied to the ebin when I use rebar3 :-(
I opened a SO question about it - but so far no resolution:
http://stackoverflow.com/questions/36231976/how-can-i-include-a-mustache-file-in-erlang-release
P.S.: erlydt has a plugin that might be useful for porting over to this tool - it basically compiles stuff in priv/templates into *_dtl modules automatically:
https://github.com/tsloughter/rebar3_erlydtl_plugin

I wanted to put mine in the priv directory but couldn't figure out how to get that to work either. I tried setting the templates_dir mustache environment variable to my priv directory but I kept getting an error.


RxSwift 6+ Compatibility Not Released to Cocoapods

@pauljohanneskraft  I see there was a PR for RxSwift 6+ compatibility. Are there any plans to release this version to Cocoapods?
https://github.com/quickbirdstudios/XCoordinator/pull/203

Bump on this thread too, it'll be great to get this released

Sorry, I missed this one - it is not released as version 2.1.0.

Sorry, I missed this one - it is not released as version 2.1.0.

Thanks so much for this @pauljohanneskraft!


Getter method is called for no reason and callstack is growing while execution is stoped at breakpoint

Godot version:
v3.2.beta.custom_build.406dac191
OS/device including version:
Windows 10
Issue description:
Getter method is called on a variable that is not being accessed when I call any method of that class

Minimal reproduction project:
GDScriptBug.zip

Can reproduce in 3.2.2.stable.
The issue does not occur if you assign a value to b at declaration (eg. var b = 1).

I am not quite shure weither it is related. But if I set a breakpoint in any getter function, I get the same behavior as soon a it is called. Even if the variables are assigned with a value at declaration. But there are no problems occuring without the breakpoint.
3.2.3.stable on Windows 10

I am not quite shure weither it is related. But if I set a breakpoint in any getter function, I get the same behavior as soon a it is called. Even if the variables are assigned with a value at declaration. But there are no problems occuring without the breakpoint.
3.2.3.stable on Windows 10

It occures because debugger try to show all variables ; then try to call getter of c ; but c is defined in term of b / a ; with b having to value (null) at this time. If you put "pass" at line 12 (after "b = 1" which is line 11) and put breakpoint on it, there is no recurrent call to c getter.
Thus there is some debugger bug that retry continuously to get c, but should stop and show something like "unreachable value" instead in variables panel.


alternative mirror

Hey, thanks a lot for the interesting work!
Is it possible to provide an alternative to pan baidu for datasets and pretrained_models?
Without a Chinese mobile phone number, it is impossible for me to create an account and download the data.

Sorry about that, and I will provide the data through google drive in this week.

Awesome, thank you!


small bugfix in readCalibration

In line 90 is says "aze" and should be "gze".
Serial.print("sensor.aze = ");
to
Serial.print("sensor.gze = ");
Best regards

Thanks, will fix it asap

done!

Tanks a lot for your work :)


[WIP] Added strict params and return types in generated code

Codecov Report

Merging #1888 (9f3c29f) into master (e6c147f) will decrease coverage by 2.95%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1888      +/-   ##
============================================
- Coverage     87.61%   84.66%   -2.96%
  Complexity     7814     7814
============================================
  Files           227      227
  Lines         21161    21156       -5
============================================
- Hits          18541    17912     -629
- Misses         2620     3244     +624

Flag
Coverage Δ

5-max
84.66% <ø> (-2.96%)
:arrow_down:

7.4
84.66% <ø> (-2.96%)
:arrow_down:

agnostic
66.99% <ø> (ø)

mysql
?

pgsql
?

sqlite
66.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/Propel/Generator/Builder/Om/ObjectBuilder.php
93.63% <ø> (-0.49%)
:arrow_down:

src/Propel/Generator/Reverse/PgsqlSchemaParser.php
0.00% <0.00%> (-86.62%)
:arrow_down:

src/Propel/Generator/Reverse/MysqlSchemaParser.php
0.00% <0.00%> (-86.24%)
:arrow_down:

src/Propel/Runtime/Adapter/Pdo/PgsqlAdapter.php
2.94% <0.00%> (-50.00%)
:arrow_down:

src/Propel/Runtime/Adapter/Pdo/MysqlAdapter.php
41.30% <0.00%> (-36.96%)
:arrow_down:

.../Propel/Generator/Reverse/AbstractSchemaParser.php
44.44% <0.00%> (-22.23%)
:arrow_down:

.../ActiveQuery/QueryExecutor/InsertQueryExecutor.php
59.37% <0.00%> (-21.88%)
:arrow_down:

src/Propel/Generator/Platform/PgsqlPlatform.php
81.36% <0.00%> (-13.67%)
:arrow_down:

...e/ActiveQuery/SqlBuilder/DeleteQuerySqlBuilder.php
87.87% <0.00%> (-12.13%)
:arrow_down:

src/Propel/Generator/Platform/MysqlPlatform.php
84.82% <0.00%> (-9.29%)
:arrow_down:

... and 14 more

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 e6c147f...9f3c29f. Read the comment docs.


Discussion: Should errors be accessible? If so, how?

Bringing this into a seperate issue (https://github.com/lino-levan/proposal-try-catch-expressions/issues/1#issuecomment-1644415916)

What happens to the error that the failable expression throws? How would it be accessed?

I'm of the opinion that the error that the expression throws should not be accessible, but potentially the syntax could incorporate that.

I think catch necessarily has to be a function.
const result = try throw 5 catch five => five + 1

In this case, the catch function recovers from the thrown number and resolves by incrementing it, result becomes 6. Presumably, this isn't too controversial.
But what if the catch function throws? How would a try-catch expression compose.
const result = try throw 5 catch five => five + throw 6

I'm of the opinion that the error that the expression throws should not be accessible

That prompts me to different syntax:
try result = (_ => throw 4)()
console.log(result) //undefined

Maybe it could be function optionally?
I believe that goes against JavaScript's long held values of being spec-compliant with baseball.  A "catch" expression that never catches sounds like blasphemy.

I'm not understanding this proposed syntax, could you elaborate?

const functionThatThrows = _ => throw 4
try result = functionThatThrows()
console.log(result) //undefined

in this case, a "try" decleration calls the function, but doesn't allow it to throw, instead resolving immediately to undefined.

I believe that goes against JavaScript's long held values of being spec-compliant with baseball.  A "catch" expression that never catches sounds like blasphemy.

Javascript already has
try {
    throw 0
} catch {
    // lost the error
}

as valid syntax, so there is prior art to this.

in this case, a "try" decleration calls the function, but doesn't allow it to throw, instead resolving immediately to undefined.

Maybe? I'd be more of a fan of
let result = try functionThatThrows() // result of function or undefined/null

otherwise we wouldn't be able to define a const expression.

Javascript already has
try {
    throw 0
} catch {
    // lost the error
}

as valid syntax, so there is prior art to this.

That is reassuring.

let result = try functionThatThrows() // result of function or undefined/null

So in the absence of an explicit catch expression, catch undefined is implied?

That is reassuring.

👍

So in the absence of an explicit catch expression, catch undefined is implied?

This sounds reasonable. I wonder if we shouldn't do catch null instead?

I'd argue the default should be the thrown value.
Back to the main topic of discussion, I consider it important to be able to access the thrown value (not just errors). They provide meaningful information and without a syntax, debugging remains an issue. You would have to switch to try catch statements to get to the cause.

I'd argue the default should be the thrown value.

Super interesting. That actually makes a lot of sense. +1 from me. This would allow for error handling using other, already existing, syntax.